<div class="gmail_quote">On Mon, Oct 17, 2011 at 4:17 PM, Rafael Ávila de Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="im">On 10/17/2011 06:26 PM, Rafael Ávila de Espíndola wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
looks like this broke the build:<br>
<br>
<br>
Linking CXX executable ../../bin/llvm-as<br>
../../lib/libLLVMAsmParser.a(<u></u>LLParser.cpp.o):/home/<u></u>espindola/llvm/llvm/lib/<u></u>AsmParser/LLParser.cpp:<u></u>function<br>
llvm::LLParser::<u></u>ParseTargetDefinition(): error: undefined reference to<br>
'llvm::TargetData::<u></u>parseSpecifier(llvm::<u></u>StringRef, llvm::TargetData*)'<br>
clang-3: error: linker command failed with exit code 1 (use -v to see<br>
invocation)<br>
</blockquote>
<br></div>
It looks like all that was missing was the attached patch.<br>
<br>
Chandler, is it OK with the original patch?</blockquote><div><br></div><div>As discussed in IRC, yes this is sufficient for the link command to succeed, but it feels completely wrong for the LL parsing to depend on the target library. I think this is a more fundamental layering issue. Currently AsmParser only depends on Support and Core, and that seems reasonable. I don't think something not in the VMCore should be required in order to parse the IR. </div>
</div>