Re: [PATCH] D13126: New static analyzer checker for loss of sign/precision

2015-12-16 Thread Daniel Marjamäki via cfe-commits
danielmarjamaki added a comment. ping. I would like to have a review on this. http://reviews.llvm.org/D13126 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r255757 - clang-format: Extend header sort category implementation.

2015-12-16 Thread Daniel Jasper via cfe-commits
Author: djasper Date: Wed Dec 16 04:10:16 2015 New Revision: 255757 URL: http://llvm.org/viewvc/llvm-project?rev=255757&view=rev Log: clang-format: Extend header sort category implementation. Specifically, it is sometimes necessary to keep certain #includes as the first #include, even before the

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

2015-12-16 Thread Asiri Rathnayake via cfe-commits
rmaprath updated this revision to Diff 42977. rmaprath added a comment. Two minor additions: - Introduced a `TEST_THROW` macro for those tests that need to throw exceptions (Note:- not using the `_LIBCPP_THROW` macro here as we want to keep the tests independent of the standard library implemen

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

2015-12-16 Thread Asiri Rathnayake via cfe-commits
rmaprath marked 2 inline comments as done. rmaprath added a comment. http://reviews.llvm.org/D14653 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] r255758 - [clang-tidy] Fix a crash in misc-new-delete-overloads

2015-12-16 Thread Alexander Kornienko via cfe-commits
Author: alexfh Date: Wed Dec 16 04:58:14 2015 New Revision: 255758 URL: http://llvm.org/viewvc/llvm-project?rev=255758&view=rev Log: [clang-tidy] Fix a crash in misc-new-delete-overloads Modified: clang-tools-extra/trunk/clang-tidy/misc/NewDeleteOverloadsCheck.cpp clang-tools-extra/trunk/

Re: r254195 - [x86] Exclusion of incorrect include headers paths for MCU target

2015-12-16 Thread Andrey Bokhanko via cfe-commits
Reid, Hal, I'm really sorry for this -- indeed what I committed (by a mistake) is still WIP. Yours, Andrey On Sat, Dec 5, 2015 at 4:56 AM, Reid Kleckner wrote: > It looks like the wrong patch got committed with the original description of > D14954. I reverted it in r254823. > > Feel free to co

Re: [PATCH] D15040: [ARM] Add command-line options for ARMv8.2-A

2015-12-16 Thread Oliver Stannard via cfe-commits
olista01 added a reviewer: rengolin. olista01 added a comment. Ping? http://reviews.llvm.org/D15040 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D15561: [OpenMP] Remove explicit call for implicit barrier

2015-12-16 Thread Jonas Hahnfeld via cfe-commits
Hahnfeld created this revision. Hahnfeld added reviewers: ABataev, hfinkel, kkwli0, rjmccall. Hahnfeld added a subscriber: cfe-commits. `#pragma omp parallel` needs an implicit barrier that is currently done by an explicit call to `__kmpc_barrier`. However, the runtime already ensures a barrier

Re: [PATCH] D15561: [OpenMP] Remove explicit call for implicit barrier

2015-12-16 Thread Jonas Hahnfeld via cfe-commits
Hahnfeld added a comment. I didn't adjust the tests, I'm sorry! Will do so if the change is acceptable in general... http://reviews.llvm.org/D15561 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listin

[clang-tools-extra] r255765 - [clang-tidy] Don't use diag() for debug output

2015-12-16 Thread Alexander Kornienko via cfe-commits
Author: alexfh Date: Wed Dec 16 07:19:08 2015 New Revision: 255765 URL: http://llvm.org/viewvc/llvm-project?rev=255765&view=rev Log: [clang-tidy] Don't use diag() for debug output Modified: clang-tools-extra/trunk/clang-tidy/readability/BracesAroundStatementsCheck.cpp Modified: clang-tools

Re: [PATCH] D15448: [analyzer] SVal Visitor.

2015-12-16 Thread Artem Dergachev via cfe-commits
NoQ added a comment. Good point, will try to make a .def file. There's a tiny inconsistency with `SVal` naming that would most likely need to be fixed in this approach: nonloc::SymbolVal => SymbolValKind loc::MemRegionVal => MemRegionKind // no "Val"! Hmm, maybe make a .def file for symbol

Re: [clang-tools-extra] r255758 - [clang-tidy] Fix a crash in misc-new-delete-overloads

2015-12-16 Thread Aaron Ballman via cfe-commits
On Wed, Dec 16, 2015 at 5:58 AM, Alexander Kornienko via cfe-commits wrote: > Author: alexfh > Date: Wed Dec 16 04:58:14 2015 > New Revision: 255758 > > URL: http://llvm.org/viewvc/llvm-project?rev=255758&view=rev > Log: > [clang-tidy] Fix a crash in misc-new-delete-overloads Good catch, thanks!

r255766 - [x86] Exclusion of incorrect include headers paths for MCU target

2015-12-16 Thread Andrey Bokhanko via cfe-commits
Author: asbokhan Date: Wed Dec 16 07:27:38 2015 New Revision: 255766 URL: http://llvm.org/viewvc/llvm-project?rev=255766&view=rev Log: [x86] Exclusion of incorrect include headers paths for MCU target Exclusion of /usr/include and /usr/local/include headers paths for MCU target. Differential Rev

Re: r254195 - [x86] Exclusion of incorrect include headers paths for MCU target

2015-12-16 Thread Andrey Bokhanko via cfe-commits
Correct patch committed in r255766. Yours, Andrey On Wed, Dec 16, 2015 at 2:11 PM, Andrey Bokhanko wrote: > Reid, Hal, > > I'm really sorry for this -- indeed what I committed (by a mistake) is > still WIP. > > Yours, > Andrey > > > On Sat, Dec 5, 2015 at 4:56 AM, Reid Kleckner wrote: >> It loo

Re: [Lldb-commits] Buildbot numbers for week of 12/06/2015 - 12/12/2015

2015-12-16 Thread Kamil Rytarowski via cfe-commits
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 On 15.12.2015 20:23, Galina Kistanova via lldb-commits wrote: > Hello everyone, > > Below are some buildbot numbers for the last week of 12/06/2015 - > 12/12/2015. > > Thanks > > Galina > > > > Top 10 fastest builders(not docs): > > lldb-amd6

Re: [PATCH] D15539: [libcxxabi] Reducing stack usage of test

2015-12-16 Thread Craig, Ben via cfe-commits
As an alternative, would it be acceptable to heap allocate these objects, using the original padding numbers? On 12/15/2015 5:11 PM, Howard Hinnant wrote: On Dec 15, 2015, at 5:30 PM, Jonathan Roelofs wrote: Could these large padding things be related to the fact that the test is used as a

Re: [PATCH] D15539: [libcxxabi] Reducing stack usage of test

2015-12-16 Thread Ben Craig via cfe-commits
bcraig added a subscriber: bcraig. bcraig added a comment. As an alternative, would it be acceptable to heap allocate these objects, using the original padding numbers? http://reviews.llvm.org/D15539 ___ cfe-commits mailing list cfe-commits@lists.l

Re: [PATCH] D15222: [Patch][Profile] add “--dependent-lib= libclang_rt.profile-x86_64.a” to the CC1 command line when enabling code coverage on PS4

2015-12-16 Thread Ying Yi via cfe-commits
MaggieYi added a comment. Could someone commit it for me please (as I do not have commit access)? Thanks http://reviews.llvm.org/D15222 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commit

Re: [PATCH] D15070: [mips] Added support for -Wa, -mips32 and similar.

2015-12-16 Thread Daniel Sanders via cfe-commits
dsanders requested changes to this revision. This revision now requires changes to proceed. Comment at: lib/Driver/Tools.cpp:2501-2524 @@ -2500,2 +2500,26 @@ } + +const char *Feature = llvm::StringSwitch(Value) + .Case("-mips1", "+

Re: [PATCH] D12834: add gcc abi_tag support

2015-12-16 Thread Evangelos Foutras via cfe-commits
foutrelis added a comment. We have received a few reports of clang crashes after applying the abi_tag support patch to our llvm/clang package in Arch Linux. Compiling F1182170: test.cc with `clang++ -std=c++14 -c test.cc` should trigger the segmentation fault

RE: [Patch][OpenCL] Custom atomic Builtin check ignores address space of a non-atomic pointer

2015-12-16 Thread Anastasia Stulova via cfe-commits
Hi Pekka, Re-attaching as a full patch again as something went wrong earlier with the diff wrapping in the email. Thanks, Anastasia -Original Message- From: cfe-commits [mailto:cfe-commits-boun...@lists.llvm.org] On Behalf Of Anastasia Stulova via cfe-commits Sent: 07 December 2015 17:

[PATCH] D15571: Add namespace support to add_new_check.py

2015-12-16 Thread Cong Liu via cfe-commits
congliu created this revision. congliu added a reviewer: alexfh. congliu added a subscriber: cfe-commits. Without namespace you can not create checks with same name in different modules http://reviews.llvm.org/D15571 Files: clang-tidy/add_new_check.py Index: clang-tidy/add_new_check.py ==

Re: [PATCH] D15571: Add namespace support to add_new_check.py

2015-12-16 Thread Alexander Kornienko via cfe-commits
alexfh added inline comments. Comment at: clang-tidy/add_new_check.py:99 @@ -97,1 +98,3 @@ + 'check_name_dashes': check_name_dashes, + 'namespace': module}) Thinking about this once again, let's replace `namespace` with `module` for consistency. ==

