<p dir="ltr">Now is fine. Thanks for waiting :-)</p>
<div class="gmail_quote">On Dec 1, 2014 1:43 PM, "Bob Wilson" <<a href="mailto:bob.wilson@apple.com">bob.wilson@apple.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">How about now? I’d like to clean up that FIXME and Xcode 4.6 has been out for a long time….<div><br><div><blockquote type="cite"><div>On Nov 11, 2013, at 1:07 PM, Nico Weber <<a href="mailto:thakis@chromium.org" target="_blank">thakis@chromium.org</a>> wrote:</div><br><div><div dir="ltr">On Mon, Nov 11, 2013 at 12:17 PM, Bob Wilson <span dir="ltr"><<a href="mailto:bob.wilson@apple.com" target="_blank">bob.wilson@apple.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">Nico,<div><br></div><div>Have you upgraded your bots to Xcode 4.6 or Xcode 5.0 yet?</div>
</div></blockquote><div><br></div><div>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.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div style="word-wrap:break-word"><div><div><div><br><div><div>On Mar 12, 2013, at 12:54 PM, Bob Wilson <<a href="mailto:bob.wilson@apple.com" target="_blank">bob.wilson@apple.com</a>> wrote:</div><br><blockquote type="cite">
<div style="word-wrap:break-word">No, sorry, I just figured that you would have updated by now.  Would you like me to put it back again?<div><br><div><div>On Mar 12, 2013, at 12:49 PM, Nico Weber <<a href="mailto:thakis@chromium.org" target="_blank">thakis@chromium.org</a>> wrote:</div>
<br><blockquote type="cite"><div dir="ltr">Is there urgency to do this? Xcode 4.6 has been out for 2 months, and upgrading bots takes a while.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Mar 12, 2013 at 12:39 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">Author: bwilson<br>
Date: Tue Mar 12 14:39:19 2013<br>
New Revision: 176887<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=176887&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=176887&view=rev</a><br>
Log:<br>
Revert r166370 and r166540 now that Xcode 4.6 has been available for a while.<br>
<br>
Those changes were added as a temporary workaround for Xcode 4.5 passing the<br>
-Wno-arc-abi option.  Xcode 4.6 does not pass that option so this should no<br>
longer be necessary.<br>
<br>
Modified:<br>
    cfe/trunk/include/clang/Basic/DiagnosticGroups.td<br>
    cfe/trunk/test/Driver/warning-options.cpp<br>
<br>
Modified: cfe/trunk/include/clang/Basic/DiagnosticGroups.td<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Basic/DiagnosticGroups.td?rev=176887&r1=176886&r2=176887&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Basic/DiagnosticGroups.td?rev=176887&r1=176886&r2=176887&view=diff</a><br>


==============================================================================<br>
--- cfe/trunk/include/clang/Basic/DiagnosticGroups.td (original)<br>
+++ cfe/trunk/include/clang/Basic/DiagnosticGroups.td Tue Mar 12 14:39:19 2013<br>
@@ -303,8 +303,6 @@ def ImplicitAtomic : DiagGroup<"implicit<br>
 def CustomAtomic : DiagGroup<"custom-atomic-properties">;<br>
 def AtomicProperties : DiagGroup<"atomic-properties",<br>
                                  [ImplicitAtomic, CustomAtomic]>;<br>
-// FIXME: Remove arc-abi once an Xcode is released that doesn't pass this flag.<br>
-def : DiagGroup<"arc-abi">;<br>
 def ARCUnsafeRetainedAssign : DiagGroup<"arc-unsafe-retained-assign">;<br>
 def ARCRetainCycles : DiagGroup<"arc-retain-cycles">;<br>
 def ARCNonPodMemAccess : DiagGroup<"arc-non-pod-memaccess">;<br>
<br>
Modified: cfe/trunk/test/Driver/warning-options.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/warning-options.cpp?rev=176887&r1=176886&r2=176887&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/warning-options.cpp?rev=176887&r1=176886&r2=176887&view=diff</a><br>


==============================================================================<br>
--- cfe/trunk/test/Driver/warning-options.cpp (original)<br>
+++ cfe/trunk/test/Driver/warning-options.cpp Tue Mar 12 14:39:19 2013<br>
@@ -3,11 +3,6 @@<br>
 // RUN: %clang -### -Wlarge-by-value-copy=128 %s 2>&1 | FileCheck -check-prefix=LARGE_VALUE_COPY_JOINED %s<br>
 // LARGE_VALUE_COPY_JOINED: -Wlarge-by-value-copy=128<br>
<br>
-// FIXME: Remove this together with -Warc-abi once an Xcode is released that doesn't pass this flag.<br>
-// RUN: %clang -### -Warc-abi -Wno-arc-abi %s 2>&1 | FileCheck -check-prefix=ARCABI %s<br>
-// ARCABI-NOT: unknown warning option '-Warc-abi'<br>
-// ARCABI-NOT: unknown warning option '-Wno-arc-abi'<br>
-<br>
 // Check that -isysroot warns on nonexistent paths.<br>
 // RUN: %clang -### -c -target i386-apple-darwin10 -isysroot /FOO %s 2>&1 | FileCheck --check-prefix=CHECK-ISYSROOT %s<br>
 // CHECK-ISYSROOT: warning: no such sysroot directory: '{{([A-Za-z]:.*)?}}/FOO'<br>
<br>
<br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu" target="_blank">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>
</blockquote></div><br></div>
</blockquote></div><br></div></div></blockquote></div><br></div></div></div></div></blockquote></div><br></div></div>
</div></blockquote></div><br></div></div></blockquote></div>