<html><head><meta http-equiv="Content-Type" content="text/html charset=iso-8859-1"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">Nico,<div><br></div><div>Have you upgraded your bots to Xcode 4.6 or Xcode 5.0 yet?</div><div><br><div><div>On Mar 12, 2013, at 12:54 PM, Bob Wilson <<a href="mailto:bob.wilson@apple.com">bob.wilson@apple.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><meta http-equiv="Content-Type" content="text/html charset=iso-8859-1"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">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">thakis@chromium.org</a>> wrote:</div><br class="Apple-interchange-newline"><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">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></body></html>