<div class="gmail_quote">On Mon, Feb 20, 2012 at 8:39 AM, Joerg Sonnenberger <span dir="ltr"><<a href="mailto:joerg@britannica.bec.de">joerg@britannica.bec.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi all,<br>
this is the minimally intrusive fix for the target-prefixed toolchain<br>
support. This is a partial revert of r148839, fixed in r149083, reverted<br>
and re-broken in r149337. Given that's been 3 weeks now with a proper<br>
solution, I think it is time to restore the status quo. See also PR<br>
12026.<br></blockquote><div><br></div><div>This is a hack though, and it's important to acknowledge that. We're now handling -target in two separate places. It would be better to sink this down to be grouped with the 'getToolChain()' call that is relevant, and add comments at the very least.</div>
<div><br></div><div>Also, to introduce a hack like this, we definitely need good test cases so that we understand what is and isn't supposed to work. Relying on the happenstance of DefaultTargetTriple is what got us into the mess in the first place.</div>
</div>