<div class="gmail_quote">On Thu, Jan 26, 2012 at 1:56 PM, Joerg Sonnenberger <span dir="ltr"><<a href="mailto:joerg@bec.de">joerg@bec.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Author: joerg<br>
Date: Thu Jan 26 15:56:28 2012<br>
New Revision: 149083<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=149083&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=149083&view=rev</a><br>
Log:<br>
Keep track of the original target the user specified before<br>
normalization. This used to be captured in DefaultTargetTriple and is<br>
used for the (optional) $triple-$tool lookup for cross-compilation.<br>
Do this properly by making it an attribute of the toolchain and use it<br>
in combination with the computed triple as index for the toolchain<br>
lookup.<br></blockquote><div><br></div><div>Why wasn't this (non-trivial or obvious cleanup) patch mailed for code review? Or generally discussed? Please don't commit w/o review.</div><div><br></div><div>This is I suspect similar to the approach I had in mind, but I haven't had time to review it or make sure it is actually sane. Personally, I would like you to revert and follow the standard developer policy by mailing a patch for review. I suspect that functionality like this is exactly the right way to go, but it needs to be discussed first.</div>
</div>