Skip to content

No std#62

Open
droundy wants to merge 2 commits into
masterfrom
no_std
Open

No std#62
droundy wants to merge 2 commits into
masterfrom
no_std

Conversation

@droundy

@droundy droundy commented Mar 1, 2025

Copy link
Copy Markdown
Owner

just to run CI

@bushrat011899

bushrat011899 commented Mar 1, 2025

Copy link
Copy Markdown
Contributor

I believe lines 114 and 115 in .github/workflows/rust.yml need to be updated to use -Cinstrument-coverage instead of -Zprofile:

-          RUSTFLAGS: '-Zprofile -Ccodegen-units=1 -Cinline-threshold=0 -Clink-dead-code -Coverflow-checks=off -Cpanic=abort -Zpanic_abort_tests'
-          RUSTDOCFLAGS: '-Zprofile -Ccodegen-units=1 -Cinline-threshold=0 -Clink-dead-code -Coverflow-checks=off -Cpanic=abort -Zpanic_abort_tests'
+          RUSTFLAGS: '-Cinstrument-coverage -Ccodegen-units=1 -Cinline-threshold=0 -Clink-dead-code -Coverflow-checks=off -Cpanic=abort -Zpanic_abort_tests'
+          RUSTDOCFLAGS: '-Cinstrument-coverage -Ccodegen-units=1 -Cinline-threshold=0 -Clink-dead-code -Coverflow-checks=off -Cpanic=abort -Zpanic_abort_tests'

I haven't personally tested it, but based on this comment on the tracking issue for -Zprofile it sounds like -Cinstrument-coverage is the stable replacement.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants