r191458 - Fix up fallout from r187156.

2013-09-26 Thread Bob Wilson
Author: bwilson Date: Thu Sep 26 16:00:51 2013 New Revision: 191458 URL: http://llvm.org/viewvc/llvm-project?rev=191458view=rev Log: Fix up fallout from r187156. The previous change caused the driver to translate -Wa,-L to the -msave-temp-labels option for cc1as, but cc1as did not accept that

Re: buildbot failure in LLVM on clang-x86_64-linux-selfhost-rel

2013-09-19 Thread Bob Wilson
That buildbot succeeded on the next run, so unless someone just reverted your patch, the issue is already resolved. On Sep 19, 2013, at 11:11 AM, John Thompson john.thompson.jtsoftw...@gmail.com wrote: Hi, I'm not sure what I need to do about this, or if I need to do something. It

Re: [PATCH] [ARM] Remove dead code for target architectures with a '-' in the name

2013-09-17 Thread Bob Wilson
. Check with Jim Grosbach or Bob Wilson before committing. http://llvm-reviews.chandlerc.com/D1695 ___ cfe-commits mailing list cfe-commits@cs.uiuc.edu http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Re: [PATCH] ARMv7A Cleanup

2013-09-10 Thread Bob Wilson
[ adding Jim Grosbach, who may have input on the architectures used in triples ] On Sep 3, 2013, at 4:56 AM, Renato Golin renato.go...@linaro.org wrote: Hi doug.gregor, This is a cleanup of the ARMv7A CPUs and their options and features. - Adding Cortex-A12 and A5 and making sure all

Re: [clang-tools-extra] r189904 - Suppress test/modularize/NoProblemsDependencies.modularize on msvc, for now. Investigating.

2013-09-04 Thread Bob Wilson
It's not just msvc that is failing. It's also breaking on Darwin and Debian, e.g.: http://lab.llvm.org:8011/builders/clang-x86_64-debian/builds/7311 On Sep 3, 2013, at 7:03 PM, NAKAMURA Takumi geek4ci...@gmail.com wrote: Author: chapuni Date: Tue Sep 3 21:03:03 2013 New Revision: 189904

[clang-tools-extra] r189957 - Revert svn 189837 Added header dependencies support.

2013-09-04 Thread Bob Wilson
Author: bwilson Date: Wed Sep 4 11:48:28 2013 New Revision: 189957 URL: http://llvm.org/viewvc/llvm-project?rev=189957view=rev Log: Revert svn 189837 Added header dependencies support. The NoProblemsDependencies.modularize test is failing on many buildbots. I have also reverted the change in

Re: [clang-tools-extra] r189837 - Added header dependencies support.

2013-09-04 Thread Bob Wilson
I have reverted this change in svn 189957. The NoProblemsDependencies.modularize test is failing on many buildbots, e.g., http://lab.llvm.org:8011/builders/clang-x86_64-debian/builds/7311. On Sep 3, 2013, at 11:48 AM, John Thompson john.thompson.jtsoftw...@gmail.com wrote: Author: jtsoftware

r188700 - Bump the value of the __APPLE_CC__ predefined macro up to 6000.

2013-08-19 Thread Bob Wilson
Author: bwilson Date: Mon Aug 19 15:23:37 2013 New Revision: 188700 URL: http://llvm.org/viewvc/llvm-project?rev=188700view=rev Log: Bump the value of the __APPLE_CC__ predefined macro up to 6000. The previous value was set to match some ancient version of Apple's GCC. The value should be higher

r187680 - Only use the Darwin linker's -export_dynamic option with supported versions.

2013-08-02 Thread Bob Wilson
Author: bwilson Date: Fri Aug 2 17:25:34 2013 New Revision: 187680 URL: http://llvm.org/viewvc/llvm-project?rev=187680view=rev Log: Only use the Darwin linker's -export_dynamic option with supported versions. Related to rdar://problem/14578094. Modified: cfe/trunk/lib/Driver/Tools.cpp

r187315 - Reduce stack frame size by avoiding a large token vector on an error path.

2013-07-27 Thread Bob Wilson
Author: bwilson Date: Sat Jul 27 16:59:57 2013 New Revision: 187315 URL: http://llvm.org/viewvc/llvm-project?rev=187315view=rev Log: Reduce stack frame size by avoiding a large token vector on an error path. Beginning with svn r186971, we noticed an internal test started to fail when using clang

r187246 - Fix up the BUILD_CLANG_ONLY code to work properly after r184794.

2013-07-26 Thread Bob Wilson
Author: bwilson Date: Fri Jul 26 16:08:22 2013 New Revision: 187246 URL: http://llvm.org/viewvc/llvm-project?rev=187246view=rev Log: Fix up the BUILD_CLANG_ONLY code to work properly after r184794. When BUILD_CLANG_ONLY is set to YES, it is supposed to simply limit the tools that get built. The

r184171 - size_t on Darwin AAPCS targets is unsigned long. rdar://problem/14136459

2013-06-17 Thread Bob Wilson
Author: bwilson Date: Tue Jun 18 00:36:04 2013 New Revision: 184171 URL: http://llvm.org/viewvc/llvm-project?rev=184171view=rev Log: size_t on Darwin AAPCS targets is unsigned long. rdar://problem/14136459 Some embedded targets use ARM's AAPCS with iOS header files that define size_t as unsigned

r183209 - Rephrase asm_mismatched_size_modifier diagnostic. rdar://problem/14050339

2013-06-04 Thread Bob Wilson
Author: bwilson Date: Tue Jun 4 01:10:09 2013 New Revision: 183209 URL: http://llvm.org/viewvc/llvm-project?rev=183209view=rev Log: Rephrase asm_mismatched_size_modifier diagnostic. rdar://problem/14050339 The text of this diagnostic was unnecessarily specific to the current ARM implementation

r183172 - Do not report -Wasm-operand-widths for ARM output operands. rdar://14050337

2013-06-03 Thread Bob Wilson
Author: bwilson Date: Mon Jun 3 18:57:13 2013 New Revision: 183172 URL: http://llvm.org/viewvc/llvm-project?rev=183172view=rev Log: Do not report -Wasm-operand-widths for ARM output operands. rdar://14050337 We're getting reports of this warning getting triggered in cases where it is not adding

