Skip to content

Commit

Permalink
auto merge of #9658 : michaelwoerister/rust/namespace_fixes, r=jdm
Browse files Browse the repository at this point in the history
This should fix some outstanding namespace issues. It also fixes an issue with LLVM metadata uniquing that caused an LLVM assertion when compiling libstd.

One thing to keep in mind is that the `-O` flag and the debug info flags are essentially incompatible. It may work but I would not consider this in any way supported at the moment. On the other hand, there is also good news: With the changes in this PR I am able to compile all of rust with extra-debug-info:
```
make RUSTFLAGS_STAGE2='-Zextra-debug-info' check
```
compiles the whole thing without warning and passes the whole test suite (given that `configure` is run with `--disable-optimize`). That's kind of nice `:)` Still, I'm reluctant to automatically close the related issues (#9167, #9190, #9268) without confirmation from the openers. I'll post to the individual threads once this gets merged.
  • Loading branch information
bors committed Oct 8, 2013
2 parents c919629 + 85deeea commit e87205c
Show file tree
Hide file tree
Showing 4 changed files with 109 additions and 222 deletions.
3 changes: 2 additions & 1 deletion src/librustc/lib/llvm.rs
Expand Up @@ -1968,7 +1968,8 @@ pub mod llvm {
DerivedFrom: DIType,
Elements: DIArray,
RunTimeLang: c_uint,
VTableHolder: ValueRef)
VTableHolder: ValueRef,
UniqueId: *c_char)
-> DICompositeType;

#[fast_ffi]
Expand Down
5 changes: 0 additions & 5 deletions src/librustc/middle/trans/base.rs
Expand Up @@ -3119,11 +3119,6 @@ pub fn trans_crate(sess: session::Session,
symbol_hasher,
link_meta,
analysis.reachable);

if ccx.sess.opts.debuginfo {
debuginfo::initialize(ccx, &crate);
}

{
let _icx = push_ctxt("text");
trans_mod(ccx, &crate.module);
Expand Down

0 comments on commit e87205c

Please sign in to comment.