How about now? I’d like to clean up that FIXME and Xcode 4.6 has been out for a 
long time….

> On Nov 11, 2013, at 1:07 PM, Nico Weber <[email protected]> wrote:
> 
> On Mon, Nov 11, 2013 at 12:17 PM, Bob Wilson <[email protected] 
> <mailto:[email protected]>> wrote:
> Nico,
> 
> Have you upgraded your bots to Xcode 4.6 or Xcode 5.0 yet?
> 
> No, not yet. But with the talk of LLVM moving to C++11 (which I suppose 
> implies using libc++ on OS X), chances are it'll happen relatively soon.
>  
> 
> On Mar 12, 2013, at 12:54 PM, Bob Wilson <[email protected] 
> <mailto:[email protected]>> wrote:
> 
>> No, sorry, I just figured that you would have updated by now.  Would you 
>> like me to put it back again?
>> 
>> On Mar 12, 2013, at 12:49 PM, Nico Weber <[email protected] 
>> <mailto:[email protected]>> wrote:
>> 
>>> Is there urgency to do this? Xcode 4.6 has been out for 2 months, and 
>>> upgrading bots takes a while.
>>> 
>>> 
>>> On Tue, Mar 12, 2013 at 12:39 PM, Bob Wilson <[email protected] 
>>> <mailto:[email protected]>> wrote:
>>> Author: bwilson
>>> Date: Tue Mar 12 14:39:19 2013
>>> New Revision: 176887
>>> 
>>> URL: http://llvm.org/viewvc/llvm-project?rev=176887&view=rev 
>>> <http://llvm.org/viewvc/llvm-project?rev=176887&view=rev>
>>> Log:
>>> Revert r166370 and r166540 now that Xcode 4.6 has been available for a 
>>> while.
>>> 
>>> Those changes were added as a temporary workaround for Xcode 4.5 passing the
>>> -Wno-arc-abi option.  Xcode 4.6 does not pass that option so this should no
>>> longer be necessary.
>>> 
>>> Modified:
>>>     cfe/trunk/include/clang/Basic/DiagnosticGroups.td
>>>     cfe/trunk/test/Driver/warning-options.cpp
>>> 
>>> Modified: cfe/trunk/include/clang/Basic/DiagnosticGroups.td
>>> URL: 
>>> http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Basic/DiagnosticGroups.td?rev=176887&r1=176886&r2=176887&view=diff
>>>  
>>> <http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Basic/DiagnosticGroups.td?rev=176887&r1=176886&r2=176887&view=diff>
>>> ==============================================================================
>>> --- cfe/trunk/include/clang/Basic/DiagnosticGroups.td (original)
>>> +++ cfe/trunk/include/clang/Basic/DiagnosticGroups.td Tue Mar 12 14:39:19 
>>> 2013
>>> @@ -303,8 +303,6 @@ def ImplicitAtomic : DiagGroup<"implicit
>>>  def CustomAtomic : DiagGroup<"custom-atomic-properties">;
>>>  def AtomicProperties : DiagGroup<"atomic-properties",
>>>                                   [ImplicitAtomic, CustomAtomic]>;
>>> -// FIXME: Remove arc-abi once an Xcode is released that doesn't pass this 
>>> flag.
>>> -def : DiagGroup<"arc-abi">;
>>>  def ARCUnsafeRetainedAssign : DiagGroup<"arc-unsafe-retained-assign">;
>>>  def ARCRetainCycles : DiagGroup<"arc-retain-cycles">;
>>>  def ARCNonPodMemAccess : DiagGroup<"arc-non-pod-memaccess">;
>>> 
>>> Modified: cfe/trunk/test/Driver/warning-options.cpp
>>> URL: 
>>> http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/warning-options.cpp?rev=176887&r1=176886&r2=176887&view=diff
>>>  
>>> <http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/warning-options.cpp?rev=176887&r1=176886&r2=176887&view=diff>
>>> ==============================================================================
>>> --- cfe/trunk/test/Driver/warning-options.cpp (original)
>>> +++ cfe/trunk/test/Driver/warning-options.cpp Tue Mar 12 14:39:19 2013
>>> @@ -3,11 +3,6 @@
>>>  // RUN: %clang -### -Wlarge-by-value-copy=128 %s 2>&1 | FileCheck 
>>> -check-prefix=LARGE_VALUE_COPY_JOINED %s
>>>  // LARGE_VALUE_COPY_JOINED: -Wlarge-by-value-copy=128
>>> 
>>> -// FIXME: Remove this together with -Warc-abi once an Xcode is released 
>>> that doesn't pass this flag.
>>> -// RUN: %clang -### -Warc-abi -Wno-arc-abi %s 2>&1 | FileCheck 
>>> -check-prefix=ARCABI %s
>>> -// ARCABI-NOT: unknown warning option '-Warc-abi'
>>> -// ARCABI-NOT: unknown warning option '-Wno-arc-abi'
>>> -
>>>  // Check that -isysroot warns on nonexistent paths.
>>>  // RUN: %clang -### -c -target i386-apple-darwin10 -isysroot /FOO %s 2>&1 
>>> | FileCheck --check-prefix=CHECK-ISYSROOT %s
>>>  // CHECK-ISYSROOT: warning: no such sysroot directory: 
>>> '{{([A-Za-z]:.*)?}}/FOO'
>>> 
>>> 
>>> _______________________________________________
>>> cfe-commits mailing list
>>> [email protected] <mailto:[email protected]>
>>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits 
>>> <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