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

configure --enable-debug --enable-optimize is not enabling optimizations #24405

Closed
pnkfelix opened this issue Apr 14, 2015 · 5 comments · Fixed by #24408
Closed

configure --enable-debug --enable-optimize is not enabling optimizations #24405

pnkfelix opened this issue Apr 14, 2015 · 5 comments · Fixed by #24408

Comments

@pnkfelix
Copy link
Member

Running configure --enable-debug --enable-optimize is not enabling optimizations.

Concretely, running ../configure --enable-debug --enable-optimize yields:

CFG_ENABLE_DEBUG     := 1
...
CFG_DISABLE_OPTIMIZE := 1
CFG_DISABLE_OPTIMIZE_CXX := 1
...
CFG_CONFIGURE_ARGS   := --enable-debug --enable-optimize

And, just out of curiosity, I tried switching the order of args; that made little difference:

CFG_ENABLE_DEBUG     := 1
...
CFG_DISABLE_OPTIMIZE := 1
CFG_DISABLE_OPTIMIZE_CXX := 1
...
CFG_CONFIGURE_ARGS   := --enable-optimize --enable-debug
@pnkfelix
Copy link
Member Author

(the fix for this looks easy... i wonder why no one else has been flummoxed by the resulting build times though...)

@richo
Copy link
Contributor

richo commented Apr 14, 2015

poking at a fix before I crash, mostly because I want to build rust overnight

richo added a commit to richo/rust that referenced this issue Apr 14, 2015
@petrochenkov
Copy link
Contributor

How much time does it take to build and test stage2 with configure --enable-debug. Days, I guess? I haven't tried yet.

@pnkfelix
Copy link
Member Author

@petrochenkov certainly hours for me, but not a full day

@pnkfelix
Copy link
Member Author

(Note that this bug is only discussing --enable-optimize and --enable-debug, and that is all that is addressed by #24408 ... but there are other flags being overridden by the relevant block of code in configure that probably have a similar problem. I will address that separately (filed as #24416); I want to land #24408 asap.)

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 a pull request may close this issue.

3 participants