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.

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

remove the -Z no-opt flag #13649

Closed
thestinger opened this issue Apr 21, 2014 · 1 comment · Fixed by #17419
Closed

remove the -Z no-opt flag #13649

thestinger opened this issue Apr 21, 2014 · 1 comment · Fixed by #17419
Labels
C-cleanup Category: PRs that clean code up or issues documenting cleanup. E-easy Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue.

Comments

@thestinger
Copy link
Contributor

This seems very pointless because you can just not pass optimization flags. It might date from a time when optimizations had frontend implications.

@huonw
Copy link
Member

huonw commented Apr 22, 2014

I use -Z no-opt to temporarily avoid the expense of -O for specific targets makefile projects (including/especially mozilla/rust) i.e. make foo RUSTFLAGS='-Z no-opt'.

This can be equally well simulated if #7493 was fixed to allow multiple --opt-levels (since RUSTFLAGS='--opt-level=0' works equally well).

(This is just a minor thing, but it is often easier to use the makefile targets than to work out the correct manual rustc invocation.)

wsddn pushed a commit to wsddn/rust that referenced this issue May 21, 2014
Manishearth pushed a commit to Manishearth/rust that referenced this issue Nov 23, 2022
Improve grammar of architecture.md

Corrects a few grammar mistakes I found while reading the documentation 😄
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-cleanup Category: PRs that clean code up or issues documenting cleanup. E-easy Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue.
Projects
None yet
2 participants