-
Notifications
You must be signed in to change notification settings - Fork 13.4k
Avoid loading the whole gdb debug scripts section. #28427
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
Conversation
a02e627
to
e11af7b
Compare
@@ -8,7 +8,7 @@ | |||
// option. This file may not be copied, modified, or distributed | |||
// except according to those terms. | |||
|
|||
// compile-flags: -C no-prepopulate-passes | |||
// compile-flags: -C no-prepopulate-passes --crate-type=lib |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can this be specified with #![crate_type = "lib"]
instead?
e11af7b
to
6ba1af3
Compare
Done. |
@bors: r+ 6ba1af34f458bc2bc4a1d2ac9d69e502bb013b2d |
⌛ Testing commit 6ba1af3 with merge a92408f... |
💔 Test failed - auto-mac-32-opt |
Crap. Is there any way to view the build dir (ie On Thu, Sep 17, 2015 at 12:45 AM, bors [email protected] wrote:
|
Nah unfortunately they're all blown away after each run |
Oh I don't need it. I just realized why it failed: the debug script is disabled on windows and on (drum roll please) Mac OSX. Derp. |
6ba1af3
to
f0e109f
Compare
Fixed. |
Looks like the failure on travis may be legit |
This is so LLVM isn't forced to load every byte of it. Also sets the alignment of the load. Adds a test for the debug script section.
f0e109f
to
9a24025
Compare
It was the result of a new codegen test. Fixed. |
…richton This is so LLVM isn't forced to load every byte of it. Also sets the alignment of the load. Adds a test for the debug script section. r? @alexcrichton
This is so LLVM isn't forced to load every byte of it. Also sets the alignment of
the load. Adds a test for the debug script section.
r? @alexcrichton