Re: [PATCH] D15571: Add namespace support to add_new_check.py

2015-12-16 Thread Cong Liu via cfe-commits
congliu updated this revision to Diff 43007. congliu added a comment. Replaced namespace with module; Removed extra space after module at line 103 http://reviews.llvm.org/D15571 Files: clang-tidy/add_new_check.py Index: clang-tidy/add_new_check.py

Re: [PATCH] D15571: Add namespace support to add_new_check.py

2015-12-16 Thread Alexander Kornienko via cfe-commits
alexfh accepted this revision. alexfh added a comment. This revision is now accepted and ready to land. Looks good. Thank you! I'll commit the patch for you. http://reviews.llvm.org/D15571 ___ cfe-commits mailing list cfe-commits@lists.llvm.org htt

Re: [PATCH] D15571: Add namespace support to add_new_check.py

2015-12-16 Thread Cong Liu via cfe-commits
congliu marked 2 inline comments as done. congliu added a comment. Replace namespace with module -- Done. Remove extra space after module -- Done. http://reviews.llvm.org/D15571 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.ll

Re: [PATCH] D15571: Add namespace support to add_new_check.py

2015-12-16 Thread Alexander Kornienko via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL255770: [clang-tidy] Add namespace support to add_new_check.py (authored by alexfh). Changed prior to commit: http://reviews.llvm.org/D15571?vs=43007&id=43008#toc Repository: rL LLVM http://reviews.

