Re: [PATCH] D14629: [analyzer] Configuration file for scan-build.

2015-11-19 Thread Laszlo Nagy via cfe-commits
rizsotto.mailinglist added a comment. I think the `scan-build` user experience would be improved by config file! And I really like how the `clang-tidy` guys were doing it. (In case if you are not familiar with it, copy from the help output) Configuration files: clang-tidy attempts to

Re: [PATCH] D14804: [clang] Disable Unicode in asm files

2015-11-19 Thread Renato Golin via cfe-commits
rengolin accepted this revision. rengolin added a comment. This revision is now accepted and ready to land. LGTM, too. Thanks! Repository: rL LLVM http://reviews.llvm.org/D14804 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

Re: [PATCH] D13351: [Power PC] add soft float support for ppc32

2015-11-19 Thread Strahinja Petrovic via cfe-commits
spetrovic added a comment. ping http://reviews.llvm.org/D13351 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r253551 - PR10235: support for vector mode attributes + warning, by Dmitry Polukhin.

2015-11-19 Thread Alexey Bataev via cfe-commits
Author: abataev Date: Thu Nov 19 04:13:11 2015 New Revision: 253551 URL: http://llvm.org/viewvc/llvm-project?rev=253551=rev Log: PR10235: support for vector mode attributes + warning, by Dmitry Polukhin. Add support for vector mode attributes like "attribute((mode(V4SF)))". Also add warning

[libcxx] r253592 - Fix some mistakes in the synopsis. No functional change. Thanks to K-ballo for the patch

2015-11-19 Thread Marshall Clow via cfe-commits
Author: marshall Date: Thu Nov 19 13:41:04 2015 New Revision: 253592 URL: http://llvm.org/viewvc/llvm-project?rev=253592=rev Log: Fix some mistakes in the synopsis. No functional change. Thanks to K-ballo for the patch Modified: libcxx/trunk/include/array Modified:

[libcxx] r253593 - Fix some mistakes in the and synopses. No functional change. Thannks to K-ballo for the patch

2015-11-19 Thread Marshall Clow via cfe-commits
Author: marshall Date: Thu Nov 19 13:45:29 2015 New Revision: 253593 URL: http://llvm.org/viewvc/llvm-project?rev=253593=rev Log: Fix some mistakes in the and synopses. No functional change. Thannks to K-ballo for the patch Modified: libcxx/trunk/include/tuple

r253595 - Update clang tests to accomendate discriminator changes.

2015-11-19 Thread Dehao Chen via cfe-commits
Author: dehao Date: Thu Nov 19 13:53:16 2015 New Revision: 253595 URL: http://llvm.org/viewvc/llvm-project?rev=253595=rev Log: Update clang tests to accomendate discriminator changes. Summary: The discriminator change in http://reviews.llvm.org/D14738 will fail these clang tests. Update the

Re: [PATCH] D9600: Add scan-build python implementation