Re: [PATCH] Attach objc-arc function attribute to function definitions compiled in a module with ARC enabled.

2013-05-15 Thread Bob Wilson
On May 15, 2013, at 3:39 PM, jahanian fjahan...@apple.com wrote: On May 15, 2013, at 2:27 PM, Michael Gottesman mgottes...@apple.com wrote: Hey Fariborz! I think that predicating this on LTO may be the wrong thing to do [but please correct me if I am wrong as always = )].

[libcxx] r180122 - PR12597: Remove chown -R root:wheel from the makefile.

2013-04-23 Thread Bob Wilson
Author: bwilson Date: Tue Apr 23 12:30:35 2013 New Revision: 180122 URL: http://llvm.org/viewvc/llvm-project?rev=180122view=rev Log: PR12597: Remove chown -R root:wheel from the makefile. Modified: libcxx/trunk/Makefile Modified: libcxx/trunk/Makefile URL:

[libcxx] r180132 - PR15820: Use tar instead of rsync to install the headers.

2013-04-23 Thread Bob Wilson
Author: bwilson Date: Tue Apr 23 13:51:51 2013 New Revision: 180132 URL: http://llvm.org/viewvc/llvm-project?rev=180132view=rev Log: PR15820: Use tar instead of rsync to install the headers. Modified: libcxx/trunk/Makefile Modified: libcxx/trunk/Makefile URL:

[libcxx] r180135 - Change makefile comment to refer to libc++ instead of libcpp.

2013-04-23 Thread Bob Wilson
Author: bwilson Date: Tue Apr 23 14:26:55 2013 New Revision: 180135 URL: http://llvm.org/viewvc/llvm-project?rev=180135view=rev Log: Change makefile comment to refer to libc++ instead of libcpp. Modified: libcxx/trunk/Makefile Modified: libcxx/trunk/Makefile URL:

r179733 - Add description of -Ofast optimization option to the man page. rdar://13660458

2013-04-17 Thread Bob Wilson
Author: bwilson Date: Wed Apr 17 17:32:43 2013 New Revision: 179733 URL: http://llvm.org/viewvc/llvm-project?rev=179733view=rev Log: Add description of -Ofast optimization option to the man page. rdar://13660458 Modified: cfe/trunk/docs/tools/clang.pod Modified:

Re: r179406 - Define Neon intrinsics as static inline to avoid warning. rdar://13108414

2013-04-16 Thread Bob Wilson
like a cleaner solution. On Apr 13, 2013 6:20 AM, Bob Wilson bob.wil...@apple.com wrote: Author: bwilson Date: Fri Apr 12 15:17:20 2013 New Revision: 179406 URL: http://llvm.org/viewvc/llvm-project?rev=179406view=rev Log: Define Neon intrinsics as static inline to avoid warning. rdar

r179406 - Define Neon intrinsics as static inline to avoid warning. rdar://13108414

2013-04-12 Thread Bob Wilson
Author: bwilson Date: Fri Apr 12 15:17:20 2013 New Revision: 179406 URL: http://llvm.org/viewvc/llvm-project?rev=179406view=rev Log: Define Neon intrinsics as static inline to avoid warning. rdar://13108414 We had been defining Neon intrinsics as static with always_inline attributes. If you use

r179309 - Add a new -mimplicit-float option for symmetry with -mno-implicit-float.

2013-04-11 Thread Bob Wilson
Author: bwilson Date: Thu Apr 11 13:53:25 2013 New Revision: 179309 URL: http://llvm.org/viewvc/llvm-project?rev=179309view=rev Log: Add a new -mimplicit-float option for symmetry with -mno-implicit-float. This new option is the default, but it is useful to have a flag to override

r177815 - Revert svn r176894 and r177658.

2013-03-22 Thread Bob Wilson
Author: bwilson Date: Sat Mar 23 00:17:59 2013 New Revision: 177815 URL: http://llvm.org/viewvc/llvm-project?rev=177815view=rev Log: Revert svn r176894 and r177658. Changing -ccc-install-dir to affect cc1's resource-dir setting broke our internal LNT tests. After discussing the situation with

r177626 - Fix a typo.

2013-03-21 Thread Bob Wilson
Author: bwilson Date: Thu Mar 21 01:09:09 2013 New Revision: 177626 URL: http://llvm.org/viewvc/llvm-project?rev=177626view=rev Log: Fix a typo. Modified: cfe/trunk/include/clang/Basic/DiagnosticGroups.td cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.td Modified:

r177165 - Revert Remove a pointless assertion.

2013-03-15 Thread Bob Wilson
Author: bwilson Date: Fri Mar 15 12:12:43 2013 New Revision: 177165 URL: http://llvm.org/viewvc/llvm-project?rev=177165view=rev Log: Revert Remove a pointless assertion. This reverts commit r177158. I'm blindly reverting this because it appears to be breaking numerous buildbots. I'll reapply

Re: r177158 - Remove a pointless assertion.

2013-03-15 Thread Bob Wilson
On Mar 15, 2013, at 8:02 AM, Nico Weber nicolaswe...@gmx.de wrote: Author: nico Date: Fri Mar 15 10:02:37 2013 New Revision: 177158 URL: http://llvm.org/viewvc/llvm-project?rev=177158view=rev Log: Remove a pointless assertion. FindNodeOrInsertPos() is called 10 lines earlier already,

Re: r177158 - Remove a pointless assertion.

2013-03-15 Thread Bob Wilson
On Mar 15, 2013, at 10:19 AM, Bob Wilson bob.wil...@apple.com wrote: On Mar 15, 2013, at 8:02 AM, Nico Weber nicolaswe...@gmx.de wrote: Author: nico Date: Fri Mar 15 10:02:37 2013 New Revision: 177158 URL: http://llvm.org/viewvc/llvm-project?rev=177158view=rev Log: Remove a pointless

r176887 - Revert r166370 and r166540 now that Xcode 4.6 has been available for a while.

2013-03-12 Thread Bob Wilson
Author: bwilson Date: Tue Mar 12 14:39:19 2013 New Revision: 176887 URL: http://llvm.org/viewvc/llvm-project?rev=176887view=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

Re: r176887 - Revert r166370 and r166540 now that Xcode 4.6 has been available for a while.

