<div class="gmail_quote">On Wed, Jan 25, 2012 at 12:15 PM, Sebastian Pop <span dir="ltr"><<a href="mailto:spop@codeaurora.org">spop@codeaurora.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div id=":2ri">Hi Chandler,<br>
<div class="im"><br>
On Wed, Jan 25, 2012 at 5:01 AM, Chandler Carruth <<a href="mailto:chandlerc@gmail.com" class="cremed">chandlerc@gmail.com</a>> wrote:<br>
> Author: chandlerc<br>
> Date: Wed Jan 25 05:01:57 2012<br>
> New Revision: 148950<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=148950&view=rev" target="_blank" class="cremed">http://llvm.org/viewvc/llvm-project?rev=148950&view=rev</a><br>
</div>[...]<br>
<div class="im">> -<br>
> -  if (!TC && !Arch.compare ("hexagon")) {<br>
> -    llvm::Triple TCTriple (getTriple());<br>
> -    TC = new toolchains::Hexagon_TC (getDriver(), TCTriple);<br>
> -  }<br>
<br>
</div>Removing this code without adding it back to getToolChain breaks<br>
our toolchain.  Please fix.</div></blockquote></div><br><div>Doh, sorry, I just missed this when scanning the host info code. Fixed in r148977.</div><div><br></div><div>However, I'd point out that not a single test or build bot caught this. =/ At the very least we really need tests for the hexagon toolchain. You can lee test/Driver/linux-ld.c and test/Driver/linux-header-search.cpp for examples of how you can test a toolchain very thoroughly in the regression test suite. Without these types of tests, the ongoing refactoring work in the driver is very likely to break your config again. =/ A build bot for the hexagon stuff would also be amazing, but I understand the logistical challenges there.</div>