2015-11-19 Thread Devin Coughlin via cfe-commits
dcoughlin added a comment. Hi Laszlo, thanks for the update! Some high-level questions/comments (and various small things I noticed inline). - I tried running scan-build in interposition mode (i.e., uncommenting out #"$SCRIPT_DIR/analyze-build" $@ in scan-build) and got python compile errors.

Re: [PATCH] D14737: Convert some ObjC msgSends to runtime calls

2015-11-19 Thread John McCall via cfe-commits
rjmccall added a comment. The casts done by emitARCValueOperation will handle the input, but they don't quite handle the result properly. The right test case here is a method named "retain" that's declared to return something completely unrelated to its receiver type, e.g. @class A;

Re: r253595 - Update clang tests to accomendate discriminator changes.

2015-11-19 Thread David Blaikie via cfe-commits
On Thu, Nov 19, 2015 at 11:53 AM, Dehao Chen via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Author: dehao > Date: Thu Nov 19 13:53:16 2015 > New Revision: 253595 > > URL: http://llvm.org/viewvc/llvm-project?rev=253595=rev > Log: > Update clang tests to accomendate discriminator changes. >

r253554 - clangFrontend: [PR25565] Quick fix for dependencies on Attributes.inc.

2015-11-19 Thread NAKAMURA Takumi via cfe-commits
Author: chapuni Date: Thu Nov 19 04:59:48 2015 New Revision: 253554 URL: http://llvm.org/viewvc/llvm-project?rev=253554=rev Log: clangFrontend: [PR25565] Quick fix for dependencies on Attributes.inc. FIXME: Attributes.inc may be an independent target. Differential Revision:

[PATCH] D14814: [libcxx] Use __make_integer_seq builtin when available

2015-11-19 Thread Agustín Bergé via cfe-commits
K-ballo created this revision. K-ballo added reviewers: mclow.lists, EricWF, majnemer, rsmith. K-ballo added a subscriber: cfe-commits. Use the `__make_integer_seq` builtin, introduced by r252036, when available. This allows for an incredibly fast `std::make_integer_sequence` implementation.

Re: [PATCH] D14616: [libcxx] Replace TEST_HAS_NO_EXCEPTIONS with _LIBCPP_NO_EXCEPTIONS [NFC]

2015-11-19 Thread Asiri Rathnayake via cfe-commits
rmaprath abandoned this revision. rmaprath added a comment. Makes sense. Thanks. http://reviews.llvm.org/D14616 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r253647 - [Myriad]: handle Preprocess job action (-E)

2015-11-19 Thread Douglas Katzman via cfe-commits
Author: dougk Date: Thu Nov 19 22:58:12 2015 New Revision: 253647 URL: http://llvm.org/viewvc/llvm-project?rev=253647=rev Log: [Myriad]: handle Preprocess job action (-E) Modified: cfe/trunk/lib/Driver/ToolChains.cpp cfe/trunk/lib/Driver/Tools.cpp

r253641 - [coroutines] Per latest wording paper, co_* are no longer permitted in any

2015-11-19 Thread Richard Smith via cfe-commits
Author: rsmith Date: Thu Nov 19 20:54:01 2015 New Revision: 253641 URL: http://llvm.org/viewvc/llvm-project?rev=253641=rev Log: [coroutines] Per latest wording paper, co_* are no longer permitted in any unevaluated operands. Modified: cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.td

r253636 - [CMake] Add a specific 'install-clang-headers' target.

2015-11-19 Thread Argyrios Kyrtzidis via cfe-commits
Author: akirtzidis Date: Thu Nov 19 20:24:03 2015 New Revision: 253636 URL: http://llvm.org/viewvc/llvm-project?rev=253636=rev Log: [CMake] Add a specific 'install-clang-headers' target. Modified: cfe/trunk/lib/Headers/CMakeLists.txt Modified: cfe/trunk/lib/Headers/CMakeLists.txt URL:

r253645 - [libclang] Make sure to use the raw module format for libclang parsing.

2015-11-19 Thread Argyrios Kyrtzidis via cfe-commits
Author: akirtzidis Date: Thu Nov 19 21:36:21 2015 New Revision: 253645 URL: http://llvm.org/viewvc/llvm-project?rev=253645=rev Log: [libclang] Make sure to use the raw module format for libclang parsing. Fixes crash when passing '-gmodules' in the compiler options. rdar://23588717 Modified:

Re: [PATCH] D14467: [MS] Fix for bug 25013 - #pragma vtordisp is unknown inside functions.

2015-11-19 Thread Alexey Bataev via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL253650: [MS] Fix for bug 25013 - #pragma vtordisp is unknown inside functions, by… (authored by ABataev). Changed prior to commit: http://reviews.llvm.org/D14467?vs=40380=40745#toc Repository: rL

r253650 - [MS] Fix for bug 25013 - #pragma vtordisp is unknown inside functions, by Denis Zobnin.

2015-11-19 Thread Alexey Bataev via cfe-commits
Author: abataev Date: Fri Nov 20 01:02:57 2015 New Revision: 253650 URL: http://llvm.org/viewvc/llvm-project?rev=253650=rev Log: [MS] Fix for bug 25013 - #pragma vtordisp is unknown inside functions, by Denis Zobnin. This patch adds support of #pragma vtordisp inside functions in attempt to

Re: [PATCH] D14796: Preserve exceptions information during calls code generation.

2015-11-19 Thread Samuel Antao via cfe-commits
sfantao added inline comments. Comment at: lib/CodeGen/CGExpr.cpp:3751 @@ +3750,3 @@ + // Preserve the non-canonical function type because things like exception + // specifications disappear in the canonical type. That information is useful + // to drive the generation of more

Re: [PATCH] D14653: [libcxx] Introduce the mechanism for fixing -fno-exceptions test failures.

2015-11-19 Thread scott douglass via cfe-commits
scott-0 added a subscriber: scott-0. scott-0 added a comment. It would be nice if the no-exceptions library called an intermediate helper, for example `__libcxx_noexceptions_abort()`, instead of calling `abort()` directly. Then the user and the tests could provide a replacement for

Re: [PATCH] D13980: Add "x87" in x86 target feature map

2015-11-19 Thread Andrey Turetskiy via cfe-commits
aturetsk added a comment. Hello Richard, Thank for the review. Comment at: lib/Basic/Targets.cpp:2538-2539 @@ -2537,1 +2537,4 @@ + // All X86 processors but i386 have X87. + if (getCPUKind(CPU) != CK_i386) +setFeatureEnabledImpl(Features, "x87", true);