[cfe-commits] r162252 - in /cfe/trunk: include/clang/Basic/ include/clang/Driver/ lib/ARCMigrate/ lib/CodeGen/ lib/Driver/ lib/Frontend/ lib/Lex/ lib/Sema/ test/Driver/

[John McCall]

On Aug 21, 2012, at 11:36 AM, Ted Kremenek wrote:
> It should just be "OS X".
> 
> On Aug 21, 2012, at 9:34 AM, Jordan Rose <jordan_rose at apple.com> wrote:
> 
>> 
>> On Aug 20, 2012, at 19:47 , John McCall <rjmccall at apple.com> wrote:
>> 
>>> +def err_arc_unsupported_on_toolchain : Error<
>>> +  "-fobjc-arc is not supported on %select{versions of Mac OS prior to 10.6}0">;
>> 
>> I guess the idea is that other toolchains could change this, but it looks funny right now. Also, marketing quibble: "Mac OS X" or "OS X" rather than "Mac OS"? (At the time of 10.6, the official name was "Mac OS X".)

It was generalized because I thought I'd need more than one case right away,
but you're right that as it stands it's a bit silly.  Fixed in r162651 (as "OS X").

John.