2013-03-12 Thread Bob Wilson
, 2013 at 12:39 PM, Bob Wilson bob.wil...@apple.com wrote: Author: bwilson Date: Tue Mar 12 14:39:19 2013 New Revision: 176887 URL: http://llvm.org/viewvc/llvm-project?rev=176887view=rev Log: Revert r166370 and r166540 now that Xcode 4.6 has been available for a while. Those changes were

r176892 - Revert Revert r166370 and r166540 now that Xcode 4.6 has been available for a while.

2013-03-12 Thread Bob Wilson
Author: bwilson Date: Tue Mar 12 15:00:34 2013 New Revision: 176892 URL: http://llvm.org/viewvc/llvm-project?rev=176892view=rev Log: Revert Revert r166370 and r166540 now that Xcode 4.6 has been available for a while. This reverts commit 176887. Nico asked for more time to move to Xcode 4.6.

r176456 - Fix confused use of llvm::StringSwitch for armv7r architecture.

2013-03-04 Thread Bob Wilson
Author: bwilson Date: Mon Mar 4 16:37:43 2013 New Revision: 176456 URL: http://llvm.org/viewvc/llvm-project?rev=176456view=rev Log: Fix confused use of llvm::StringSwitch for armv7r architecture. svn 170909 added support for cortex-r5 but in this case it was done incorrectly. The last argument

r176457 - Tidy up lists of Cortex-A series processors, adding entries for A7.

2013-03-04 Thread Bob Wilson
Author: bwilson Date: Mon Mar 4 16:37:46 2013 New Revision: 176457 URL: http://llvm.org/viewvc/llvm-project?rev=176457view=rev Log: Tidy up lists of Cortex-A series processors, adding entries for A7. Also fix a missing entry for cortex-r5 in one copy of getLLVMArchSuffixForARM. Modified:

r176458 - Add ARM v6m, v7m, and v7em architectures for Cortex-M series processors.

2013-03-04 Thread Bob Wilson
Author: bwilson Date: Mon Mar 4 16:37:49 2013 New Revision: 176458 URL: http://llvm.org/viewvc/llvm-project?rev=176458view=rev Log: Add ARM v6m, v7m, and v7em architectures for Cortex-M series processors. rdar://problem/11314476 Added: cfe/trunk/test/Driver/arm-cortex-cpus.c Modified:

r174836 - Recognize -mno-implicit-float option for x86 as well as ARM. rdar://13180731

2013-02-10 Thread Bob Wilson
Author: bwilson Date: Sun Feb 10 09:25:44 2013 New Revision: 174836 URL: http://llvm.org/viewvc/llvm-project?rev=174836view=rev Log: Recognize -mno-implicit-float option for x86 as well as ARM. rdar://13180731 For x86 targets, we've been using the -msoft-float option to control passing the

r174837 - Delete an extra blank line.

2013-02-10 Thread Bob Wilson
Author: bwilson Date: Sun Feb 10 10:01:38 2013 New Revision: 174837 URL: http://llvm.org/viewvc/llvm-project?rev=174837view=rev Log: Delete an extra blank line. Modified: cfe/trunk/lib/Driver/ArgList.cpp Modified: cfe/trunk/lib/Driver/ArgList.cpp URL:

r174838 - Use -mno-implicit-float by default for kernel/kext code. rdar://13177960

2013-02-10 Thread Bob Wilson
Author: bwilson Date: Sun Feb 10 10:01:41 2013 New Revision: 174838 URL: http://llvm.org/viewvc/llvm-project?rev=174838view=rev Log: Use -mno-implicit-float by default for kernel/kext code. rdar://13177960 Apple's kernel engineers have been expecting this behavior even though we've never

Re: [cfe-commits] [patch] Make ARM NEON polynomial types unsigned

2013-01-16 Thread Bob Wilson
On Jan 16, 2013, at 6:44 AM, Tim Northover t.p.northo...@gmail.com wrote: I'd say that if you can get GCC to make the change, then Clang should do the same. If GCC continues to use signed polynomial types, then I'm not convinced that Clang should change to unsigned types. We've decided

Re: [cfe-commits] [patch] Make ARM NEON polynomial types unsigned

