Ron, please post this patch back on the review thread where you got feedback from Duncan and lets finish discussing it there.<div><br></div><div>I'll try to actually write my thoughts there, but I still don't see the need for or benefit of this patch.... But forking the discussion into a new thread makes it really hard to have a useful discussion and keep everyone on the same page.</div>
<div class="gmail_extra"><br><br><div class="gmail_quote">On Sun, Sep 9, 2012 at 3:16 PM, Ron Lieberman <span dir="ltr"><<a href="mailto:ronl@codeaurora.org" target="_blank">ronl@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 lang="EN-US" link="blue" vlink="purple"><div><p class="MsoNormal">This is a revised patch incorporating the new spelling<u></u><u></u></p>
<p class="MsoNormal">   llvm-vendor<u></u><u></u></p><p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal">[PATCH] Provide the capability of defining CLANG_VENDOR via<u></u><u></u></p><p class="MsoNormal"> configure or cmake.<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal">Default for the new configure option:<u></u><u></u></p><p class="MsoNormal">  --with-llvm-vendor defaults to null.<u></u><u></u></p><p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">The vendor name is configured using the following option<u></u><u></u></p><p class="MsoNormal">  --with-llvm-vendor="INTLWICKET " \<u></u><u></u></p><p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal">
Examples:<u></u><u></u></p><p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal">$ clang -v<u></u><u></u></p><p class="MsoNormal">INTLWICKET clang version 3.2  (based on LLVM 3.2svn)<u></u><u></u></p><p class="MsoNormal">
Target: x86_64-unknown-linux-gnu<u></u><u></u></p><p class="MsoNormal">Thread model: posix<u></u><u></u></p><p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal">The __VERSION__ macro becomes:<u></u><u></u></p><p class="MsoNormal">
 define __VERSION__ "4.2.1 Compatible INTLWICKET Clang 3.2 "<u></u><u></u></p><p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal">Testing:<u></u><u></u></p><p class="MsoNormal">cmake<u></u><u></u></p><p class="MsoNormal">
cmake -DLLVM_VENDOR=<u></u><u></u></p><p class="MsoNormal">cmake -DLLVM_VENDOR="INTLWICKET "<u></u><u></u></p><p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal">configure<u></u><u></u></p><p class="MsoNormal">
configure --with-llvm-vendor=<u></u><u></u></p><p class="MsoNormal">configure --with-llvm-vendor="INTLWICKET "<u></u><u></u></p><p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal">---<u></u><u></u></p><p class="MsoNormal">
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation<u></u><u></u></p><p class="MsoNormal"><u></u> <u></u></p></div></div><br>_______________________________________________<br>

llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br></div>