[clang-tools-extra] r272132 - [include-fixer] do not add enum forward declaration into symbol index table.

2016-06-08 Thread Eric Liu via cfe-commits
Author: ioeric Date: Wed Jun 8 07:13:14 2016 New Revision: 272132 URL: http://llvm.org/viewvc/llvm-project?rev=272132=rev Log: [include-fixer] do not add enum forward declaration into symbol index table. Modified: clang-tools-extra/trunk/include-fixer/find-all-symbols/FindAllSymbols.cpp

r272119 - [AVX512] Reformat macro intrinsics, ensure arguments have proper typecasts, ensure result is typecasted back to the generic types.

2016-06-08 Thread Craig Topper via cfe-commits
Author: ctopper Date: Wed Jun 8 01:08:07 2016 New Revision: 272119 URL: http://llvm.org/viewvc/llvm-project?rev=272119=rev Log: [AVX512] Reformat macro intrinsics, ensure arguments have proper typecasts, ensure result is typecasted back to the generic types. Modified:

r272118 - [X86] Put parentheses around macro arguments in intrinsics.

2016-06-08 Thread Craig Topper via cfe-commits
Author: ctopper Date: Wed Jun 8 01:08:04 2016 New Revision: 272118 URL: http://llvm.org/viewvc/llvm-project?rev=272118=rev Log: [X86] Put parentheses around macro arguments in intrinsics. Modified: cfe/trunk/lib/Headers/avx512fintrin.h Modified: cfe/trunk/lib/Headers/avx512fintrin.h URL:

[PATCH] D21120: clang-rename: implement renaming of classes inside dynamic_cast

2016-06-08 Thread Miklos Vajna via cfe-commits
vmiklos created this revision. vmiklos added a reviewer: klimek. vmiklos added a subscriber: cfe-commits. Refactor to do the same as what is done already for static_cast. http://reviews.llvm.org/D21120 Files: clang-rename/USRLocFinder.cpp test/clang-rename/DynamicCastExpr.cpp Index:

[PATCH] D21122: CodeGen: Start emitting checked loads when both trapping CFI and -fwhole-program-vtables are enabled.

2016-06-08 Thread Peter Collingbourne via cfe-commits
pcc created this revision. pcc added reviewers: eugenis, mehdi_amini, kcc. pcc added a subscriber: cfe-commits. pcc added a dependency: D21121: IR: Introduce llvm.type.checked.load intrinsic.. Depends on D21121 http://reviews.llvm.org/D21122 Files: lib/CodeGen/CGClass.cpp

r272244 - [X86] Add void to the argument list of intrinsics that don't take arguments since empty argument list mean something else in C.

2016-06-08 Thread Craig Topper via cfe-commits
Author: ctopper Date: Thu Jun 9 00:14:28 2016 New Revision: 272244 URL: http://llvm.org/viewvc/llvm-project?rev=272244=rev Log: [X86] Add void to the argument list of intrinsics that don't take arguments since empty argument list mean something else in C. Modified:

r272246 - [X86] Handle AVX2 pslldqi and psrldqi intrinsics shufflevector creation directly in the header file instead of in CGBuiltin.cpp. Simplify the sse2 equivalents as well.

2016-06-08 Thread Craig Topper via cfe-commits
Author: ctopper Date: Thu Jun 9 00:15:12 2016 New Revision: 272246 URL: http://llvm.org/viewvc/llvm-project?rev=272246=rev Log: [X86] Handle AVX2 pslldqi and psrldqi intrinsics shufflevector creation directly in the header file instead of in CGBuiltin.cpp. Simplify the sse2 equivalents as

r272245 - [X86] Reuse the EmitX86Select routine to handle the select for masked palignr too.

2016-06-08 Thread Craig Topper via cfe-commits
Author: ctopper Date: Thu Jun 9 00:15:00 2016 New Revision: 272245 URL: http://llvm.org/viewvc/llvm-project?rev=272245=rev Log: [X86] Reuse the EmitX86Select routine to handle the select for masked palignr too. Modified: cfe/trunk/lib/CodeGen/CGBuiltin.cpp Modified:

r272247 - [Sema] Don't crash when a field w/ a mem-initializer clashes with a record name

2016-06-08 Thread David Majnemer via cfe-commits
Author: majnemer Date: Thu Jun 9 00:26:56 2016 New Revision: 272247 URL: http://llvm.org/viewvc/llvm-project?rev=272247=rev Log: [Sema] Don't crash when a field w/ a mem-initializer clashes with a record name It is possible for a field and a class to have the same name. In such cases,

Re: [PATCH] D20428: Tracking exception specification source locations

2016-06-08 Thread Aaron Ballman via cfe-commits
aaron.ballman added a comment. One thing this patch does not do but needs to is fix `ASTContext::adjustExceptionSpec()` (Thanks to Don Hinton for pointing this out off-list!), however, I am at a bit of a loss for how best to rebuild the type location. Would it be correct to call

[PATCH] D21134: clang-tidy: new check readability-misplaced-array-index

2016-06-08 Thread Daniel Marjamäki via cfe-commits
danielmarjamaki created this revision. danielmarjamaki added a reviewer: alexfh. danielmarjamaki added a subscriber: cfe-commits. danielmarjamaki set the repository for this revision to rL LLVM. this is a new check for clang-tidy that diagnoses when it see unusual array index syntax. there is