[clang-tools-extra] r255770 - [clang-tidy] Add namespace support to add_new_check.py

2015-12-16 Thread Alexander Kornienko via cfe-commits
Author: alexfh Date: Wed Dec 16 09:05:27 2015 New Revision: 255770 URL: http://llvm.org/viewvc/llvm-project?rev=255770&view=rev Log: [clang-tidy] Add namespace support to add_new_check.py Summary: Without namespace you can not create checks with same name in different modules Reviewers: alexfh

[clang-tools-extra] r255772 - [clang-tidy] google-readability-namespace-comments: ignore stray semicolons

2015-12-16 Thread Alexander Kornienko via cfe-commits
Author: alexfh Date: Wed Dec 16 09:44:42 2015 New Revision: 255772 URL: http://llvm.org/viewvc/llvm-project?rev=255772&view=rev Log: [clang-tidy] google-readability-namespace-comments: ignore stray semicolons Modified: clang-tools-extra/trunk/clang-tidy/readability/NamespaceCommentCheck.cpp

r255773 - Make matchers thread compatible, so we can pass them to different threads and run them on multiple translation units in parallel.

2015-12-16 Thread Manuel Klimek via cfe-commits
Author: klimek Date: Wed Dec 16 09:48:58 2015 New Revision: 255773 URL: http://llvm.org/viewvc/llvm-project?rev=255773&view=rev Log: Make matchers thread compatible, so we can pass them to different threads and run them on multiple translation units in parallel. Modified: cfe/trunk/include/c

Re: r254195 - [x86] Exclusion of incorrect include headers paths for MCU target

2015-12-16 Thread Reid Kleckner via cfe-commits
No problem, stuff happens. :) On Wed, Dec 16, 2015 at 5:32 AM, Andrey Bokhanko wrote: > Correct patch committed in r255766. > > Yours, > Andrey > > On Wed, Dec 16, 2015 at 2:11 PM, Andrey Bokhanko > wrote: > > Reid, Hal, > > > > I'm really sorry for this -- indeed what I committed (by a mistake

Re: [PATCH] D15528: Teach clang-tidy how to -Werror checks.

2015-12-16 Thread Alexander Kornienko via cfe-commits
alexfh added a comment. Thank you for explaining. The use case seems to be important enough to support it. And the solution seems to be good for now. A few concerns: 1. `Werrors` isn't a good name for this. The only reason why a similar thing is called `-Werror` in compilers is that they use `-

Re: [PATCH] D15222: [Patch][Profile] add “--dependent-lib= libclang_rt.profile-x86_64.a” to the CC1 command line when enabling code coverage on PS4

2015-12-16 Thread Paul Robinson via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL255784: [PS4][Profile] add "--dependent-lib=libclang_rt.profile-x86_64.a" to (authored by probinson). Changed prior to commit: http://reviews.llvm.org/D15222?vs=42874&id=43020#toc Repository: rL LLVM

r255784 - [PS4][Profile] add "--dependent-lib=libclang_rt.profile-x86_64.a" to

2015-12-16 Thread Paul Robinson via cfe-commits
Author: probinson Date: Wed Dec 16 11:25:27 2015 New Revision: 255784 URL: http://llvm.org/viewvc/llvm-project?rev=255784&view=rev Log: [PS4][Profile] add "--dependent-lib=libclang_rt.profile-x86_64.a" to the CC1 command line when enabling code coverage. Patch by Ying Yi! Differential Revision:

Re: [PATCH] D15222: [Patch][Profile] add “--dependent-lib= libclang_rt.profile-x86_64.a” to the CC1 command line when enabling code coverage on PS4

2015-12-16 Thread Paul Robinson via cfe-commits
probinson added a comment. In http://reviews.llvm.org/D15222#311896, @MaggieYi wrote: > Could someone commit it for me please (as I do not have commit access)? Thanks Done. Thanks Maggie! Repository: rL LLVM http://reviews.llvm.org/D15222 ___

[PATCH] D15577: [PS4] Fix the unit test to be compatible with clang driver. NFC

2015-12-16 Thread Sumanth Gundapaneni via cfe-commits
sgundapa created this revision. sgundapa added reviewers: eugenis, filcab. sgundapa added a subscriber: cfe-commits. ".exe" extension is inherently checked by llvm::fs::can_execute() This patch fixes the linker extension in clang driver and updates the unit test to accommodate the the check string

Re: [PATCH] D12614: [OpenMP] Offloading descriptor registration and device codegen.

2015-12-16 Thread John McCall via cfe-commits
rjmccall added a comment. I'm sorry for the delay. It looks fine, thanks. http://reviews.llvm.org/D12614 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D15579: Fix the failing windows clang unit tests. NFC

2015-12-16 Thread Sumanth Gundapaneni via cfe-commits
sgundapa created this revision. sgundapa added reviewers: martell, yaron.keren, dougk, chapuni. sgundapa added a subscriber: cfe-commits. All the tests are missing the {{(.exe)?}} suffix on the exectables which the FileCheck is grepping for. http://reviews.llvm.org/D15579 Files: test/Driver/m

Re: [PATCH] D15579: Fix the failing windows clang unit tests. NFC

