Skip to content
This repository has been archived by the owner. It is now read-only.

use -O0 when building release LLVM on sparc #61

Merged
merged 1 commit into from Feb 21, 2017

Conversation

Projects
None yet
2 participants
@binarycrusader
Copy link

binarycrusader commented Feb 21, 2017

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.

use -O0 when building release LLVM on sparc
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

This comment has been minimized.

Copy link
Member

alexcrichton commented Feb 21, 2017

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

@binarycrusader

This comment has been minimized.

Copy link
Author

binarycrusader commented Feb 21, 2017

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

This comment has been minimized.

Copy link
Member

alexcrichton commented Feb 21, 2017

Ok sounds reasonable to me!

@alexcrichton alexcrichton merged commit 4af19ce into rust-lang:rust-llvm-2016-10-29 Feb 21, 2017

TimNN added a commit to TimNN/llvm that referenced this pull request Apr 17, 2017

TimNN added a commit that referenced this pull request Apr 23, 2017

Merge pull request #61 from binarycrusader/rust-llvm-2016-10-29
use -O0 when building release LLVM on sparc

arielb1 added a commit that referenced this pull request Jun 27, 2017

Merge pull request #61 from binarycrusader/rust-llvm-2016-10-29
use -O0 when building release LLVM on sparc

TimNN added a commit that referenced this pull request Jul 20, 2017

Merge pull request #61 from binarycrusader/rust-llvm-2016-10-29
use -O0 when building release LLVM on sparc

TimNN added a commit that referenced this pull request Jul 20, 2017

Merge pull request #61 from binarycrusader/rust-llvm-2016-10-29
use -O0 when building release LLVM on sparc

alexcrichton added a commit that referenced this pull request Jul 30, 2017

Merge pull request #61 from binarycrusader/rust-llvm-2016-10-29
use -O0 when building release LLVM on sparc

alexcrichton added a commit that referenced this pull request Aug 4, 2017

Merge pull request #61 from binarycrusader/rust-llvm-2016-10-29
use -O0 when building release LLVM on sparc

alexcrichton added a commit that referenced this pull request Aug 15, 2017

Merge pull request #61 from binarycrusader/rust-llvm-2016-10-29
use -O0 when building release LLVM on sparc

alexcrichton added a commit that referenced this pull request Oct 10, 2017

Merge pull request #61 from binarycrusader/rust-llvm-2016-10-29
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.
You can’t perform that action at this time.