Skip to content

Revert "feat: add frame-pointers profile option (#16742)"#16999

Merged
weihanglo merged 2 commits into
rust-lang:masterfrom
ehuss:revert-frame-pointers-master
May 15, 2026
Merged

Revert "feat: add frame-pointers profile option (#16742)"#16999
weihanglo merged 2 commits into
rust-lang:masterfrom
ehuss:revert-frame-pointers-master

Conversation

@ehuss
Copy link
Copy Markdown
Contributor

@ehuss ehuss commented May 15, 2026

This reverts commit 3c95ccc, reversing changes made to da53118.

A concern was raised about the force-off option which may be confusing because it doesn't force them "off", it turns off the "force on" option. Force-off is the default and can change per target.

ehuss added 2 commits May 15, 2026 08:33
This reverts commit 3c95ccc, reversing
changes made to da53118.

A concern was raised about the force-off option which may be confusing
because it doesn't force them "off", it turns off the "force on" option.
Force-off is the default and can change per target.
This is being reverted for now.
@rustbot rustbot added A-build-execution Area: anything dealing with executing the compiler A-documenting-cargo-itself Area: Cargo's documentation A-manifest Area: Cargo.toml issues labels May 15, 2026
@rustbot
Copy link
Copy Markdown
Collaborator

rustbot commented May 15, 2026

r? @weihanglo

rustbot has assigned @weihanglo.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: @ehuss, @epage, @weihanglo
  • @ehuss, @epage, @weihanglo expanded to ehuss, epage, weihanglo
  • Random selection from epage, weihanglo

@rustbot rustbot added A-profiles Area: profiles S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels May 15, 2026
@ehuss
Copy link
Copy Markdown
Contributor Author

ehuss commented May 15, 2026

I think the version-bump check is unhappy about downgrading the version. I think it should probably resolve itself after this merges.

I confirmed that there were no other breaking changes made, so I don't think it is necessary to go to 0.14.

@weihanglo weihanglo added this pull request to the merge queue May 15, 2026
Merged via the queue into rust-lang:master with commit 4d1f984 May 15, 2026
30 of 31 checks passed
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label May 15, 2026
rust-bors Bot pushed a commit to rust-lang/rust that referenced this pull request May 16, 2026
Update cargo submodule

7 commits in a343accce8526b128adc517d33348573d22920a3..4d1f984518c77fad6eeef4f40153b002a659e662
2026-05-08 22:41:35 +0000 to 2026-05-15 17:12:01 +0000
- Revert "feat: add frame-pointers profile option (rust-lang/cargo#16742)" (rust-lang/cargo#16999)
- CI: update to macos-15 (rust-lang/cargo#16996)
- fix(clean): respect `build.target` config for `clean -p` (rust-lang/cargo#16988)
- fix(diag): Consolidate verify/run diagnostics passes  (rust-lang/cargo#16989)
- fix(diag): Track Cargo diagnostic warning/error count like is done for rustc (rust-lang/cargo#16981)
- fix: suggest 'fmt' when user types 'cargo rustfmt' (rust-lang/cargo#16985)
- Rebuild when -Zpublic-dependency changes (rust-lang/cargo#16965)

r? ghost
rust-bors Bot pushed a commit to rust-lang/rust that referenced this pull request May 16, 2026
Update cargo submodule

7 commits in a343accce8526b128adc517d33348573d22920a3..4d1f984518c77fad6eeef4f40153b002a659e662
2026-05-08 22:41:35 +0000 to 2026-05-15 17:12:01 +0000
- Revert "feat: add frame-pointers profile option (rust-lang/cargo#16742)" (rust-lang/cargo#16999)
- CI: update to macos-15 (rust-lang/cargo#16996)
- fix(clean): respect `build.target` config for `clean -p` (rust-lang/cargo#16988)
- fix(diag): Consolidate verify/run diagnostics passes  (rust-lang/cargo#16989)
- fix(diag): Track Cargo diagnostic warning/error count like is done for rustc (rust-lang/cargo#16981)
- fix: suggest 'fmt' when user types 'cargo rustfmt' (rust-lang/cargo#16985)
- Rebuild when -Zpublic-dependency changes (rust-lang/cargo#16965)

r? ghost
rust-bors Bot pushed a commit to rust-lang/rust that referenced this pull request May 17, 2026
Update cargo submodule

7 commits in a343accce8526b128adc517d33348573d22920a3..4d1f984518c77fad6eeef4f40153b002a659e662
2026-05-08 22:41:35 +0000 to 2026-05-15 17:12:01 +0000
- Revert "feat: add frame-pointers profile option (rust-lang/cargo#16742)" (rust-lang/cargo#16999)
- CI: update to macos-15 (rust-lang/cargo#16996)
- fix(clean): respect `build.target` config for `clean -p` (rust-lang/cargo#16988)
- fix(diag): Consolidate verify/run diagnostics passes  (rust-lang/cargo#16989)
- fix(diag): Track Cargo diagnostic warning/error count like is done for rustc (rust-lang/cargo#16981)
- fix: suggest 'fmt' when user types 'cargo rustfmt' (rust-lang/cargo#16985)
- Rebuild when -Zpublic-dependency changes (rust-lang/cargo#16965)

r? ghost
@rustbot rustbot added this to the 1.97.0 milestone May 17, 2026
pull Bot pushed a commit to Kokoro2336/rust-analyzer that referenced this pull request May 17, 2026
Update cargo submodule

7 commits in a343accce8526b128adc517d33348573d22920a3..4d1f984518c77fad6eeef4f40153b002a659e662
2026-05-08 22:41:35 +0000 to 2026-05-15 17:12:01 +0000
- Revert "feat: add frame-pointers profile option (rust-lang/cargo#16742)" (rust-lang/cargo#16999)
- CI: update to macos-15 (rust-lang/cargo#16996)
- fix(clean): respect `build.target` config for `clean -p` (rust-lang/cargo#16988)
- fix(diag): Consolidate verify/run diagnostics passes  (rust-lang/cargo#16989)
- fix(diag): Track Cargo diagnostic warning/error count like is done for rustc (rust-lang/cargo#16981)
- fix: suggest 'fmt' when user types 'cargo rustfmt' (rust-lang/cargo#16985)
- Rebuild when -Zpublic-dependency changes (rust-lang/cargo#16965)

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

Labels

A-build-execution Area: anything dealing with executing the compiler A-documenting-cargo-itself Area: Cargo's documentation A-manifest Area: Cargo.toml issues A-profiles Area: profiles

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants