Looks good, please commit.<div><br></div><div>(modulo the style nit i mentioned in IRC)</div><div><br></div><div>Thanks for driving down to the right fix. =D<br><br><div class="gmail_quote">On Thu, Mar 1, 2012 at 2:36 PM, Michael Spencer <span dir="ltr"><<a href="mailto:bigcheesegs@gmail.com">bigcheesegs@gmail.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="HOEnZb"><div class="h5">On Thu, Mar 1, 2012 at 2:16 PM, Aaron Ballman <<a href="mailto:aaron@aaronballman.com">aaron@aaronballman.com</a>> wrote:<br>

> On Thu, Mar 1, 2012 at 4:09 PM, Michael Spencer <<a href="mailto:bigcheesegs@gmail.com">bigcheesegs@gmail.com</a>> wrote:<br>
>> On Thu, Mar 1, 2012 at 1:18 PM, Michael Spencer <<a href="mailto:bigcheesegs@gmail.com">bigcheesegs@gmail.com</a>> wrote:<br>
>>> On Thu, Mar 1, 2012 at 12:02 PM, Benjamin Kramer<br>
>>> <<a href="mailto:benny.kra@googlemail.com">benny.kra@googlemail.com</a>> wrote:<br>
>>>><br>
>>>> On 01.03.2012, at 20:54, Michael Spencer wrote:<br>
>>>><br>
>>>>> diff --git a/cmake/modules/HandleLLVMOptions.cmake b/cmake/modules/HandleLLVMOptions.cmake<br>
>>>>> index 0a5c02f..96f091b 100644<br>
>>>>> --- a/cmake/modules/HandleLLVMOptions.cmake<br>
>>>>> +++ b/cmake/modules/HandleLLVMOptions.cmake<br>
>>>>> @@ -136,6 +136,10 @@ endif()<br>
>>>>>  if( MSVC )<br>
>>>>>    include(ChooseMSVCCRT)<br>
>>>>><br>
>>>>> +  if (MSVC11)<br>
>>>>> +    add_llvm_definitions(-D_VARIADIC_MAX=10)<br>
>>>>> +  endif()<br>
>>>><br>
>>>> ok<br>
>>>><br>
>>>>> +<br>
>>>>>    # Add definitions that make MSVC much less annoying.<br>
>>>>>    add_llvm_definitions(<br>
>>>>>      # For some reason MS wants to deprecate a bunch of standard functions...<br>
>>>>> diff --git a/lib/CodeGen/AsmPrinter/DwarfDebug.cpp b/lib/CodeGen/AsmPrinter/DwarfDebug.cpp<br>
>>>>> index 920f82d..006bc28 100644<br>
>>>>> --- a/lib/CodeGen/AsmPrinter/DwarfDebug.cpp<br>
>>>>> +++ b/lib/CodeGen/AsmPrinter/DwarfDebug.cpp<br>
>>>>> @@ -530,7 +530,9 @@ unsigned DwarfDebug::GetOrCreateSourceID(StringRef FileName,<br>
>>>>><br>
>>>>>    std::map<std::pair<std::string, std::string>, unsigned>::iterator I;<br>
>>>>>    bool NewlyInserted;<br>
>>>>> -  tie(I, NewlyInserted) = SourceIdMap.insert(Entry);<br>
>>>>> +  std::pair<std::map<std::pair<std::string, std::string>, unsigned>::iterator, bool> blah = SourceIdMap.insert(Entry);<br>
>>>>> +  I = blah.first;<br>
>>>>> +  NewlyInserted = blah.second;<br>
>>>><br>
>>>> ugly, but I don't see what's wrong here either. Can you write the declaration & initialization in a single line and make sure you don't violate 80 cols?<br>
>>>><br>
>>>>>    if (!NewlyInserted)<br>
>>>>>      return I->second;<br>
>>>>><br>
>>>>> diff --git a/lib/Target/X86/MCTargetDesc/X86MCTargetDesc.cpp b/lib/Target/X86/MCTargetDesc/X86MCTargetDesc.cpp<br>
>>>>> index 5ca592d..88c5635 100644<br>
>>>>> --- a/lib/Target/X86/MCTargetDesc/X86MCTargetDesc.cpp<br>
>>>>> +++ b/lib/Target/X86/MCTargetDesc/X86MCTargetDesc.cpp<br>
>>>>> @@ -36,6 +36,8 @@<br>
>>>>>  #define GET_SUBTARGETINFO_MC_DESC<br>
>>>>>  #include "X86GenSubtargetInfo.inc"<br>
>>>>><br>
>>>>> +#include <intrin.h><br>
>>>>> +<br>
>>>><br>
>>>> <intrin.h> doesn't exist on non-windows platforms. Why is it needed here?<br>
>>>><br>
>>>>>  using namespace llvm;<br>
>>>>><br>
>>>>><br>
>>><br>
>>> Ugg, I sent the wrong version of the patch. I already fixed both of<br>
>>> those issues. intrin.h is needed for __cpuid.<br>
>>><br>
>>> Attached is the correct version.<br>
>>><br>
>>> - Michael Spencer<br>
>><br>
>> And here it is with a typedef.<br>
><br>
> So much nicer-looking now!  :-)  Patch LGTM.<br>
><br>
> ~Aaron<br>
<br>
</div></div>And this one should look even better :P.<br>
<span class="HOEnZb"><font color="#888888"><br>
- Michael Spencer<br>
</font></span><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>