r270353 - [AST] Cleanup comments regarding CXXRecordDecl::isEmpty

2016-05-21 Thread David Majnemer via cfe-commits
Author: majnemer Date: Sun May 22 00:34:26 2016 New Revision: 270353 URL: http://llvm.org/viewvc/llvm-project?rev=270353=rev Log: [AST] Cleanup comments regarding CXXRecordDecl::isEmpty We were missing references to the standard, some of our home-grown verbiage didn't make any sense. No

r270352 - Driver: sink getLinuxDynamicLoader into the Toolchain

2016-05-21 Thread Saleem Abdulrasool via cfe-commits
Author: compnerd Date: Sat May 21 22:12:19 2016 New Revision: 270352 URL: http://llvm.org/viewvc/llvm-project?rev=270352=rev Log: Driver: sink getLinuxDynamicLoader into the Toolchain The parameter already requires the toolchain, sink the method into the class. This also enables the use of the

r270351 - Driver: simplify getDynameLinker

2016-05-21 Thread Saleem Abdulrasool via cfe-commits
Author: compnerd Date: Sat May 21 20:37:36 2016 New Revision: 270351 URL: http://llvm.org/viewvc/llvm-project?rev=270351=rev Log: Driver: simplify getDynameLinker Convert the cascading if/else to a switch. This makes it easier to follow the logic. Minor difference is that we no longer default

Re: [PATCH] D18035: [GCC] PR23529 Mangler part of attrbute abi_tag support

2016-05-21 Thread Luchesar V. ILIEV via cfe-commits
kerberizer added a comment. Dmitry, for a few days now I'm seeing another regression test failure. It has most likely appeared in the 6 hours frame after r269852, because that's the period of my automatic builds, and r269852 was the last to pass the tests successfully. The failure this time is

Re: [PATCH] D20468: [X86][AVX] Ensure zero-extension of _mm256_extract_epi8 and _mm256_extract_epi16

2016-05-21 Thread Simon Pilgrim via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL270330: [X86][AVX] Ensure zero-extension of _mm256_extract_epi8 and _mm256_extract_epi16 (authored by RKSimon). Changed prior to commit: http://reviews.llvm.org/D20468?vs=57927=58045#toc Repository:

r270330 - [X86][AVX] Ensure zero-extension of _mm256_extract_epi8 and _mm256_extract_epi16

2016-05-21 Thread Simon Pilgrim via cfe-commits
Author: rksimon Date: Sat May 21 16:14:35 2016 New Revision: 270330 URL: http://llvm.org/viewvc/llvm-project?rev=270330=rev Log: [X86][AVX] Ensure zero-extension of _mm256_extract_epi8 and _mm256_extract_epi16 Ensure _mm256_extract_epi8 and _mm256_extract_epi16 zero extend their i8/i16 result

[PATCH] D20512: [PATCH] Bug 27475 - Request header guard check processes .hpp files as well as .h files

2016-05-21 Thread Mads Ravn via cfe-commits
madsravn created this revision. madsravn added reviewers: alexfh, flx. madsravn added a subscriber: cfe-commits. Patch for bug 27475: https://llvm.org/bugs/show_bug.cgi?id=27475 I used the suggestion of alexfh to check the header extension. http://reviews.llvm.org/D20512 Files:

Re: [PATCH] D19783: Fix cv-qualification of '*this' captures (and nasty bug PR27507 introduced by commit 263921 "Implement Lambda Capture of *this by Value as [=, *this]")

2016-05-21 Thread Taewook Oh via cfe-commits
twoh added a comment. Hmm, yes it seems not so simple to move the call to another place. If I come up with a better idea I'll submit it as a separate patch, as I don't want to block this one. Thank you for your work, @faisalv! http://reviews.llvm.org/D19783

[PATCH] D20511: Fix crash while parsing variable template with variadic template arguments

2016-05-21 Thread Olivier Goffart via cfe-commits
ogoffart created this revision. ogoffart added reviewers: cfe-commits, rsmith. It is only a crash if the compiler optimize for this!=nullptr because LocalInstantiationScope::getPartiallySubstitutedPack checks if 'this' is null. (This is crashing when clang is compiled with GCC6)

[PATCH] D20510: [PATCH] Fix for bug 27802 misc-macro-parentheses breaks variadic macros

2016-05-21 Thread Mads Ravn via cfe-commits
madsravn created this revision. madsravn added reviewers: alexfh, flx. madsravn added a subscriber: cfe-commits. This is fix for bug 27802: https://llvm.org/bugs/show_bug.cgi?id=27802 I have added a check to clang-tidy which refrains from putting parantheses around macros which are variadic.

Re: [PATCH] D20365: [PATCH] clang-tidy: Bug 27731 - modernize-pass-by-value suggest using std::move for types that perform copies on move

2016-05-21 Thread Mads Ravn via cfe-commits
madsravn added a comment. @Prazek thanks. I will look into it :) http://reviews.llvm.org/D20365 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D20365: [PATCH] clang-tidy: Bug 27731 - modernize-pass-by-value suggest using std::move for types that perform copies on move

2016-05-21 Thread Piotr Padlewski via cfe-commits
Prazek added a subscriber: Prazek. Prazek added a comment. You have to push it by yourself. It's ain't fun if someone do it for you :D You have to obtain commit access http://llvm.org/docs/DeveloperPolicy.html#obtaining-commit-access and then push it (you can also find in docs how to do it)

Re: [PATCH] D20365: [PATCH] clang-tidy: Bug 27731 - modernize-pass-by-value suggest using std::move for types that perform copies on move

2016-05-21 Thread Mads Ravn via cfe-commits
madsravn added a comment. Just curious, as I'm sort of new to this. How long will it take before its merged in? http://reviews.llvm.org/D20365 ___ cfe-commits mailing list cfe-commits@lists.llvm.org