Skip to content
This repository was archived by the owner on Feb 5, 2019. It is now read-only.

use -O0 when building release LLVM on sparc #61

Merged
merged 1 commit into from
Feb 21, 2017
Merged

use -O0 when building release LLVM on sparc #61

merged 1 commit into from
Feb 21, 2017

Conversation

binarycrusader
Copy link

The bits produced by gcc above -O0 when building rust's private copy of llvm on
sparc result in either versions of llvm-tblgen that consistently segfault or
produce otherwise unuseable code. This does not affect cross-compiles for some
reason. Investigation with gcc upstream is in progress.

The bits produced by gcc above -O0 when building rust's private copy of llvm on
sparc result in either versions of llvm-tblgen that consistently segfault or
produce otherwise unuseable code.  This does not affect cross-compiles for some
reason.  Investigation with gcc upstream is in progress.
@alexcrichton
Copy link
Member

Weird! Is the resulting LLVM usable? I figured that an LLVM at O0 would be unbearably slow...

@binarycrusader
Copy link
Author

It's usable with -O0, just really slow (but not quite as slow as a normal debug build). But considering that it doesn't work at all otherwise, there's not much choice. Investigation is being done with one of the gcc maintainers.

It's possible that some other combination of options might work here, or that I could potentially limit the application of -O0 to specific files. If you'd prefer to wait until I can refine this further to minimize the performance impact, that's perfectly fine with me.

While I'm waiting on a response from the gcc maintainer, I'm investigating other combinations of options to see if they can workaround the issue. But the change I proposed here is the only one that works reliably so far.

@alexcrichton
Copy link
Member

Ok sounds reasonable to me!

@alexcrichton alexcrichton merged commit 4af19ce into rust-lang:rust-llvm-2016-10-29 Feb 21, 2017
TimNN pushed a commit to TimNN/llvm that referenced this pull request Apr 17, 2017
TimNN pushed a commit that referenced this pull request Apr 23, 2017
use -O0 when building release LLVM on sparc
arielb1 pushed a commit that referenced this pull request Jun 27, 2017
use -O0 when building release LLVM on sparc
TimNN added a commit that referenced this pull request Jul 20, 2017
use -O0 when building release LLVM on sparc
TimNN added a commit that referenced this pull request Jul 20, 2017
use -O0 when building release LLVM on sparc
alexcrichton pushed a commit that referenced this pull request Jul 30, 2017
use -O0 when building release LLVM on sparc
alexcrichton pushed a commit that referenced this pull request Aug 4, 2017
use -O0 when building release LLVM on sparc
alexcrichton pushed a commit that referenced this pull request Aug 15, 2017
use -O0 when building release LLVM on sparc
alexcrichton pushed a commit that referenced this pull request Oct 10, 2017
use -O0 when building release LLVM on sparc
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants