<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">Hey Chandler,<div><br></div><div>Backed out in r142323. Will recommit when I find a home for this parser that doesn't require this dependency.</div><div><br></div><div>Cheers,</div><div>Lang.</div><div><br><div><div>On Oct 17, 2011, at 4:30 PM, Chandler Carruth wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><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>
_______________________________________________<br>llvm-commits mailing list<br><a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits<br></blockquote></div><br></div></body></html>