2015-12-16 Thread Sumanth Gundapaneni via cfe-commits
sgundapa added a comment. I can not seem to possibly find a windows community bot. It would be nice if some one can point me to that. http://reviews.llvm.org/D15579 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-b

Re: [PATCH] D15539: [libcxxabi] Reducing stack usage of test

2015-12-16 Thread Howard Hinnant via cfe-commits
That would destroy the timing information, but no one seems to be paying attention to that anyway. I recommend just picking smaller buffer sizes. But 11 is too small. Small enough to pass on your embedded platform, but not in the range of a typical alignof(max_align_t). Howard On Dec 16, 20

Re: [PATCH] D10599: [OPENMP 4.0] Initial support for '#pragma omp declare simd' directive.

2015-12-16 Thread John McCall via cfe-commits
rjmccall added a comment. Richard should really be the one to sign off on this, since he's been managing the review so far. I am curious why we decided to handle this separately instead of treating it as a different attribute spelling kind. http://reviews.llvm.org/D10599 __

Re: [PATCH] D15534: [CUDA] renamed cuda_runtime.h wrapper to __clang_cuda_runtime_wrapper.h

2015-12-16 Thread Artem Belevich via cfe-commits
tra updated this revision to Diff 43025. tra marked 3 inline comments as done. tra added a comment. Incorporated Chandler's suggestions. Fixed #include_next -> #include. http://reviews.llvm.org/D15534 Files: lib/Driver/ToolChains.cpp lib/Headers/CMakeLists.txt lib/Headers/__clang_cuda_run

Re: [PATCH] D15534: [CUDA] renamed cuda_runtime.h wrapper to __clang_cuda_runtime_wrapper.h

2015-12-16 Thread Artem Belevich via cfe-commits
tra added inline comments. Comment at: lib/Headers/__clang_cuda_runtime_wrapper.h:95 @@ -78,3 +94,3 @@ #define __CUDACC__ #include_next "cuda_runtime.h" jhen wrote: > Now that the name of this header has been changed, would it be appropriate to > change this

Re: [PATCH] D15524: [GCC] Attribute ifunc support in clang

