Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

-Z randomize-layout compiler flag is not documented in the unstable book #130462

Closed
fasterthanlime opened this issue Sep 17, 2024 · 1 comment · Fixed by #133777
Closed

-Z randomize-layout compiler flag is not documented in the unstable book #130462

fasterthanlime opened this issue Sep 17, 2024 · 1 comment · Fixed by #133777
Labels
-Zrandomize-layout Unstable option: Randomize the layout of types. A-docs Area: Documentation for any part of the project, including the compiler, standard library, and tools T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Comments

@fasterthanlime
Copy link
Contributor

Location

https://doc.rust-lang.org/nightly/unstable-book/compiler-flags.html

Summary

I expected to find a page for -Z randomize-layout, as described in #106764, but I didn't find any.
@fasterthanlime fasterthanlime added the A-docs Area: Documentation for any part of the project, including the compiler, standard library, and tools label Sep 17, 2024
@rustbot rustbot added the needs-triage This issue may need triage. Remove it if it has been sufficiently triaged. label Sep 17, 2024
@ChrisDenton
Copy link
Member

A meta note is that we need to do a better job of making sure unstable flags are documented. This isn't the first time this has happened and I suspect there are a number of other unintentionally undocumented flags.

Ideally we'd have CI check for it but I understand that may be more complex than it sounds.

@jieyouxu jieyouxu added T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. -Zrandomize-layout Unstable option: Randomize the layout of types. and removed -Zrandomize-layout Unstable option: Randomize the layout of types. needs-triage This issue may need triage. Remove it if it has been sufficiently triaged. labels Sep 17, 2024
@the8472 the8472 mentioned this issue Dec 3, 2024
Merged
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this issue Dec 3, 2024
@matthiaskrgr
Rollup merge of rust-lang#133777 - the8472:document-randomize-layout,…
d00ed3e 
… r=jieyouxu

document -Zrandomize-layout in the unstable book

tracking issue: rust-lang#106764
fixes rust-lang#130462
jhpratt added a commit to jhpratt/rust that referenced this issue Dec 3, 2024
@jhpratt
Rollup merge of rust-lang#133777 - the8472:document-randomize-layout,…
c541427 
… r=jieyouxu

document -Zrandomize-layout in the unstable book

tracking issue: rust-lang#106764
fixes rust-lang#130462
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this issue Dec 3, 2024
@matthiaskrgr
Rollup merge of rust-lang#133777 - the8472:document-randomize-layout,…
767a5aa 
… r=jieyouxu

document -Zrandomize-layout in the unstable book

tracking issue: rust-lang#106764
fixes rust-lang#130462
@bors bors closed this as completed in 6323109 Dec 3, 2024
rust-timer added a commit to rust-lang-ci/rust that referenced this issue Dec 3, 2024
@rust-timer
Unrolled build for rust-lang#133777
649e6bf 
Rollup merge of rust-lang#133777 - the8472:document-randomize-layout, r=jieyouxu

document -Zrandomize-layout in the unstable book

tracking issue: rust-lang#106764
fixes rust-lang#130462
fasterthanlime added a commit to bearcove/rubicon that referenced this issue Dec 6, 2024
@fasterthanlime
Link to the new -Z randomize-layout doc page
97aede1 
cf. rust-lang/rust#133777
and rust-lang/rust#130462
thanks to @the8472 !
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
-Zrandomize-layout Unstable option: Randomize the layout of types. A-docs Area: Documentation for any part of the project, including the compiler, standard library, and tools T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

document -Zrandomize-layout in the unstable book the8472/rust
4 participants
@ChrisDenton @fasterthanlime @jieyouxu @rustbot