<div dir="ltr">This runline is sufficient, then:<div>// RUN: %clang_cc1 %s -triple x86_64-pc-win32-macho -emit-llvm-only</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Jun 10, 2014 at 2:46 PM, Bob Wilson <span dir="ltr"><<a href="mailto:bob.wilson@apple.com" target="_blank">bob.wilson@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">The most important check is for header search, but I was hoping it might serve as an overall sanity check for other issues. I already found one issue with i386 that is not exposed with -fsyntax-only. (I’m looking at that now and will fix it if there’s an easy solution.) I don’t feel strongly about this test, so in the interest of reducing testing time, I’ll change it to use -fsyntax-only.<br>

<div class="HOEnZb"><div class="h5"><br>
> On Jun 10, 2014, at 2:29 PM, Eric Christopher <<a href="mailto:echristo@gmail.com">echristo@gmail.com</a>> wrote:<br>
><br>
> What are you trying to test with this test? It's currently invoking<br>
> the backend and I'm not sure I see a reason for it given the original<br>
> change is only to header search?<br>
><br>
> -eric<br>
><br>
> On Tue, Jun 10, 2014 at 2:07 PM, Bob Wilson <<a href="mailto:bob.wilson@apple.com">bob.wilson@apple.com</a>> wrote:<br>
>> Author: bwilson<br>
>> Date: Tue Jun 10 16:07:12 2014<br>
>> New Revision: 210584<br>
>><br>
>> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=210584&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=210584&view=rev</a><br>
>> Log:<br>
>> Fix crash with x86_64-pc-win32-macho target. <rdar://problem/17235840><br>
>><br>
>> The changes in r204978 broke win32-macho targets. There were checks added for<br>
>> MSVC and Itanium environments as special cases, and win32-macho needs to be<br>
>> treated the same way.<br>
>><br>
>> Added:<br>
>>    cfe/trunk/test/Misc/win32-macho.c<br>
>> Modified:<br>
>>    cfe/trunk/lib/Frontend/InitHeaderSearch.cpp<br>
>><br>
>> Modified: cfe/trunk/lib/Frontend/InitHeaderSearch.cpp<br>
>> URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Frontend/InitHeaderSearch.cpp?rev=210584&r1=210583&r2=210584&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Frontend/InitHeaderSearch.cpp?rev=210584&r1=210583&r2=210584&view=diff</a><br>

>> ==============================================================================<br>
>> --- cfe/trunk/lib/Frontend/InitHeaderSearch.cpp (original)<br>
>> +++ cfe/trunk/lib/Frontend/InitHeaderSearch.cpp Tue Jun 10 16:07:12 2014<br>
>> @@ -472,7 +472,8 @@ void InitHeaderSearch::AddDefaultInclude<br>
>><br>
>>   case llvm::Triple::Win32:<br>
>>     if (triple.getEnvironment() == llvm::Triple::MSVC ||<br>
>> -        triple.getEnvironment() == llvm::Triple::Itanium)<br>
>> +        triple.getEnvironment() == llvm::Triple::Itanium ||<br>
>> +        triple.getObjectFormat() == llvm::Triple::MachO)<br>
>>       return;<br>
>>     break;<br>
>>   }<br>
>><br>
>> Added: cfe/trunk/test/Misc/win32-macho.c<br>
>> URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Misc/win32-macho.c?rev=210584&view=auto" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Misc/win32-macho.c?rev=210584&view=auto</a><br>

>> ==============================================================================<br>
>> --- cfe/trunk/test/Misc/win32-macho.c (added)<br>
>> +++ cfe/trunk/test/Misc/win32-macho.c Tue Jun 10 16:07:12 2014<br>
>> @@ -0,0 +1,2 @@<br>
>> +// Check that basic use of win32-macho targets works.<br>
>> +// RUN: %clang -c -target x86_64-pc-win32-macho %s<br>
>><br>
>><br>
>> _______________________________________________<br>
>> cfe-commits mailing list<br>
>> <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
>> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br>
<br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</div></div></blockquote></div><br></div>