2015-12-16 Thread John McCall via cfe-commits
rjmccall added inline comments. Comment at: include/clang/Basic/AttrDocs.td:1866 @@ +1865,3 @@ + let Content = [{ +The attribute ``__attribute__((ifunc("resolver")))`` is used to mark a function as an indirect function using the STT_GNU_IFUNC symbol type extension to the ELF st

Re: [PATCH] D15579: Fix the failing windows clang unit tests. NFC

2015-12-16 Thread Yaron Keren via cfe-commits
yaron.keren added a comment. http://bb.pgr.jp/grid http://reviews.llvm.org/D15579 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r255798 - [CMake] ExternalProject for compiler-rt needs to depend on llvm-config and clang

2015-12-16 Thread Chris Bieneman via cfe-commits
Author: cbieneman Date: Wed Dec 16 12:42:14 2015 New Revision: 255798 URL: http://llvm.org/viewvc/llvm-project?rev=255798&view=rev Log: [CMake] ExternalProject for compiler-rt needs to depend on llvm-config and clang The add_dependencies call on compiler-rt-configure adds llvm-config and clang t

r255799 - [CMake] Make CLANG_BOOTSTRAP_TARGETS overridable

2015-12-16 Thread Chris Bieneman via cfe-commits
Author: cbieneman Date: Wed Dec 16 12:45:53 2015 New Revision: 255799 URL: http://llvm.org/viewvc/llvm-project?rev=255799&view=rev Log: [CMake] Make CLANG_BOOTSTRAP_TARGETS overridable This allows exposing a custom list of targets from the next stage build up. Modified: cfe/trunk/CMakeLists.

Re: [PATCH] D15577: [PS4] Fix the unit test to be compatible with clang driver. NFC

2015-12-16 Thread Paul Robinson via cfe-commits
probinson added a subscriber: probinson. probinson added a comment. So, mainly you want calls to GetProgramPath not to provide the .exe extension, even on Windows? Comment at: test/Driver/sanitizer-ld.c:382 @@ -381,3 +381,3 @@ // RUN: | FileCheck --check-prefix=CHECK-UBSAN-P

r255801 - [CMake] If you're building compiler-rt, the bootstrap build should depend on it.

2015-12-16 Thread Chris Bieneman via cfe-commits
Author: cbieneman Date: Wed Dec 16 12:49:12 2015 New Revision: 255801 URL: http://llvm.org/viewvc/llvm-project?rev=255801&view=rev Log: [CMake] If you're building compiler-rt, the bootstrap build should depend on it. Adding optional dependency for the bootstrap targets on compiler-rt. Modified:

r255802 - [CUDA] renamed cuda_runtime.h wrapper to __cuda_runtime.h

2015-12-16 Thread Artem Belevich via cfe-commits
Author: tra Date: Wed Dec 16 12:51:59 2015 New Revision: 255802 URL: http://llvm.org/viewvc/llvm-project?rev=255802&view=rev Log: [CUDA] renamed cuda_runtime.h wrapper to __cuda_runtime.h Currently it's easy to break CUDA compilation by passing "-isystem /path/to/cuda/include" to compiler which l

Re: [PATCH] D15534: [CUDA] renamed cuda_runtime.h wrapper to __clang_cuda_runtime_wrapper.h

2015-12-16 Thread Artem Belevich via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL255802: [CUDA] renamed cuda_runtime.h wrapper to __cuda_runtime.h (authored by tra). Changed prior to commit: http://reviews.llvm.org/D15534?vs=43025&id=43034#toc Repository: rL LLVM http://reviews.

Re: [PATCH] D15577: [PS4] Fix the unit test to be compatible with clang driver. NFC

2015-12-16 Thread Sumanth Gundapaneni via cfe-commits
sgundapa added a comment. GetProgramPath will take care of the extension. I will change the line to "{{.*}}ld{{(.gold)?(.exe)?}}" as per your suggestion and push a new patch http://reviews.llvm.org/D15577 ___ cfe-commits mailing list cfe-commits@li

Re: [PATCH] D15577: [PS4] Fix the unit test to be compatible with clang driver. NFC

2015-12-16 Thread Sumanth Gundapaneni via cfe-commits
sgundapa updated this revision to Diff 43038. http://reviews.llvm.org/D15577 Files: lib/Driver/Tools.cpp test/Driver/sanitizer-ld.c Index: test/Driver/sanitizer-ld.c === --- test/Driver/sanitizer-ld.c +++ test/Driver/sanitizer-l

r255804 - Fix the failing windows clang unit tests. NFC

2015-12-16 Thread Sumanth Gundapaneni via cfe-commits
Author: sgundapa Date: Wed Dec 16 13:09:51 2015 New Revision: 255804 URL: http://llvm.org/viewvc/llvm-project?rev=255804&view=rev Log: Fix the failing windows clang unit tests. NFC Some tests are missing the {{(.exe)?}} suffix on the exectables which the FileCheck is grepping for. This will ensur

Re: [PATCH] D15579: Fix the failing windows clang unit tests. NFC

2015-12-16 Thread Sumanth Gundapaneni via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL255804: Fix the failing windows clang unit tests. NFC (authored by sgundapa). Changed prior to commit: http://reviews.llvm.org/D15579?vs=43022&id=43040#toc Repository: rL LLVM http://reviews.llvm.or

Re: [PATCH] D15539: [libcxxabi] Reducing stack usage of test

2015-12-16 Thread Ben Craig via cfe-commits
bcraig updated this revision to Diff 43041. bcraig added a comment. Starting primes at 13, and skipping the larger twin in twin prime pairs. Using long double instead of char for padding to help alleviate fears of multiple structs having the same actual size. http://reviews.llvm.org/D15539 F

Re: [PATCH] D15539: [libcxxabi] Reducing stack usage of test

2015-12-16 Thread Jonathan Roelofs via cfe-commits
jroelofs added a comment. What does having them be `long double`s give over just multiplying the counts by 16 (or however big it is on your platform)? Alignment? Seems like it'd be better to start with a prime that's ~16x larger, say 211, than to have that factor of 16 floating around everywher

Re: [PATCH] D15577: [PS4] Fix the unit test to be compatible with clang driver. NFC

2015-12-16 Thread Paul Robinson via cfe-commits
probinson accepted this revision. probinson added a reviewer: probinson. probinson added a comment. This revision is now accepted and ready to land. LGTM http://reviews.llvm.org/D15577 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://l

r255809 - Add -fsyntax-only to fix failure in read-only directories.

2015-12-16 Thread Diego Novillo via cfe-commits
Author: dnovillo Date: Wed Dec 16 13:52:05 2015 New Revision: 255809 URL: http://llvm.org/viewvc/llvm-project?rev=255809&view=rev Log: Add -fsyntax-only to fix failure in read-only directories. Internally, this test is executed in a read-only directory, which causes it to fail because the driver

Re: [PATCH] D15539: [libcxxabi] Reducing stack usage of test

2015-12-16 Thread Ben Craig via cfe-commits
bcraig added a comment. In http://reviews.llvm.org/D15539#312319, @jroelofs wrote: > What does having them be `long double`s give over just multiplying the counts > by 16 (or however big it is on your platform)? Alignment? > > Seems like it'd be better to start with a prime that's ~16x larger, s

Re: [PATCH] D15539: [libcxxabi] Reducing stack usage of test

2015-12-16 Thread Jonathan Roelofs via cfe-commits
jroelofs added a comment. In http://reviews.llvm.org/D15539#312332, @bcraig wrote: > In http://reviews.llvm.org/D15539#312319, @jroelofs wrote: > > > What does having them be `long double`s give over just multiplying the > > counts by 16 (or however big it is on your platform)? Alignment? > > >

r255812 - Let -Wdelete-non-virtual-dtor mention final.

2015-12-16 Thread Nico Weber via cfe-commits
Author: nico Date: Wed Dec 16 14:07:24 2015 New Revision: 255812 URL: http://llvm.org/viewvc/llvm-project?rev=255812&view=rev Log: Let -Wdelete-non-virtual-dtor mention final. Modified: cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.td cfe/trunk/test/SemaCXX/destructor.cpp Modified: c

Re: [patch] Let -Wdelete-non-virtual-dtor mention that making the class final is a good fix too

2015-12-16 Thread Nico Weber via cfe-commits
r255812, thanks! On Mon, Dec 14, 2015 at 6:45 PM, David Blaikie wrote: > LGTM, please commit (if you like, if you want to wait for other feedback > that's OK too) > > On Tue, Dec 15, 2015 at 2:15 AM, Nico Weber wrote: > >> On Sun, Dec 13, 2015 at 6:57 PM, David Blaikie >> wrote: >> >>> (attach

Re: [PATCH] D15539: [libcxxabi] Reducing stack usage of test

2015-12-16 Thread Jonathan Roelofs via cfe-commits
jroelofs added a comment. In http://reviews.llvm.org/D15539#312336, @jroelofs wrote: > In http://reviews.llvm.org/D15539#312332, @bcraig wrote: > > > In http://reviews.llvm.org/D15539#312319, @jroelofs wrote: > > > > > What does having them be `long double`s give over just multiplying the > > >

Re: [PATCH] D15539: [libcxxabi] Reducing stack usage of test

2015-12-16 Thread Ben Craig via cfe-commits
bcraig added a comment. In http://reviews.llvm.org/D15539#312336, @jroelofs wrote: > In http://reviews.llvm.org/D15539#312332, @bcraig wrote: > > > In http://reviews.llvm.org/D15539#312319, @jroelofs wrote: > > > > > What does having them be `long double`s give over just multiplying the > > > co

r255813 - [CMake] Name the bootstrap stages stage[0-9]

2015-12-16 Thread Chris Bieneman via cfe-commits
Author: cbieneman Date: Wed Dec 16 14:17:07 2015 New Revision: 255813 URL: http://llvm.org/viewvc/llvm-project?rev=255813&view=rev Log: [CMake] Name the bootstrap stages stage[0-9] When you start chaining bootstrap stages the CMake-generated targets get unwieldy. This change supports naming the

r255814 - [PS4] Fix the unit test to be compatible with clang driver. NFC

2015-12-16 Thread Sumanth Gundapaneni via cfe-commits
Author: sgundapa Date: Wed Dec 16 14:18:12 2015 New Revision: 255814 URL: http://llvm.org/viewvc/llvm-project?rev=255814&view=rev Log: [PS4] Fix the unit test to be compatible with clang driver. NFC ".exe" extension is inherently checked by llvm::fs::can_execute() This patch fixes the linker exte

Re: [PATCH] D15577: [PS4] Fix the unit test to be compatible with clang driver. NFC

2015-12-16 Thread Sumanth Gundapaneni via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL255814: [PS4] Fix the unit test to be compatible with clang driver. NFC (authored by sgundapa). Changed prior to commit: http://reviews.llvm.org/D15577?vs=43038&id=43049#toc Repository: rL LLVM http

[PATCH] D15584: [CMake] Support a simple case for bootstrap builds to generate PGO data

2015-12-16 Thread Chris Bieneman via cfe-commits
beanz created this revision. beanz added reviewers: bogner, silvas, chandlerc. beanz added a subscriber: cfe-commits. This patch adds support for the clang multi-stage bootstrapping to support PGO profdata generation, and can build a 2 or 3 stage compiler. With this patch applied you can configu

Re: [PATCH] D15006: Driver: Better detection of mingw-gcc

2015-12-16 Thread Yaron Keren via cfe-commits
yaron.keren accepted this revision. yaron.keren added a comment. This revision is now accepted and ready to land. LGTM with some tests cases. http://reviews.llvm.org/D15006 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.or

[PATCH] D15586: [WebAssembly] Initial linking support.

2015-12-16 Thread Dan Gohman via cfe-commits
sunfish created this revision. sunfish added subscribers: cfe-commits, jfb, dschuff. sunfish set the repository for this revision to rL LLVM. This begins minimal support for invoking 'ld' from clang for WebAssembly targets. Repository: rL LLVM http://reviews.llvm.org/D15586 Files: lib/Driv

r255827 - Fix CFI tests in sanitizer-ld.

2015-12-16 Thread Evgeniy Stepanov via cfe-commits
Author: eugenis Date: Wed Dec 16 16:03:39 2015 New Revision: 255827 URL: http://llvm.org/viewvc/llvm-project?rev=255827&view=rev Log: Fix CFI tests in sanitizer-ld. This test is not testing what it is supposed to test because of a mixup with the CHECK lines. Modified: cfe/trunk/test/Driver/s

Re: [PATCH] D15585: [PPC] Add long long/double support for vec_cts, vec_ctu and vec_ctf in altivec.h

2015-12-16 Thread Tim Shen via cfe-commits
timshen abandoned this revision. timshen added a comment. Due to unknown reasons, Phabricator is not sending emails to the list. Kill this revision and create another one to see. http://reviews.llvm.org/D15585 ___ cfe-commits mailing list cfe-commi

[libcxx] [PATCH] unordered_map: Avoid unnecessary mallocs when no insert occurs

2015-12-16 Thread Duncan P. N. Exon Smith via cfe-commits
This is a follow-up to r239666: "Fix PR12999 - unordered_set::insert calls operator new when no insert occurs". That fix didn't apply to `unordered_map` because unordered_map::value_type gets packed inside: -- union __value_type { pair __nc; // Only C++11 or higher. pair __cc; // Always.

[PATCH] D15590: [Bugfix] Disqualify unaddressable overload candidates when a function is directly called indirectly.

2015-12-16 Thread George Burgess IV via cfe-commits
george.burgess.iv created this revision. george.burgess.iv added a reviewer: rsmith. george.burgess.iv added a subscriber: cfe-commits. "directly called indirectly" wasn't a typo; I can't think of a better name for this pattern. :) Currently, Sema treats `foo()` and `(&foo)()` both as direct cal

[PATCH] D15589: [PPC] Add long long/double support for vec_cts, vec_ctu and vec_ctf.

2015-12-16 Thread Tim Shen via cfe-commits
timshen created this revision. timshen added a subscriber: cfe-commits. Add long long/double support for vec_cts, vec_ctu and vec_ctf. Similar to this change in GCC: https://gcc.gnu.org/ml/gcc-patches/2014-08/msg02653.html http://reviews.llvm.org/D15589 Files: lib/Headers/altivec.h test/Co

[PATCH] D15591: [Bugfix] Make type deduction work more nicely with unaddressable functions

2015-12-16 Thread George Burgess IV via cfe-commits
george.burgess.iv created this revision. george.burgess.iv added a reviewer: rsmith. george.burgess.iv added a subscriber: cfe-commits. Given: ``` void foo(int i) __attribute__((enable_if(i < 0, ""))); void foo(double d); auto fooToCall = foo; ``` ...We currently can't determine the type of `fo

r255838 - Update for llvm API change.

2015-12-16 Thread Rafael Espindola via cfe-commits
Author: rafael Date: Wed Dec 16 16:59:09 2015 New Revision: 255838 URL: http://llvm.org/viewvc/llvm-project?rev=255838&view=rev Log: Update for llvm API change. Modified: cfe/trunk/lib/Frontend/CompilerInstance.cpp cfe/trunk/lib/Frontend/HeaderIncludeGen.cpp Modified: cfe/trunk/lib/Front

Re: [PATCH] D13357: [Concepts] Diagnose when 'concept' is specified on a specialization

2015-12-16 Thread Nathan Wilson via cfe-commits
nwilson updated this revision to Diff 43072. nwilson added a comment. - Move IsConcept bit and associated member functions from FunctionDecl to FunctionTemplateDecl. - Set the IsConcept flag using getDescribedFunctionTemplate. http://reviews.llvm.org/D13357 Files: include/clang/AST/DeclTempl

r255840 - Fix funciton->function typo.

2015-12-16 Thread Eric Christopher via cfe-commits
Author: echristo Date: Wed Dec 16 17:10:46 2015 New Revision: 255840 URL: http://llvm.org/viewvc/llvm-project?rev=255840&view=rev Log: Fix funciton->function typo. Modified: cfe/trunk/lib/ARCMigrate/ObjCMT.cpp cfe/trunk/lib/AST/MicrosoftMangle.cpp cfe/trunk/lib/CodeGen/CGDebugInfo.h

r255843 - Update for llvm api change.

2015-12-16 Thread Rafael Espindola via cfe-commits
Author: rafael Date: Wed Dec 16 17:16:37 2015 New Revision: 255843 URL: http://llvm.org/viewvc/llvm-project?rev=255843&view=rev Log: Update for llvm api change. Modified: cfe/trunk/lib/CodeGen/CodeGenAction.cpp Modified: cfe/trunk/lib/CodeGen/CodeGenAction.cpp URL: http://llvm.org/viewvc/ll

r255848 - [WebAssembly] Initial linking support.

2015-12-16 Thread Dan Gohman via cfe-commits
Author: djg Date: Wed Dec 16 17:30:41 2015 New Revision: 255848 URL: http://llvm.org/viewvc/llvm-project?rev=255848&view=rev Log: [WebAssembly] Initial linking support. This begins minimal support for invoking 'ld' from clang for WebAssembly targets. Differential Revision: http://reviews.llvm.or

Re: [PATCH] D15586: [WebAssembly] Initial linking support.

2015-12-16 Thread Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL255848: [WebAssembly] Initial linking support. (authored by djg). Changed prior to commit: http://reviews.llvm.org/D15586?vs=43059&id=43075#toc Repository: rL LLVM http://reviews.llvm.org/D15586 Fi

Re: [PATCH] D15305: [CUDA] Do not allow dynamic initialization of global device side variables.

2015-12-16 Thread Artem Belevich via cfe-commits
tra added a comment. Ping. @rsmith -- Richard, can you take a look? http://reviews.llvm.org/D15305 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D15309: [CUDA] emit vtables only for classes with methods usable on this side of compilation.

2015-12-16 Thread Artem Belevich via cfe-commits
tra added a comment. Ping. @rsmith -- Richard, can you take a look at this change as well? http://reviews.llvm.org/D15309 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[libcxx] r255851 - Add CMake option LIBCXX_CONFIGURE_IDE to allow for other IDE's.

2015-12-16 Thread Eric Fiselier via cfe-commits
Author: ericwf Date: Wed Dec 16 17:41:05 2015 New Revision: 255851 URL: http://llvm.org/viewvc/llvm-project?rev=255851&view=rev Log: Add CMake option LIBCXX_CONFIGURE_IDE to allow for other IDE's. CLion needs similar configuration changes as MSVC_IDE and XCODE. Modified: libcxx/trunk/CMakeLi

Re: [PATCH] D15455: [Driver] Let -static override the toolchain default PIC setting.

2015-12-16 Thread Bob Wilson via cfe-commits
bob.wilson added a subscriber: bob.wilson. bob.wilson added a comment. There has been a long-standing convention in both gcc and clang for Darwin that -static changes the default for PIC. Changing that convention is breaking stuff for us. The commit message for r245667 says "This new behavior al

Re: [PATCH] D13750: [libcxx] Fix LWG Issue #2367 - Fixing std::tuple and std::pair's default constructors.

2015-12-16 Thread Eric Fiselier via cfe-commits
EricWF updated this revision to Diff 43082. EricWF added a comment. Fix merge conflicts. http://reviews.llvm.org/D13750 Files: include/__config include/utility test/std/utilities/utility/pairs/pairs.pair/default.pass.cpp Index: test/std/utilities/utility/pairs/pairs.pair/default.pass.cpp

Re: [PATCH] D13796: [libcxx] Use "-stdlib=libc++" when possible instead of '-nodefaultlibs' in the test suite.

2015-12-16 Thread Eric Fiselier via cfe-commits
EricWF updated this revision to Diff 43084. EricWF added a comment. Update diff against ToT. http://reviews.llvm.org/D13796 Files: test/libcxx/compiler.py test/libcxx/test/config.py Index: test/libcxx/test/config.py === --- te

Re: [PATCH] D12297: [libcxx] Fix most undefined behavior in

2015-12-16 Thread Eric Fiselier via cfe-commits
EricWF abandoned this revision. EricWF added a comment. Abandoning for better solution in http://reviews.llvm.org/D12297. http://reviews.llvm.org/D12297 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/li

Re: [PATCH] D15596: Add -L/path/to/cuda/lib if any of our inputs are CUDA files.

2015-12-16 Thread Artem Belevich via cfe-commits
tra added inline comments. Comment at: lib/Driver/ToolChains.cpp:4125 @@ +4124,3 @@ + ArgStringList &LDArgs) const { + if (DriverArgs.hasArg(options::OPT_nocudalib) || !CudaInstallation.isValid()) +return; We may need a new option

Re: [PATCH] D15596: Add -L/path/to/cuda/lib if any of our inputs are CUDA files.

2015-12-16 Thread Artem Belevich via cfe-commits
tra added a comment. You also need a test case for the changes. http://reviews.llvm.org/D15596 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D15596: Add -L/path/to/cuda/lib if any of our inputs are CUDA files.

2015-12-16 Thread Artem Belevich via cfe-commits
tra added inline comments. Comment at: lib/Driver/Tools.cpp:259-261 @@ -249,1 +258,5 @@ + // Add -L/path/to/cuda/lib if any of our inputs are .cu files. + if (std::any_of(C.getActions().begin(), C.getActions().end(), isCudaAction)) { +TC.AddCudaLinkerArgs(Args, CmdArgs); +

Re: [PATCH] D13796: [libcxx] Use "-stdlib=libc++" when possible instead of '-nodefaultlibs' in the test suite.

2015-12-16 Thread Jonathan Roelofs via cfe-commits
jroelofs added inline comments. Comment at: test/libcxx/test/config.py:364 @@ +363,3 @@ +deduced_use_stdlib_eq_libcxx = (self.cxx.isClang() and +cxx_abi == 'none' and +not use_llvm_unwinder) --

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

2015-12-16 Thread Devin Coughlin via cfe-commits
dcoughlin added a comment. Thanks for adding `bear_get_environment()` to handle the environment weirdness on Darwin. Comment at: tools/scan-build-py/libscanbuild/intercept.py:146 @@ +145,3 @@ +}) +elif sys.platform == 'darwin': +logging.debug('intercept gonn

r255859 - [analyzer] Better detect when C++ object was constructed into existing region.

2015-12-16 Thread Devin Coughlin via cfe-commits
Author: dcoughlin Date: Wed Dec 16 18:28:33 2015 New Revision: 255859 URL: http://llvm.org/viewvc/llvm-project?rev=255859&view=rev Log: [analyzer] Better detect when C++ object was constructed into existing region. When the analyzer evaluates a CXXConstructExpr, it looks ahead in the CFG for the

r255860 - [TLS on Darwin] use CXX_FAST_TLS calling convention for access functions.

2015-12-16 Thread Manman Ren via cfe-commits
Author: mren Date: Wed Dec 16 18:42:36 2015 New Revision: 255860 URL: http://llvm.org/viewvc/llvm-project?rev=255860&view=rev Log: [TLS on Darwin] use CXX_FAST_TLS calling convention for access functions. Also set nounwind attribute. rdar://problem/9001553 Modified: cfe/trunk/lib/CodeGen/It

Re: r255840 - Fix funciton->function typo.

2015-12-16 Thread Rafael Espíndola via cfe-commits
Thanks :-) On Dec 16, 2015 6:13 PM, "Eric Christopher via cfe-commits" < cfe-commits@lists.llvm.org> wrote: > Author: echristo > Date: Wed Dec 16 17:10:46 2015 > New Revision: 255840 > > URL: http://llvm.org/viewvc/llvm-project?rev=255840&view=rev > Log: > Fix funciton->function typo. > > Modified

Re: [PATCH] D15363: [UBSan] Implement runtime suppressions (PR25066).

2015-12-16 Thread Alexey Samsonov via cfe-commits
samsonov added a comment. Kostya, PTAL. IIRC it was your feature request :) http://reviews.llvm.org/D15363 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: r252960 - [modules] Simplify and generalize the existing rule for finding hidden

2015-12-16 Thread Bob Wilson via cfe-commits
> On Nov 12, 2015, at 2:19 PM, Richard Smith via cfe-commits > wrote: > > Author: rsmith > Date: Thu Nov 12 16:19:45 2015 > New Revision: 252960 > > URL: http://llvm.org/viewvc/llvm-project?rev=252960&view=rev > Log: > [modules] Simplify and generalize the existing rule for finding hidden > de

  1   2   >