2013-01-14 Thread Bob Wilson
On Jan 14, 2013, at 4:00 AM, Tim Northover tim.northo...@arm.com wrote: Hi, As suggested by John in his review of the AArch64 backend, I'm splitting this change into a separate thread. I'd like to make the ARM NEON polynomial types be based on unsigned variants (unsigned char,

Re: [cfe-commits] [patch] Make ARM NEON polynomial types unsigned

2013-01-14 Thread Bob Wilson
On Jan 14, 2013, at 10:04 AM, Tim Northover t.p.northo...@gmail.com wrote: This makes me a little nervous. What do GCC and other ARM compilers use? If all the other ARM compilers use unsigned poly types, then it is probably a good thing to bring Clang in sync with them. After a quick

Re: [cfe-commits] [patch] Make ARM NEON polynomial types unsigned

2013-01-14 Thread Bob Wilson
On Jan 14, 2013, at 11:00 AM, Tim Northover t.p.northo...@gmail.com wrote: Hi Bob, What is the motivation for switching to unsigned? I personally believe it's a more sane for a polynomial, by quite a margin. For example it means that casting a poly8_t to a poly16_t will use the natural

Re: [cfe-commits] [patch] Make ARM NEON polynomial types unsigned

2013-01-14 Thread Bob Wilson
On Jan 14, 2013, at 11:22 AM, Tim Northover t.p.northo...@gmail.com wrote: So you're more willing to change the 32-bit ABI(s) than the 64-bit ABI? That doesn't make sense to me. The way I see it no 32-bit code should be relying on this behaviour since it's not specified in any ABI

Re: [cfe-commits] [PATCH] Warn if -isysroot is given an invalid path

2012-12-14 Thread Bob Wilson
On Dec 14, 2012, at 9:34 AM, Eric Christopher echri...@gmail.com wrote: On Fri, Dec 14, 2012 at 9:29 AM, Jordan Rose jordan_r...@apple.com wrote: I can see bootstrapping scenarios such that you wouldn't want it. Possible to only put this warning into -Weverything and not into any other

Re: [cfe-commits] [PATCH] Warn if -isysroot is given an invalid path

2012-12-14 Thread Bob Wilson
On Dec 14, 2012, at 10:28 AM, Argyrios Kyrtzidis akyr...@gmail.com wrote: On Dec 14, 2012, at 9:55 AM, Bob Wilson bob.wil...@apple.com wrote: On Dec 14, 2012, at 9:34 AM, Eric Christopher echri...@gmail.com wrote: On Fri, Dec 14, 2012 at 9:29 AM, Jordan Rose jordan_r...@apple.com

[cfe-commits] r168505 - in /cfe/trunk/lib/Driver: ToolChains.cpp Tools.cpp Tools.h

2012-11-22 Thread Bob Wilson
Author: bwilson Date: Fri Nov 23 00:14:39 2012 New Revision: 168505 URL: http://llvm.org/viewvc/llvm-project?rev=168505view=rev Log: Reapply a subset of r167567 to clean up Darwin-specific code for invoking gcc. Unlike my previous attempt at this, this patch leaves intact the check for whether

[cfe-commits] r167598 - in /cfe/trunk/lib/Driver: ToolChains.cpp ToolChains.h

2012-11-08 Thread Bob Wilson
Author: bwilson Date: Thu Nov 8 17:21:22 2012 New Revision: 167598 URL: http://llvm.org/viewvc/llvm-project?rev=167598view=rev Log: Remove more code related to invoking llvm-gcc. rdar://problem/11991320 Modified: cfe/trunk/lib/Driver/ToolChains.cpp cfe/trunk/lib/Driver/ToolChains.h

[cfe-commits] r167603 - in /cfe/trunk: lib/ARCMigrate/ARCMT.cpp lib/Driver/ToolChains.cpp lib/Driver/ToolChains.h test/Driver/ios-simulator-arcruntime.c

2012-11-08 Thread Bob Wilson
Author: bwilson Date: Thu Nov 8 19:59:30 2012 New Revision: 167603 URL: http://llvm.org/viewvc/llvm-project?rev=167603view=rev Log: Remove old driver code to grab the iOS simulator version from the -D option. We can now rely on the -mios-simulator-version-min command line option and remove the

[cfe-commits] r167566 - in /cfe/trunk: include/clang/Driver/Driver.h lib/Driver/Driver.cpp lib/Driver/ToolChains.cpp lib/Frontend/CreateInvocationFromCommandLine.cpp test/Driver/apple-kext-i386.cpp

2012-11-07 Thread Bob Wilson
Author: bwilson Date: Wed Nov 7 19:03:29 2012 New Revision: 167566 URL: http://llvm.org/viewvc/llvm-project?rev=167566view=rev Log: Remove code to fall back to llvm-gcc for i386 kexts. More cleanups to follow in separate commits Removed: cfe/trunk/test/Driver/apple-kext-i386.cpp

[cfe-commits] r167567 - in /cfe/trunk: include/clang/Driver/Driver.h include/clang/Driver/Types.h lib/Driver/Driver.cpp lib/Driver/ToolChains.cpp lib/Driver/Tools.cpp lib/Driver/Tools.h lib/Driver/Typ

2012-11-07 Thread Bob Wilson
Author: bwilson Date: Wed Nov 7 19:03:34 2012 New Revision: 167567 URL: http://llvm.org/viewvc/llvm-project?rev=167567view=rev Log: Rip out a bunch of code for invoking gcc from clang. Modified: cfe/trunk/include/clang/Driver/Driver.h cfe/trunk/include/clang/Driver/Types.h

Re: [cfe-commits] r167432 - /cfe/trunk/tools/c-index-test/Makefile

2012-11-06 Thread Bob Wilson
If you're trying to get c-index-test back into Apple builds, you'll need to do a bit more than this. We now build with BUILD_CLANG_ONLY=YES, and that disables building c-index-test. See the install-clang target in the top-level llvm Makefile and also the DIRS list in clang's tools/Makefile.

Re: [cfe-commits] r167440 - in /cfe/trunk: lib/CodeGen/TargetInfo.cpp test/CodeGen/arm-arguments.c

2012-11-06 Thread Bob Wilson
On Nov 5, 2012, at 8:58 PM, Manman Ren m...@apple.com wrote: Author: mren Date: Mon Nov 5 22:58:01 2012 New Revision: 167440 URL: http://llvm.org/viewvc/llvm-project?rev=167440view=rev Log: ARM byval: when type alignment is bigger than ABI alignment, instead of disabling byval, we set

Re: [cfe-commits] r167058 - in /cfe/trunk: lib/CodeGen/TargetInfo.cpp test/CodeGen/arm-homogenous.c

2012-10-31 Thread Bob Wilson
On Oct 30, 2012, at 4:21 PM, manman ren m...@apple.com wrote: Author: mren Date: Tue Oct 30 18:21:41 2012 New Revision: 167058 URL: http://llvm.org/viewvc/llvm-project?rev=167058view=rev Log: ARM AAPCS-VFP: fix handling of homogeneous aggreate. If HA can only partially fit into VFP

Re: [cfe-commits] r164907 - in /cfe/trunk: include/clang/Basic/ObjCRuntime.h lib/CodeGen/CGObjC.cpp lib/Driver/ToolChains.cpp lib/Driver/Tools.cpp runtime/compiler-rt/Makefile test/CodeGenObjC/optimiz

2012-10-19 Thread Bob Wilson
On Oct 8, 2012, at 3:33 PM, Anton Korobeynikov an...@korobeynikov.info wrote: Bob, const Driver D = getToolChain().getDriver(); - llvm::Triple Triple = getToolChain().getTriple(); + // Get the effective triple, which takes into account the deployment target. + std::string TripleStr

Re: [cfe-commits] r165988 - in /cfe/trunk: include/clang/Basic/ObjCRuntime.h lib/CodeGen/CGObjC.cpp lib/Driver/ToolChains.cpp lib/Driver/Tools.cpp lib/Sema/SemaExpr.cpp runtime/compiler-rt/Makefile te

2012-10-17 Thread Bob Wilson
On Oct 17, 2012, at 8:55 AM, Nico Weber tha...@chromium.org wrote: On Wed, Oct 17, 2012 at 1:03 AM, Jean-Daniel Dupas devli...@shadowlab.org wrote: Le 17 oct. 2012 à 08:10, Nico Weber tha...@chromium.org a écrit : On Tue, Oct 16, 2012 at 10:33 PM, Bob Wilson bob.wil...@apple.com wrote

[cfe-commits] r166164 - /cfe/trunk/lib/CodeGen/BackendUtil.cpp

2012-10-17 Thread Bob Wilson
Author: bwilson Date: Thu Oct 18 00:23:42 2012 New Revision: 166164 URL: http://llvm.org/viewvc/llvm-project?rev=166164view=rev Log: Revert svn r165741 Add TargetTransformInfo to the clang driver. Nadav's llvm change r165665 caused problems with an LTO bootstrap of clang, so I'm reverting it for

Re: [cfe-commits] r165988 - in /cfe/trunk: include/clang/Basic/ObjCRuntime.h lib/CodeGen/CGObjC.cpp lib/Driver/ToolChains.cpp lib/Driver/Tools.cpp lib/Sema/SemaExpr.cpp runtime/compiler-rt/Makefile te

2012-10-16 Thread Bob Wilson
On Oct 16, 2012, at 4:48 PM, Nico Weber tha...@chromium.org wrote: On Mon, Oct 15, 2012 at 4:45 PM, Nico Weber tha...@chromium.org wrote: On Mon, Oct 15, 2012 at 4:33 PM, Daniel Dunbar daniel.dun...@gmail.com wrote: On Mon, Oct 15, 2012 at 4:21 PM, Nico Weber tha...@chromium.org wrote: On

Re: [cfe-commits] r165988 - in /cfe/trunk: include/clang/Basic/ObjCRuntime.h lib/CodeGen/CGObjC.cpp lib/Driver/ToolChains.cpp lib/Driver/Tools.cpp lib/Sema/SemaExpr.cpp runtime/compiler-rt/Makefile te

2012-10-16 Thread Bob Wilson
On Oct 16, 2012, at 9:18 PM, Nico Weber tha...@chromium.org wrote: It's also pretty disrupting to land a CL that breaks the build and then ignore the bug report for it for 2 weeks. Yeah, sorry about the slow response. Repro steps are: 1. Get a 10.6 box I don't have a 10.6 box. Certainly

Re: [cfe-commits] r165988 - in /cfe/trunk: include/clang/Basic/ObjCRuntime.h lib/CodeGen/CGObjC.cpp lib/Driver/ToolChains.cpp lib/Driver/Tools.cpp lib/Sema/SemaExpr.cpp runtime/compiler-rt/Makefile te

2012-10-16 Thread Bob Wilson
On Oct 16, 2012, at 9:25 PM, Bob Wilson bob.wil...@apple.com wrote: On Oct 16, 2012, at 9:18 PM, Nico Weber tha...@chromium.org wrote: It's also pretty disrupting to land a CL that breaks the build and then ignore the bug report for it for 2 weeks. Yeah, sorry about the slow response

[cfe-commits] r165573 - /cfe/trunk/tools/libclang/Makefile

2012-10-09 Thread Bob Wilson
Author: bwilson Date: Tue Oct 9 18:57:08 2012 New Revision: 165573 URL: http://llvm.org/viewvc/llvm-project?rev=165573view=rev Log: Revert Use a special path to place the .o files in. This reverts commit 165429 in an attempt to get our buildbots going. Modified:

[cfe-commits] r164900 - /cfe/trunk/include/clang/StaticAnalyzer/Core/PathSensitive/SValBuilder.h

2012-09-29 Thread Bob Wilson
Author: bwilson Date: Sat Sep 29 17:08:54 2012 New Revision: 164900 URL: http://llvm.org/viewvc/llvm-project?rev=164900view=rev Log: Whitespace. Modified: cfe/trunk/include/clang/StaticAnalyzer/Core/PathSensitive/SValBuilder.h Modified:

[cfe-commits] r164905 - in /cfe/trunk/lib: Basic/Targets.cpp Driver/ToolChain.cpp Driver/ToolChains.cpp Driver/Tools.cpp

2012-09-29 Thread Bob Wilson
Author: bwilson Date: Sat Sep 29 18:52:50 2012 New Revision: 164905 URL: http://llvm.org/viewvc/llvm-project?rev=164905view=rev Log: Add armv7s and some other arm variants supported by Mach-O files. Modified: cfe/trunk/lib/Basic/Targets.cpp cfe/trunk/lib/Driver/ToolChain.cpp

[cfe-commits] r164906 - /cfe/trunk/lib/Basic/Targets.cpp

2012-09-29 Thread Bob Wilson
Author: bwilson Date: Sat Sep 29 18:52:52 2012 New Revision: 164906 URL: http://llvm.org/viewvc/llvm-project?rev=164906view=rev Log: Add ARM VFPv4 feature and enable it by default for Swift. Modified: cfe/trunk/lib/Basic/Targets.cpp Modified: cfe/trunk/lib/Basic/Targets.cpp URL:

[cfe-commits] r164907 - in /cfe/trunk: include/clang/Basic/ObjCRuntime.h lib/CodeGen/CGObjC.cpp lib/Driver/ToolChains.cpp lib/Driver/Tools.cpp runtime/compiler-rt/Makefile test/CodeGenObjC/optimized-s

2012-09-29 Thread Bob Wilson
Author: bwilson Date: Sat Sep 29 18:52:58 2012 New Revision: 164907 URL: http://llvm.org/viewvc/llvm-project?rev=164907view=rev Log: Add Clang support for iOS6. Added: cfe/trunk/test/CodeGenObjC/optimized-setter-ios-device.m - copied, changed from r164906,

[cfe-commits] r164904 - in /cfe/trunk: include/clang/Basic/arm_neon.td lib/CodeGen/CGBuiltin.cpp

2012-09-29 Thread Bob Wilson
Author: bwilson Date: Sat Sep 29 18:52:48 2012 New Revision: 164904 URL: http://llvm.org/viewvc/llvm-project?rev=164904view=rev Log: Add an FMA intrinsic for ARM Neon. Modified: cfe/trunk/include/clang/Basic/arm_neon.td cfe/trunk/lib/CodeGen/CGBuiltin.cpp Modified:

Re: [cfe-commits] r164907 - in /cfe/trunk: include/clang/Basic/ObjCRuntime.h lib/CodeGen/CGObjC.cpp lib/Driver/ToolChains.cpp lib/Driver/Tools.cpp runtime/compiler-rt/Makefile test/CodeGenObjC/optimiz

2012-09-29 Thread Bob Wilson
Yeah, I'm looking at the problems now. On Sep 29, 2012, at 5:38 PM, Chandler Carruth chandl...@google.com wrote: FYI, it looks like the bots don't like one of these tests: http://bb.pgr.jp/builders/cmake-clang-x86_64-linux/builds/1849 The lab build bots haven't caught up yet, but it seems

[cfe-commits] r164908 - /cfe/trunk/test/Driver/pic.c

2012-09-29 Thread Bob Wilson
Author: bwilson Date: Sat Sep 29 19:58:28 2012 New Revision: 164908 URL: http://llvm.org/viewvc/llvm-project?rev=164908view=rev Log: Specify a full target in an attempt to appease buildbots. Modified: cfe/trunk/test/Driver/pic.c Modified: cfe/trunk/test/Driver/pic.c URL:

[cfe-commits] r164608 - /cfe/trunk/lib/Sema/SemaStmtAsm.cpp

2012-09-25 Thread Bob Wilson
Author: bwilson Date: Tue Sep 25 11:30:16 2012 New Revision: 164608 URL: http://llvm.org/viewvc/llvm-project?rev=164608view=rev Log: Examine the last, not the first, instruction from the MC matcher. If an MS-style inline asm is matched to multiple instructions, e.g., with a a WAIT-prefix, then

[cfe-commits] r164550 - /cfe/trunk/lib/Sema/SemaStmtAsm.cpp

2012-09-24 Thread Bob Wilson
Author: bwilson Date: Mon Sep 24 14:57:55 2012 New Revision: 164550 URL: http://llvm.org/viewvc/llvm-project?rev=164550view=rev Log: Fix a comment typo and clean up formatting. Modified: cfe/trunk/lib/Sema/SemaStmtAsm.cpp Modified: cfe/trunk/lib/Sema/SemaStmtAsm.cpp URL:

[cfe-commits] r164551 - in /cfe/trunk: include/clang/Basic/DiagnosticSemaKinds.td lib/Sema/SemaStmtAsm.cpp test/Sema/ms-inline-asm.c

2012-09-24 Thread Bob Wilson
Author: bwilson Date: Mon Sep 24 14:57:59 2012 New Revision: 164551 URL: http://llvm.org/viewvc/llvm-project?rev=164551view=rev Log: Replace an assertion with an error for empty __asm statements. Added: cfe/trunk/test/Sema/ms-inline-asm.c Modified:

[cfe-commits] [libcxx] r164413 - /libcxx/trunk/Makefile

2012-09-21 Thread Bob Wilson
Author: bwilson Date: Fri Sep 21 15:49:54 2012 New Revision: 164413 URL: http://llvm.org/viewvc/llvm-project?rev=164413view=rev Log: Fix installheaders target to do what it did prior to r161760. rdar://12348765 Modified: libcxx/trunk/Makefile Modified: libcxx/trunk/Makefile URL:

[cfe-commits] r163870 - /cfe/trunk/lib/Driver/Driver.cpp

2012-09-13 Thread Bob Wilson
Author: bwilson Date: Thu Sep 13 22:35:42 2012 New Revision: 163870 URL: http://llvm.org/viewvc/llvm-project?rev=163870view=rev Log: Improve the driver title as shown in the --help message. rdar://12297538 Modified: cfe/trunk/lib/Driver/Driver.cpp Modified: cfe/trunk/lib/Driver/Driver.cpp

[cfe-commits] r161444 - in /cfe/trunk: lib/Driver/Tools.cpp test/Driver/arclite-link.c

2012-08-07 Thread Bob Wilson
Author: bwilson Date: Tue Aug 7 14:58:00 2012 New Revision: 161444 URL: http://llvm.org/viewvc/llvm-project?rev=161444view=rev Log: Don't complain about -fobjc-link-runtime being unused with -fobjc-arc. If you build with -fobjc-arc, then -fobjc-link-runtime is implied but we don't need to warn

Re: [cfe-commits] [patch] Allow complex long double to be homogeneous in ARM hardfloat

2012-07-20 Thread Bob Wilson
On Jul 20, 2012, at 1:00 PM, Tim Northover t.p.northo...@gmail.com wrote: Hi, Under ARM's AAPCS long double is just another name for double which means that it can be part of a homogeneous aggregate and get passed in VFP registers; but the function isHomogeneousAggregate which is part of

Re: [cfe-commits] r160484 - in /cfe/trunk: lib/Driver/ToolChains.cpp test/Driver/ios-simulator-arcruntime.c

2012-07-19 Thread Bob Wilson
: Hi Bob, How this patch would impact if user ran into the known issue previously and passed down, say, 10.7 to fix the problem? Would appearance of two deployment target be handled properly by cc1? - Fariborz On Jul 18, 2012, at 6:35 PM, Bob Wilson wrote: Author: bwilson Date: Wed Jul

[cfe-commits] r160484 - in /cfe/trunk: lib/Driver/ToolChains.cpp test/Driver/ios-simulator-arcruntime.c

2012-07-18 Thread Bob Wilson
Author: bwilson Date: Wed Jul 18 20:35:55 2012 New Revision: 160484 URL: http://llvm.org/viewvc/llvm-project?rev=160484view=rev Log: Force the OS X version to 10.6 for old-style simulator builds. The hack of recognizing a -D__IPHONE_OS_VERSION_MIN_REQUIRED option in place of

[cfe-commits] r160491 - in /cfe/trunk: include/clang/Basic/LangOptions.def include/clang/Driver/Options.td lib/Driver/Tools.cpp lib/Frontend/CompilerInvocation.cpp lib/Frontend/InitPreprocessor.cpp te

2012-07-18 Thread Bob Wilson
Author: bwilson Date: Wed Jul 18 22:52:53 2012 New Revision: 160491 URL: http://llvm.org/viewvc/llvm-project?rev=160491view=rev Log: Define __FINITE_MATH_ONLY__ based on -ffast-math and -ffinite-math-only. This macro was being unconditionally set to zero, preceded by a FIXME comment. This fixes

Re: [cfe-commits] [PATCH] - gcc compat: -ffast-math does not set __FINITE_MATH_ONLY__.

2012-07-18 Thread Bob Wilson
LGTM. Committed as svn r160491 On Jul 18, 2012, at 5:00 PM, Michael Gottesman mgottes...@apple.com wrote: The attached patch fixes the following gcc compatibility issues: 1.-ffast-math does not set the __FINITE_MATH_ONLY__ preprocessor symbol to 1, while llvm-gcc does do so. 2.

[cfe-commits] r159683 - in /cfe/trunk: lib/Driver/Tools.cpp test/Driver/darwin-ld.c

2012-07-03 Thread Bob Wilson
Author: bwilson Date: Tue Jul 3 15:42:10 2012 New Revision: 159683 URL: http://llvm.org/viewvc/llvm-project?rev=159683view=rev Log: When using -pg targeting OS X 10.8, pass -no_new_main to the linker. By default on OS X 10.8, we don't link with a crt1.o file and the linker knows to use _main as

[cfe-commits] r159696 - /cfe/trunk/lib/Driver/Tools.cpp

2012-07-03 Thread Bob Wilson
Author: bwilson Date: Tue Jul 3 19:18:41 2012 New Revision: 159696 URL: http://llvm.org/viewvc/llvm-project?rev=159696view=rev Log: Move a comment from the commit message into the code. Modified: cfe/trunk/lib/Driver/Tools.cpp Modified: cfe/trunk/lib/Driver/Tools.cpp URL:

[cfe-commits] r158370 - /cfe/trunk/test/Driver/clang-translation.c

2012-06-12 Thread Bob Wilson
Author: bwilson Date: Tue Jun 12 15:22:45 2012 New Revision: 158370 URL: http://llvm.org/viewvc/llvm-project?rev=158370view=rev Log: Fix test to work when clang is built without powerpc64 support. Modified: cfe/trunk/test/Driver/clang-translation.c Modified:

Re: [cfe-commits] r156993 - /cfe/trunk/include/clang/Basic/DiagnosticDriverKinds.td

2012-05-18 Thread Bob Wilson
On May 17, 2012, at 8:45 AM, Chad Rosier mcros...@apple.com wrote: Author: mcrosier Date: Thu May 17 10:45:13 2012 New Revision: 156993 URL: http://llvm.org/viewvc/llvm-project?rev=156993view=rev Log: [driver] Reword the warning message for missing value to a joined argument.

[cfe-commits] r156841 - in /cfe/trunk: lib/Driver/Tools.cpp test/Driver/arclite-link.c

2012-05-15 Thread Bob Wilson
Author: bwilson Date: Tue May 15 13:57:39 2012 New Revision: 156841 URL: http://llvm.org/viewvc/llvm-project?rev=156841view=rev Log: Do not link with Objective-C libraries with -nostdlib or -nodefaultlibs. rdar://problem/11433499 Modified: cfe/trunk/lib/Driver/Tools.cpp

Re: [cfe-commits] r156630 - in /cfe/trunk: include/clang/Basic/BuiltinsHexagon.def lib/CodeGen/CGBuiltin.cpp

2012-05-15 Thread Bob Wilson
On May 11, 2012, at 11:13 PM, John McCall rjmcc...@apple.com wrote: On May 11, 2012, at 8:57 PM, Sirish Pande wrote: I would love to have a macro-generated switch statement - but that would require a target dependent file - which I don't think clang folks would like in lib/CodeGen. Since

[cfe-commits] r156487 - /cfe/trunk/lib/Driver/Tools.cpp

2012-05-09 Thread Bob Wilson
Author: bwilson Date: Wed May 9 12:53:10 2012 New Revision: 156487 URL: http://llvm.org/viewvc/llvm-project?rev=156487view=rev Log: Ignore a generic return value from getHostCPUName. rdar://problem/11314502 Modified: cfe/trunk/lib/Driver/Tools.cpp Modified: cfe/trunk/lib/Driver/Tools.cpp

Re: [cfe-commits] r156487 - /cfe/trunk/lib/Driver/Tools.cpp

2012-05-09 Thread Bob Wilson
On May 9, 2012, at 11:01 AM, Chandler Carruth chandl...@google.com wrote: On Wed, May 9, 2012 at 10:53 AM, Bob Wilson bob.wil...@apple.com wrote: Author: bwilson Date: Wed May 9 12:53:10 2012 New Revision: 156487 URL: http://llvm.org/viewvc/llvm-project?rev=156487view=rev Log: Ignore

[cfe-commits] r156092 - /cfe/trunk/lib/Sema/SemaChecking.cpp

2012-05-03 Thread Bob Wilson
Author: bwilson Date: Thu May 3 14:47:19 2012 New Revision: 156092 URL: http://llvm.org/viewvc/llvm-project?rev=156092view=rev Log: Disable -Wformat-extra-args for arguments defined in system headers. Some of the NSAssert macros in OS X 10.7 are implemented in a way that adds extra arguments

[cfe-commits] r156132 - in /cfe/trunk/test/Sema: Inputs/format-unused-system-args.h format-strings.c

2012-05-03 Thread Bob Wilson
Author: bwilson Date: Thu May 3 18:38:51 2012 New Revision: 156132 URL: http://llvm.org/viewvc/llvm-project?rev=156132view=rev Log: Add a test for r156092. Added: cfe/trunk/test/Sema/Inputs/format-unused-system-args.h Modified: cfe/trunk/test/Sema/format-strings.c Added:

[cfe-commits] r156149 - /cfe/trunk/lib/Driver/Tools.cpp

2012-05-03 Thread Bob Wilson
Author: bwilson Date: Thu May 3 22:09:46 2012 New Revision: 156149 URL: http://llvm.org/viewvc/llvm-project?rev=156149view=rev Log: Remove obsolete code for finding a fallback iOS SDK. rdar://problem/11378535 Modified: cfe/trunk/lib/Driver/Tools.cpp Modified:

[cfe-commits] r156058 - in /cfe/trunk/lib/Driver: ToolChains.cpp ToolChains.h Tools.cpp

2012-05-02 Thread Bob Wilson
Author: bwilson Date: Thu May 3 00:23:52 2012 New Revision: 156058 URL: http://llvm.org/viewvc/llvm-project?rev=156058view=rev Log: Rip out old code for finding libraries in GCC's directories. Modified: cfe/trunk/lib/Driver/ToolChains.cpp cfe/trunk/lib/Driver/ToolChains.h

Re: [cfe-commits] r155263 - /cfe/trunk/lib/Driver/Tools.cpp

2012-04-23 Thread Bob Wilson
On Apr 20, 2012, at 5:33 PM, Chandler Carruth chandl...@google.com wrote: On Fri, Apr 20, 2012 at 5:21 PM, Bob Wilson bob.wil...@apple.com wrote: Link with Foundation whenever -fobjc-link-runtime is used. rdar://10976177 The check for excluding libarclite on i386 MacOSX should not apply

[cfe-commits] r155353 - /cfe/trunk/test/Driver/arclite-link.c

2012-04-23 Thread Bob Wilson
Author: bwilson Date: Mon Apr 23 11:22:22 2012 New Revision: 155353 URL: http://llvm.org/viewvc/llvm-project?rev=155353view=rev Log: Add a test for svn r155263. Modified: cfe/trunk/test/Driver/arclite-link.c Modified: cfe/trunk/test/Driver/arclite-link.c URL:

[cfe-commits] r155263 - /cfe/trunk/lib/Driver/Tools.cpp

2012-04-20 Thread Bob Wilson
Author: bwilson Date: Fri Apr 20 19:21:42 2012 New Revision: 155263 URL: http://llvm.org/viewvc/llvm-project?rev=155263view=rev Log: Link with Foundation whenever -fobjc-link-runtime is used. rdar://10976177 The check for excluding libarclite on i386 MacOSX should not apply to Foundation.

Re: [cfe-commits] Fix handling of ARM homogenous aggregates

2012-04-12 Thread Bob Wilson
There's no need to add a blank line at the end of CGCall.cpp, but otherwise, it looks fine to me. On Apr 11, 2012, at 11:26 PM, Anton Korobeynikov an...@korobeynikov.info wrote: Ping! On Mon, Apr 9, 2012 at 18:03, Anton Korobeynikov an...@korobeynikov.info wrote: Hi Bob et al Attached

Re: [cfe-commits] r153854 - /cfe/trunk/lib/Driver/Tools.cpp

2012-04-04 Thread Bob Wilson
On Apr 1, 2012, at 4:47 PM, David Blaikie wrote: On Sun, Apr 1, 2012 at 4:03 PM, Bob Wilson bob.wil...@apple.com wrote: Author: bwilson Date: Sun Apr 1 18:03:29 2012 New Revision: 153854 URL: http://llvm.org/viewvc/llvm-project?rev=153854view=rev Log: Ignore unused --serialize

[cfe-commits] r154001 - /cfe/trunk/test/Driver/apple-kext-i386.cpp

2012-04-04 Thread Bob Wilson
Author: bwilson Date: Wed Apr 4 02:06:30 2012 New Revision: 154001 URL: http://llvm.org/viewvc/llvm-project?rev=154001view=rev Log: Add a testcase for svn r153854. Modified: cfe/trunk/test/Driver/apple-kext-i386.cpp Modified: cfe/trunk/test/Driver/apple-kext-i386.cpp URL:

Re: [cfe-commits] r153854 - /cfe/trunk/lib/Driver/Tools.cpp

2012-04-04 Thread Bob Wilson
On Apr 1, 2012, at 4:50 PM, Chandler Carruth wrote: On Sun, Apr 1, 2012 at 4:03 PM, Bob Wilson bob.wil...@apple.com wrote: Ignore unused --serialize-diagnostics option when using llvm-gcc. llvm-gcc doesn't handle --serialize-diagnostics so when compiling i386 kernel/kext code with -Werror

[cfe-commits] r154018 - /cfe/trunk/test/Driver/apple-kext-i386.cpp

2012-04-04 Thread Bob Wilson
Author: bwilson Date: Wed Apr 4 09:43:40 2012 New Revision: 154018 URL: http://llvm.org/viewvc/llvm-project?rev=154018view=rev Log: Simplify test a bit now that it's not actually running cc1plus. Modified: cfe/trunk/test/Driver/apple-kext-i386.cpp Modified:

[cfe-commits] r153854 - /cfe/trunk/lib/Driver/Tools.cpp

2012-04-01 Thread Bob Wilson
Author: bwilson Date: Sun Apr 1 18:03:29 2012 New Revision: 153854 URL: http://llvm.org/viewvc/llvm-project?rev=153854view=rev Log: Ignore unused --serialize-diagnostics option when using llvm-gcc. llvm-gcc doesn't handle --serialize-diagnostics so when compiling i386 kernel/kext code with

Re: [cfe-commits] [llvm-commits] Fix bug 11753 - Incorrect code generated for 64bit types on ARM

2012-03-31 Thread Bob Wilson
The old APCS ABI does not require aligned register pairs for i64 values. On Mar 30, 2012, at 1:01 PM, Weiming Zhao wrote: Hi Sandeep, I can't find the spec for aps-gnu. But my tests show that, in term of passing i64, it requires an aligned register pair, which is the same requirement in

Re: [cfe-commits] Fix handling of ARM homogenous aggregates

2012-03-30 Thread Bob Wilson
On Mar 28, 2012, at 12:53 AM, Anton Korobeynikov an...@korobeynikov.info wrote: Hello Everyone Attached is the patch which (I hope) fixes almost all remaining issues with handling of homogenous aggregates. In particular: - Unions are now handled - C++ case is now handled (previously

Re: [cfe-commits] Fix handling of ARM homogenous aggregates

2012-03-30 Thread Bob Wilson
On Mar 30, 2012, at 11:08 AM, Anton Korobeynikov an...@korobeynikov.info wrote: Hi Bob * Given that ABIArgInfo::Expand will only be used for unions that are homogeneous, does it really make sense to put code in CodeGenTypes::GetExpandedTypes and CodeGenFunction::ExpandTypeToArgs to

<    1   2   3   4   5   6   7   8   9   >