r272142 - clang-format: [JS] fix an assertion failure caused by shrinking sources.

2016-06-08 Thread Martin Probst via cfe-commits
Author: mprobst Date: Wed Jun 8 09:04:04 2016 New Revision: 272142 URL: http://llvm.org/viewvc/llvm-project?rev=272142=rev Log: clang-format: [JS] fix an assertion failure caused by shrinking sources. Summary: The JavaScript import sorter has a corner condition that can cause the overall source

r272143 - clang-format: [JS] fix a FIXME.

2016-06-08 Thread Martin Probst via cfe-commits
Author: mprobst Date: Wed Jun 8 09:05:26 2016 New Revision: 272143 URL: http://llvm.org/viewvc/llvm-project?rev=272143=rev Log: clang-format: [JS] fix a FIXME. Modified: cfe/trunk/lib/Format/SortJavaScriptImports.cpp Modified: cfe/trunk/lib/Format/SortJavaScriptImports.cpp URL:

[PATCH] D21132: [include-fixer] Keep dot dot in SymbolInfo file paths.

2016-06-08 Thread Haojian Wu via cfe-commits
hokein created this revision. hokein added reviewers: bkramer, ioeric. hokein added a subscriber: cfe-commits. Currently, removing dot dot in header's path doesn't make include-fixer minimize path correctly in some cases, for example, specify a relative search path based on the build

r272141 - [AVX512] Emit select instruction instead of using x86 specific instrinsics.

2016-06-08 Thread Igor Breger via cfe-commits
Author: ibreger Date: Wed Jun 8 08:59:20 2016 New Revision: 272141 URL: http://llvm.org/viewvc/llvm-project?rev=272141=rev Log: [AVX512] Emit select instruction instead of using x86 specific instrinsics. This will allow us to remove the x86 instrinics from the backend. Differential Revision:

Re: [PATCH] D21120: clang-rename: implement renaming of classes inside dynamic_cast

2016-06-08 Thread Miklos Vajna via cfe-commits
vmiklos added a comment. Hmm, http://lab.llvm.org:8011/builders/llvm-clang-lld-x86_64-scei-ps4-ubuntu-fast/builds/14332/steps/test/logs/stdio fails with "error: cannot use dynamic_cast with -fno-rtti"; is there some way to detect if "-fno-rtti" is the default, and in that case don't execute

r272198 - [DebugInfo] Add calling conventions to DISubroutineType

2016-06-08 Thread Reid Kleckner via cfe-commits
Author: rnk Date: Wed Jun 8 15:41:54 2016 New Revision: 272198 URL: http://llvm.org/viewvc/llvm-project?rev=272198=rev Log: [DebugInfo] Add calling conventions to DISubroutineType Summary: This should have been a very simple change, but it was greatly complicated by the construction of new

[libcxx] r272202 - Mark 2441 as ready

2016-06-08 Thread Marshall Clow via cfe-commits
Author: marshall Date: Wed Jun 8 16:25:38 2016 New Revision: 272202 URL: http://llvm.org/viewvc/llvm-project?rev=272202=rev Log: Mark 2441 as ready Modified: libcxx/trunk/www/upcoming_meeting.html Modified: libcxx/trunk/www/upcoming_meeting.html URL:

Re: [PATCH] D20748: Handle recursion in LLVMIRGeneration Timer

2016-06-08 Thread Davide Italiano via cfe-commits
I agree with Rafael. I ran with -ftime-report for a couple of weeks on a lot of tests and never hit this problem. It actually seems weird we have to handle the timer recursively here as well (I'd like to avoid it at all), but let's see what Richard has to say when he gets to this review. On Wed,

[PATCH] D21163: Strip Android version when looking up toolchain paths.

2016-06-08 Thread Josh Gao via cfe-commits
jmgao created this revision. jmgao added reviewers: srhines, danalbert. jmgao added a subscriber: cfe-commits. Herald added subscribers: srhines, danalbert, tberghammer, aemerson. Android target triples can include a version number in the abi field (e.g. 'aarch64-linux-android21'), used for

Re: [PATCH] D20748: Handle recursion in LLVMIRGeneration Timer

2016-06-08 Thread Rafael Espíndola via cfe-commits
Since we found only one user, I think my preference is to handle it there. Cheers, Rafael On 8 June 2016 at 13:49, Vedant Kumar wrote: > vsk added a comment. > > Ping, any updates on this patch? > > > http://reviews.llvm.org/D20748 > > >

[clang-tools-extra] r272144 - [clang-tidy] correct clang-tidy-diff.py help message

2016-06-08 Thread Alexander Kornienko via cfe-commits
Author: alexfh Date: Wed Jun 8 09:27:43 2016 New Revision: 272144 URL: http://llvm.org/viewvc/llvm-project?rev=272144=rev Log: [clang-tidy] correct clang-tidy-diff.py help message Summary: Looks like the original code was copied from clang-format-diff.py. Update help message to make it

Re: [PATCH] D21050: [clang-tidy] correct clang-tidy-diff.py help message

2016-06-08 Thread Alexander Kornienko via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL272144: [clang-tidy] correct clang-tidy-diff.py help message (authored by alexfh). Changed prior to commit: http://reviews.llvm.org/D21050?vs=59813=60035#toc Repository: rL LLVM

<    1   2