<p dir="ltr">Parsing twice is actually a time versus memory tradeoff.</p>
<p dir="ltr">This patch LGTM.</p>
<p dir="ltr">At some point we have to figure out a way of lazy loading metadata and not copying metadata that is not used, but that is not now.</p>
<p dir="ltr">Cheers,<br>
Rafael</p>
<div class="gmail_quote">On Mar 9, 2016 12:24 AM, "Sean Silva" <<a href="mailto:chisophugis@gmail.com">chisophugis@gmail.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">silvas created this revision.<br>
silvas added a reviewer: rafael.<br>
silvas added subscribers: llvm-commits, Bigcheese, ruiu.<br>
Herald added a subscriber: joker.eph.<br>
<br>
At the very least we hit<br>
<br>
    Assertion failed: (((Flags & RF_HaveUnmaterializedMetadata) || Node->isResolved()) && "Unexpected unresolved node"), function MapMetadataImpl, file /Users/Sean/pg/llvm/lib/Transforms/Utils/ValueMapper.cpp, line 375.<br>
<br>
on the included test case.<br>
<br>
We currently do things like parse the module twice to keep the<br>
implementation minimal. I think it makes sense to add start with eager<br>
loading for similar reasons.<br>
<br>
<a href="http://reviews.llvm.org/D17982" rel="noreferrer" target="_blank">http://reviews.llvm.org/D17982</a><br>
<br>
Files:<br>
  ELF/SymbolTable.cpp<br>
  test/ELF/lto/metadata.ll<br>
<br>
Index: test/ELF/lto/metadata.ll<br>
===================================================================<br>
--- /dev/null<br>
+++ test/ELF/lto/metadata.ll<br>
@@ -0,0 +1,13 @@<br>
+; REQUIRES: x86<br>
+; RUN: llvm-as %s -o %t1.o<br>
+; RUN: ld.lld -m elf_x86_64 %t1.o %t1.o -o %t.so -shared<br>
+<br>
+target datalayout = "e-m:e-i64:64-f80:128-n8:16:32:64-S128"<br>
+target triple = "x86_64-unknown-linux-gnu"<br>
+<br>
+define weak void @foo(i32* %p) {<br>
+  store i32 5, i32* %p, align 4, !tbaa !0<br>
+  ret void<br>
+}<br>
+<br>
+!0 = !{!"Simple C/C++ TBAA"}<br>
Index: ELF/SymbolTable.cpp<br>
===================================================================<br>
--- ELF/SymbolTable.cpp<br>
+++ ELF/SymbolTable.cpp<br>
@@ -132,7 +132,7 @@<br>
       MemoryBuffer::getMemBuffer(F.MB, false);<br>
   std::unique_ptr<Module> M =<br>
       check(getLazyBitcodeModule(std::move(Buffer), Context,<br>
-                                 /*ShouldLazyLoadMetadata*/ true));<br>
+                                 /*ShouldLazyLoadMetadata*/ false));<br>
   std::vector<GlobalValue *> Keep;<br>
   for (SymbolBody *B : F.getSymbols()) {<br>
     if (B->repl() != B)<br>
<br>
<br>
</blockquote></div>