<div class="gmail_quote">On Thu, Jan 26, 2012 at 8:22 PM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@google.com">chandlerc@google.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="gmail_quote">On Thu, Jan 26, 2012 at 1:56 PM, Joerg Sonnenberger <span dir="ltr"><<a href="mailto:joerg@bec.de" target="_blank">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>
</blockquote></div><br><div>Having now read the patch, I definitely want this to be reverted. It is similar to the design I had planned for this feature, but it introduces a lot of boiler plate that is not needed, and misses some key elements. We need to discuss the design and implementation of this before you commit, and that should have happened with a patch, not a commit.</div>