On Jul 19, 2011, at 12:47 PM, Chad Rosier wrote:

> On Jul 19, 2011, at 12:42 PM, Eric Christopher wrote:
> 
>> 
>> On Jul 19, 2011, at 12:36 PM, Chad Rosier wrote:
>> 
>>> +  } else if (PlatformName == "win32") {
>>> +    // Due to option -ccc-host-triple arch-pc-win32-macho.
>>> +    // Don't emit __ENVIRONMENT_XX_OS_VERSION_MIN_REQUIRED__ as we're 
>>> generating
>>> +    // code for Win32 ABI.
>>>   } else {
>> 
>> An empty else if conditional is a little icky. Is it possible to rework the 
>> surrounding code to change the conditions here so we can avoid that?
> 
> Anything is possible. :)  I'll see what I can do.

Refactored in clang revision 135510.

>  Chad
> 
>> -eric
> 
> _______________________________________________
> cfe-commits mailing list
> [email protected]
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to