Skip to content

Remove -Z no-opt flag.#17419

Merged
bors merged 1 commit intorust-lang:masterfrom
colrdavidson:remove_no_opt
Sep 22, 2014
Merged

Remove -Z no-opt flag.#17419
bors merged 1 commit intorust-lang:masterfrom
colrdavidson:remove_no_opt

Conversation

@colrdavidson
Copy link
Contributor

@colrdavidson colrdavidson commented Sep 21, 2014

Closes #13649

@rust-highfive
Copy link
Contributor

rust-highfive commented Sep 21, 2014

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @nikomatsakis (or someone else) soon.

@huonw
Copy link
Contributor

huonw commented Sep 21, 2014

I'm not in favour of this just yet, at least until we have (a plan for) an equivalent way to disable optimisations with the Rust makeflies. If you want to build something unoptimised the choices are make <target> RUSTFLAGS='-Z no-opt', or extracting the correct compiler invocations and removing the -O flags, the former is definitely easier. (Reconfiguring with --disable-optimize doesn't quite do the same thing, since that would require redoing the whole bootstrap without optimisation, while RUSTFLAGS allows for a quicker edit-compile cycle.)

If, e.g., the Rust makefiles removed the -O flag when passed NO_OPT, or set --opt-level to n when given OPT_LEVEL=n then this is good. (This handling all seems to be around line 111 of mk/main.mk, if you feel like tackling it.)

@colrdavidson
Copy link
Contributor Author

colrdavidson commented Sep 21, 2014

It looks like this can already be achieved with CFG_DISABLE_OPTIMIZE=1 make.

bors added a commit that referenced this pull request Sep 21, 2014
@bors
auto merge of #17419 : anchovieshat/rust/remove_no_opt, r=cmr
4e5b626 
Closes #13649
@bors bors closed this Sep 22, 2014
@bors bors merged commit b2b0737 into rust-lang:master Sep 22, 2014
lnicola pushed a commit to lnicola/rust that referenced this pull request Jun 23, 2024
@bors
Auto merge of rust-lang#17419 - ishanjain28:filter_builtin_macro_expa…
e871775 
…nsion, r=Veykril

Filter builtin macro expansion

This PR adds a filter on the types of built in macros that are allowed to be expanded.

Currently, This list of allowed macros contains, `stringify, cfg, core_panic, std_panic, concat, concat_bytes, include, include_str, include_bytes, env` and `option_env`.

Fixes rust-lang#14177
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

remove the -Z no-opt flag

4 participants

@colrdavidson @rust-highfive @huonw @bors