<div dir="ltr">On Wed, Jul 3, 2013 at 8:36 AM, Bill Schmidt <span dir="ltr"><<a href="mailto:wschmidt@linux.vnet.ibm.com" target="_blank">wschmidt@linux.vnet.ibm.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: wschmidt<br>
Date: Wed Jul  3 10:36:02 2013<br>
New Revision: 185544<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=185544&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=185544&view=rev</a><br>
Log:<br>
Fix PR16454: Don't #include altivec.h when preprocessing assembly.<br>
<br>
When the -maltivec flag is present, altivec.h is auto-included for the<br>
compilation.  This is not appropriate when the job action is to<br>
preprocess a file containing assembly code.  So don't do that.<br>
<br>
I was unable to convert the test in the bug report into a regression<br>
test.  The original symptom was exposed with:<br>
<br>
  % touch x.S<br>
  % ./bin/clang -target powerpc64-unknown-linux-gnu -maltivec -S -o - x.S<br>
<br>
I tried this test (and numerous variants) on a PPC64 system:<br>
<br>
----------------------------------------------------------------------------<br>
// RUN: touch %t<br>
// RUN: %clang -maltivec -S %t -o - | FileCheck %s<br>
<br>
// Verify that assembling an empty file does not auto-include altivec.h.<br>
<br>
// CHECK-NOT: static vector<br>
----------------------------------------------------------------------------<br>
<br>
However, this test passes for some reason even on a clang built<br>
without the fix.  I'd be happy to add a test case but at this point<br>
I'm not able to figure one out, and I don't want to hold up the patch<br>
unnecessarily.  Please let me know if you have ideas.<br><br></blockquote><div> </div><div>Umm, why are you committing a patch for an issue you can't reproduce?</div><div><br></div><div>-Eli </div></div><br></div></div>