r271777 - [AVX512] Remove space in -1 constants. NFC

Author: ctopper Date: Sat Jun 4 00:43:37 2016 New Revision: 271777 URL: http://llvm.org/viewvc/llvm-project?rev=271777&view=rev Log: [AVX512] Remove space in -1 constants. NFC Modified: cfe/trunk/lib/Headers/avx512dqintrin.h cfe/trunk/lib/Headers/avx512ifmaintrin.h cfe/trunk/lib/Head

r271778 - [X86] Use unsigned types for vector arithmetic in intrinsics to avoid undefined behavior for signed integer overflow.

Author: ctopper Date: Sat Jun 4 00:43:41 2016 New Revision: 271778 URL: http://llvm.org/viewvc/llvm-project?rev=271778&view=rev Log: [X86] Use unsigned types for vector arithmetic in intrinsics to avoid undefined behavior for signed integer overflow. This is really only needed for addition, sub

Re: [PATCH] D19843: Use the name of the file on disk to issue a new diagnostic about non-portable #include and #import paths.

twoh added a comment. Reverted in r271761. http://reviews.llvm.org/D19843 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: r271708 - Use the name of the file on disk to issue a new diagnostic about non-portable #include and #import paths.

I reverted the patch with r271761 and r271764. Thanks, Taewook On 6/3/16, 6:40 PM, "Bruno Cardoso Lopes" wrote: >> The information about whether the file is in a system path is already being >> computed, so it would incur no extra overhead. I'm not sure that's the right >> fix. You are more tha

Re: r271761 - Revert commit r271708

On Fri, Jun 3, 2016 at 8:21 PM Taewook Oh via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Author: twoh > Date: Fri Jun 3 22:14:43 2016 > New Revision: 271761 > > URL: http://llvm.org/viewvc/llvm-project?rev=271761&view=rev > Log: > Revert commit r271708 > Please actually explain why you a

r271762 - Sema: do not attempt to sizeof a dependent type

Author: compnerd Date: Fri Jun 3 22:16:21 2016 New Revision: 271762 URL: http://llvm.org/viewvc/llvm-project?rev=271762&view=rev Log: Sema: do not attempt to sizeof a dependent type We would attempt to evaluate the sizeof a dependent type to check for an integral overflow. However, because the

r271761 - Revert commit r271708

Author: twoh Date: Fri Jun 3 22:14:43 2016 New Revision: 271761 URL: http://llvm.org/viewvc/llvm-project?rev=271761&view=rev Log: Revert commit r271708 Removed: cfe/trunk/test/Lexer/Inputs/case-insensitive-include.h cfe/trunk/test/Lexer/case-insensitive-include-ms.c cfe/trunk/test/L

Re: [clang-tools-extra] r261991 - [clang-tidy] Fix a crash issue when clang-tidy runs with compilation database.

On Thu, Jun 02, 2016 at 01:27:51PM +0200, Alexander Kornienko wrote: > Thank you, Tom. I'm not very familiar with the release process and it will > indeed be better, if you merge the patch. > Hi, There were some merge conflicts, so I wasn't sure about merging the patch. Can you either port the p

Re: [PATCH] D20933: Preallocate ExplodedNode hash table

zaks.anna added a comment. Ben, By the way, thanks for the patch! It's a clever idea. > The implementation of FoldingSet has a vector of bucket pointers. Reserve > preallocates that vector of > bucket pointers to the correct size to avoid rehashing. The allocation of > the nodes themselves

Re: r271708 - Use the name of the file on disk to issue a new diagnostic about non-portable #include and #import paths.

> The information about whether the file is in a system path is already being > computed, so it would incur no extra overhead. I'm not sure that's the right > fix. You are more than welcome to revert the (clang) commit while we think > of the right fix. I'm not sure what the right fix is either, b

Re: [PATCH] D20857: [clang-tidy] Add modernize-explicit-operator-bool check.

alexfh added a comment. Looks like a useful check to have. I'm not sure though, that it has anything to do with "modernize". I'd suggest adding a new "bugprone" module (should be added by http://reviews.llvm.org/D18821, hopefully soon) and moving the check there. Comment at:

Re: r271293 - Fixed bug

Hi, please write better change descriptions. I just looked through a regression range with `svn log -r271271:271293`, and for this change I had to look up the change instead of skimming the change description before I could be sure that this change isn't related to what I'm currently looking at.

Re: [PATCH] D19105: Changes in clang after running http://reviews.llvm.org/D18821

alexfh added a comment. Most of the fixits are still useless, but I didn't notice any false positives (i.e. all warnings would be useful to some degree). So I suggest disabling the fixes completely at least for now. Comment at: include/llvm-c/Core.h:604 @@ -603,3 +603,3 @@ *

r271758 - [Modules] Improve diagnostics for LockFileManager errors

Author: bruno Date: Fri Jun 3 20:13:22 2016 New Revision: 271758 URL: http://llvm.org/viewvc/llvm-project?rev=271758&view=rev Log: [Modules] Improve diagnostics for LockFileManager errors Uses error message now provided by LockFileManager in LLVM r271755. rdar://problem/26529101 Modified:

Re: r271708 - Use the name of the file on disk to issue a new diagnostic about non-portable #include and #import paths.

The information about whether the file is in a system path is already being computed, so it would incur no extra overhead. I'm not sure that's the right fix. You are more than welcome to revert the (clang) commit while we think of the right fix. \e On Jun 3, 2016 5:25 PM, "Bruno Cardoso Lopes" wr

Re: [PATCH] D18821: Add bugprone-bool-to-integer-conversion

alexfh requested changes to this revision. This revision now requires changes to proceed. Comment at: docs/clang-tidy/checks/bugprone-bool-to-integer-conversion.rst:37 @@ +36,3 @@ + +It turns out that the common bug is to have function returning only bools but having int as retur

Re: [PATCH] D20942: [LockFileManager] Improve error output by adding error messages

bruno closed this revision. bruno added a comment. Applied your suggestions and committed in r271755! Thanks http://reviews.llvm.org/D20942 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-c

r271754 - PR27989: only enqueue binary operators into the data recursive int expression

Author: rsmith Date: Fri Jun 3 19:22:31 2016 New Revision: 271754 URL: http://llvm.org/viewvc/llvm-project?rev=271754&view=rev Log: PR27989: only enqueue binary operators into the data recursive int expression evaluator if they are actually int expressions. Modified: cfe/trunk/lib/AST/ExprCo

Re: r271708 - Use the name of the file on disk to issue a new diagnostic about non-portable #include and #import paths.

I think this should be reverted until we figure out how to solve it. Moreover, it looks expensive to check for each file if it's within a system path. I would really like to measure compile time for this change before it goes definitely in. On Fri, Jun 3, 2016 at 4:53 PM, Justin Bogner via cfe-com

Re: [PATCH] D20985: [CUDA] Add implicit conversion of __launch_bounds__ arguments to rvalue.

jlebar added inline comments. Comment at: lib/Sema/SemaDeclAttr.cpp:4079 @@ +4078,3 @@ + if (ValArg.isInvalid()) +return nullptr; + OK, so then we want an assert, not an if? http://reviews.llvm.org/D20985 ___ c

Re: r271708 - Use the name of the file on disk to issue a new diagnostic about non-portable #include and #import paths.

Taewook Oh via cfe-commits writes: > Author: twoh > Date: Fri Jun 3 13:52:51 2016 > New Revision: 271708 > > URL: http://llvm.org/viewvc/llvm-project?rev=271708&view=rev > Log: > Use the name of the file on disk to issue a new diagnostic about > non-portable #include and #import paths. > > Differ

r271750 - CodeGen: correct assertion

Author: compnerd Date: Fri Jun 3 18:26:30 2016 New Revision: 271750 URL: http://llvm.org/viewvc/llvm-project?rev=271750&view=rev Log: CodeGen: correct assertion The assertion added earlier was overly strict. We need to strip the pointer casts (as when constructing the GV). Correct the types (F

r271747 - Bump libclang API minor version after r271351.

Author: mren Date: Fri Jun 3 18:11:41 2016 New Revision: 271747 URL: http://llvm.org/viewvc/llvm-project?rev=271747&view=rev Log: Bump libclang API minor version after r271351. Also use the next enum value for CXObjCPropertyAttr_class. Modified: cfe/trunk/include/clang-c/Index.h Modified:

Re: r271708 - Use the name of the file on disk to issue a new diagnostic about non-portable #include and #import paths.

On 6/3/16, 3:24 PM, "tha...@google.com on behalf of Nico Weber" wrote: > On Fri, Jun 3, 2016 at 6:14 PM, Eric Niebler wrote: >> I just checked, and warnings are not emitted from files in an -isystem path. >> I didn’t have to do anything special to get that behavior. >> I don’t know about -imsvc

Re: r271708 - Use the name of the file on disk to issue a new diagnostic about non-portable #include and #import paths.

On Fri, Jun 3, 2016 at 7:07 PM, Eric Niebler wrote: > On 6/3/16, 3:24 PM, "tha...@google.com on behalf of Nico Weber" < > tha...@google.com on behalf of tha...@chromium.org> wrote: > > On Fri, Jun 3, 2016 at 6:14 PM, Eric Niebler wrote: > >> I just checked, and warnings are not emitted from file

Re: [PATCH] D19165: [clang-tidy] Add modernize-increment-bool check.

alexfh requested changes to this revision. alexfh added a comment. This revision now requires changes to proceed. > -Wdeprecated-increment-bool does it. But what I see from SemaExpr.cpp in > CheckIncrementDecrementOperand, it doesn't have any fixits. If an automated fix for this issue makes sen

Re: [PATCH] D20693: [clang-tidy] New checker to replace dynamic exception specifications

alexfh requested changes to this revision. alexfh added a comment. This revision now requires changes to proceed. Removing from my dashboard until http://reviews.llvm.org/D20428 is submitted. http://reviews.llvm.org/D20693 ___ cfe-commits mailing li

Re: [PATCH] D20196: [clang-tidy] Inefficient string operation

alexfh requested changes to this revision. This revision now requires changes to proceed. Comment at: clang-tidy/performance/InefficientStringConcatenationCheck.cpp:31 @@ +30,3 @@ +MatchFinder *Finder) { + if (!getLangOpts().CPlusPlus) return; + clang-format

Re: r271708 - Use the name of the file on disk to issue a new diagnostic about non-portable #include and #import paths.

On Fri, Jun 3, 2016 at 6:14 PM, Eric Niebler wrote: > I just checked, and warnings are not emitted from files in an -isystem > path. I didn’t have to do anything special to get that behavior. I don’t > know about -imsvc. Is that a clang-cl thing? I can’t say at this point why > the diagnostics sy

Re: [PATCH] D18575: [clang-tidy] New checker to replace deprecated throw() specifications

hintonda abandoned this revision. hintonda added a comment. This revision is OBE. http://reviews.llvm.org/D18575 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: r271708 - Use the name of the file on disk to issue a new diagnostic about non-portable #include and #import paths.

I can investigate the system header issue. And I could add a special exception for if folks feel that that’s the right thing to do. If we’re really worried about how noisy this warning could be, the warning could be disabled by default. Thoughts? Eric On 6/3/16, 2:46 PM, "tha...@google.com

Re: r271708 - Use the name of the file on disk to issue a new diagnostic about non-portable #include and #import paths.

I just checked, and warnings are not emitted from files in an -isystem path. I didn’t have to do anything special to get that behavior. I don’t know about -imsvc. Is that a clang-cl thing? I can’t say at this point why the diagnostics system treats -isystem and -imsvc differently. How about thi

Re: [PATCH] D18575: [clang-tidy] New checker to replace deprecated throw() specifications

alexfh resigned from this revision. alexfh removed a reviewer: alexfh. alexfh added a comment. Cleaning up my Phab dashboard. If http://reviews.llvm.org/D20428 doesn't happen and we'll have to return to this implementation, please re-add me to the reviewers. http://reviews.llvm.org/D18575 _

Re: [PATCH] D20689: [clang-tidy] Suspicious Call Argument checker

alexfh added a comment. In http://reviews.llvm.org/D20689#443266, @varjujan wrote: > Yes, I did. The results from running the checker on LLVM are in the attached > file. Sadly, I could'nt find any real mistakes but as I wrote in the summary, > false positives can still indicate bad naming conve

Re: r271708 - Use the name of the file on disk to issue a new diagnostic about non-portable #include and #import paths.

It's not just windows.h, but windows sdk headers in general (#include , #include #include #include etc). Here's what the warning does on the first few files in Chromium: https://build.chromium.org/p/chromium.fyi/builders/ClangToTWin/builds/8155/steps/compile/logs/stdio (We add /FIIntrin.h -- but

Re: [PATCH] D20985: [CUDA] Add implicit conversion of __launch_bounds__ arguments to rvalue.

tra marked 3 inline comments as done. Comment at: lib/Sema/SemaDeclAttr.cpp:4046 @@ +4045,3 @@ +// non-nullptr Expr result on success. Returns nullptr otherwise and +// may output an error. +static Expr *makeLaunchBoundsArgExpr(Sema &S, Expr *E, jlebar wrote: > Pr

Re: [PATCH] D20985: [CUDA] Add implicit conversion of __launch_bounds__ arguments to rvalue.

tra updated this revision to Diff 59631. tra marked an inline comment as done. tra added a comment. Rephrased comments http://reviews.llvm.org/D20985 Files: lib/Sema/SemaDeclAttr.cpp test/CodeGenCUDA/launch-bounds.cu test/SemaCUDA/pr27778.cu Index: test/SemaCUDA/pr27778.cu ==

Re: [PATCH] D20133: [OpenCL] Fix __builtin_astype for vec3 types.

yaxunl updated this revision to Diff 59630. yaxunl marked 10 inline comments as done. yaxunl added a comment. Add a sema test for mismatched type size. Fix codegen test. http://reviews.llvm.org/D20133 Files: lib/CodeGen/CGExprScalar.cpp test/CodeGenOpenCL/as_type.cl test/SemaOpenCL/as_typ

Re: [PATCH] D20133: [OpenCL] Fix __builtin_astype for vec3 types.

yaxunl added inline comments. Comment at: lib/CodeGen/CGExprScalar.cpp:3394 @@ +3393,3 @@ + if (NumElementsDst == 4) +Args.push_back(llvm::UndefValue::get(CGF.Int32Ty)); + llvm::Constant *Mask = llvm::ConstantVector::get(Args); Anastasia wrote: > should this

Re: r271708 - Use the name of the file on disk to issue a new diagnostic about non-portable #include and #import paths.

Also, once that is resolved, this warning tells people that #include is wrong: ..\..\third_party\ffmpeg\compat/w32pthreads.h(39,10): warning: non-portable path to file ''; specified path differs in case from file name on disk [-Wnonportable-include-path] #include ^~~

[libcxx] r271741 - [libcxx] Fix c++98 test failures.

Author: asiri Date: Fri Jun 3 16:40:03 2016 New Revision: 271741 URL: http://llvm.org/viewvc/llvm-project?rev=271741&view=rev Log: [libcxx] Fix c++98 test failures. Adds XFAIL/UNSUPPORTED lit tags as appropriate. Gets a clean test run for -std=c++98 on Fedora 20. NFC. Modified: libcxx/trun

Re: [PATCH] D20853: [clang-tidy] misc-macro-parentheses: Don't insert parentheses in variable declarations. Fixes bugzilla 26273

alexfh accepted this revision. alexfh added a comment. This revision is now accepted and ready to land. Looks good with a few style comments. Comment at: clang-tidy/misc/MacroParenthesesCheck.cpp:82 @@ +81,3 @@ + + // If we see int/short/struct etc just assume this is a variabl

Re: r271708 - Use the name of the file on disk to issue a new diagnostic about non-portable #include and #import paths.

I'm getting lots of warnings like so: In file included from ..\..\tools\win\static_initializers\static_initializers.cc:5: C:\b\depot_tools\win_toolchain\vs_files\95ddda401ec5678f15eeed01d2bee08fcbc5ee97\DIA SDK\include\dia2.h(30,10): error: non-portable path to file '"Windows.h"'; specified path

Re: [PATCH] D20856: [clang-tidy] readability-identifier-naming - Support for Type Aliases

alexfh accepted this revision. alexfh added a comment. This revision is now accepted and ready to land. Looks good. Thank you for the patch! Do you need me to submit it? http://reviews.llvm.org/D20856 ___ cfe-commits mailing list cfe-commits@lists.l

Re: [PATCH] D20985: [CUDA] Add implicit conversion of __launch_bounds__ arguments to rvalue.

jlebar accepted this revision. This revision is now accepted and ready to land. Comment at: lib/Sema/SemaDeclAttr.cpp:4044 @@ +4043,3 @@ +// Checks whether an argument of launch_bounds attribute is +// acceptable, performs implicit conversion to Rvalue and returns +// non-nullptr

Re: [PATCH] D20985: [CUDA] Add implicit conversion of __launch_bounds__ arguments to rvalue.

tra updated this revision to Diff 59624. tra added a comment. Addressed Justin's comments. http://reviews.llvm.org/D20985 Files: lib/Sema/SemaDeclAttr.cpp test/CodeGenCUDA/launch-bounds.cu test/SemaCUDA/pr27778.cu Index: test/SemaCUDA/pr27778.cu ==

Re: [PATCH] D20917: [clang-tidy] modernize-use-auto: don't remove stars by default

This revision was automatically updated to reflect the committed changes. Closed by commit rL271739: [clang-tidy] modernize-use-auto: don't remove stars by default (authored by alexfh). Changed prior to commit: http://reviews.llvm.org/D20917?vs=59599&id=59625#toc Repository: rL LLVM http://

[clang-tools-extra] r271739 - [clang-tidy] modernize-use-auto: don't remove stars by default

Author: alexfh Date: Fri Jun 3 16:22:58 2016 New Revision: 271739 URL: http://llvm.org/viewvc/llvm-project?rev=271739&view=rev Log: [clang-tidy] modernize-use-auto: don't remove stars by default Summary: By default, modernize-use-auto check will retain stars when replacing an explicit type with

[libunwind] r271737 - [libunwind] Remove unused code.

Author: asiri Date: Fri Jun 3 15:57:48 2016 New Revision: 271737 URL: http://llvm.org/viewvc/llvm-project?rev=271737&view=rev Log: [libunwind] Remove unused code. The whole file is guarded with #if _LIBUNWIND_ARM_EHABI, and then in the middle we have these two blocks, which render them pretty un

Re: [PATCH] D20985: [CUDA] Add implicit conversion of __launch_bounds__ arguments to rvalue.

jlebar added a comment. In http://reviews.llvm.org/D20985#448836, @tra wrote: > In http://reviews.llvm.org/D20985#448822, @jlebar wrote: > > > How is this different from test/SemaCUDA/launch_bounds.cu:27-28? It does > > > > const int constint = 512; > > __launch_bounds__(constint) void TestC

Re: [PATCH] D20985: [CUDA] Add implicit conversion of __launch_bounds__ arguments to rvalue.

tra added a comment. In http://reviews.llvm.org/D20985#448822, @jlebar wrote: > How is this different from test/SemaCUDA/launch_bounds.cu:27-28? It does > > const int constint = 512; > __launch_bounds__(constint) void TestConstInt(void); > > > which looks verbatim the same as this testcas

Re: [PATCH] D20985: [CUDA] Add implicit conversion of __launch_bounds__ arguments to rvalue.

jlebar added a comment. How is this different from test/SemaCUDA/launch_bounds.cu:27-28? It does const int constint = 512; __launch_bounds__(constint) void TestConstInt(void); which looks verbatim the same as this testcase. http://reviews.llvm.org/D20985 ___

[PATCH] D20985: [CUDA] Add implicit conversion of __launch_bounds__ arguments to rvalue.

tra created this revision. tra added reviewers: rsmith, jlebar. tra added a subscriber: cfe-commits. Fixes clang crash reported in PR27778. http://reviews.llvm.org/D20985 Files: lib/Sema/SemaDeclAttr.cpp test/CodeGenCUDA/launch-bounds.cu test/SemaCUDA/pr27778.cu Index: test/SemaCUDA/pr277

Re: [PATCH] D20917: [clang-tidy] modernize-use-auto: don't remove stars by default

Prazek added inline comments. Comment at: clang-tidy/modernize/UseAutoCheck.cpp:338 @@ -329,8 +337,3 @@ -// Remove explicitly written '*' from declarations where there's more than -// one declaration in the declaration list. -if (Dec == *D->decl_begin()) - conti

Re: r271692 - Don't pass --build-id to ld by default.

On 3 June 2016 at 15:53, Nico Weber via cfe-commits wrote: > Can you add this to the release notes? It'll for example break chromium's > crash server (we can fix this on our end by explicitly passing > -Wl,--build-id for release builds, but we only saw this commit fly by by > chance.) Good point.

Re: [PATCH] D20917: [clang-tidy] modernize-use-auto: don't remove stars by default

alexfh added inline comments. Comment at: clang-tidy/modernize/UseAutoCheck.cpp:338 @@ -329,8 +337,3 @@ -// Remove explicitly written '*' from declarations where there's more than -// one declaration in the declaration list. -if (Dec == *D->decl_begin()) - conti

Re: [PATCH] D20382: Add postorder support to RecursiveASTVisitor

teemperor added a comment. The previous stats were wrong (only applied this patch, not the patch using the code): Release: 63311672 Byte -> 77212960 Byte (+22% or +13.8 MB) http://reviews.llvm.org/D20382 ___ cfe-commits mailing list cfe-commits@li

Re: r271692 - Don't pass --build-id to ld by default.

Can you add this to the release notes? It'll for example break chromium's crash server (we can fix this on our end by explicitly passing -Wl,--build-id for release builds, but we only saw this commit fly by by chance.) On Fri, Jun 3, 2016 at 1:26 PM, Rafael Espindola via cfe-commits < cfe-commits@

Re: [PATCH] D20917: [clang-tidy] modernize-use-auto: don't remove stars by default

Prazek added a subscriber: Prazek. Prazek added a comment. besides it lgtm Comment at: clang-tidy/modernize/UseAutoCheck.cpp:338 @@ -329,8 +337,3 @@ -// Remove explicitly written '*' from declarations where there's more than -// one declaration in the declaration list.

Re: [PATCH] D20917: [clang-tidy] modernize-use-auto: don't remove stars by default

alexfh added a comment. In http://reviews.llvm.org/D20917#447046, @sbenza wrote: > Is it a typo in the description when it says that when RemoveStar is on we > will write 'auto*' instead if 'auto'? Yep, the whole patch description is a typo, fixed ;) http://reviews.llvm.org/D20917 ___

Re: [PATCH] D20917: [clang-tidy] modernize-use-auto: don't remove stars by default

alexfh updated this revision to Diff 59599. alexfh marked an inline comment as done. alexfh added a comment. - Updated formatting in the doc. http://reviews.llvm.org/D20917 Files: clang-tidy/modernize/UseAutoCheck.cpp clang-tidy/modernize/UseAutoCheck.h docs/clang-tidy/checks/modernize-us

Re: [PATCH] D19843: Use the name of the file on disk to issue a new diagnostic about non-portable #include and #import paths.

twoh added a subscriber: twoh. twoh closed this revision. twoh added a comment. I have commit in r271708: http://reviews.llvm.org/rL271708 http://reviews.llvm.org/D19843 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/c

r271708 - Use the name of the file on disk to issue a new diagnostic about non-portable #include and #import paths.

Author: twoh Date: Fri Jun 3 13:52:51 2016 New Revision: 271708 URL: http://llvm.org/viewvc/llvm-project?rev=271708&view=rev Log: Use the name of the file on disk to issue a new diagnostic about non-portable #include and #import paths. Differential Revision: http://reviews.llvm.org/D19843 Corre

[PATCH] D20979: [OpenCL] Use function attribute/metadata to represent kernel attributes

yaxunl created this revision. yaxunl added reviewers: Anastasia, bader, pxli168. yaxunl added subscribers: cfe-commits, tstellarAMD. This patch attempts to use target-dependent function attribute to represent reqd_work_group_size, work_group_size_hint and vector_type_hint kernel attributes and u

r271702 - [Title] Revert test commit

Author: twoh Date: Fri Jun 3 13:30:12 2016 New Revision: 271702 URL: http://llvm.org/viewvc/llvm-project?rev=271702&view=rev Log: [Title] Revert test commit Summary: Revert test commit Trac Bug: # Blame Rev: Reviewed By: Test Plan: Revert Plan: Database Impact: Memcache Impact: Other Not

[RFC] Embedded bitcode and related upstream (Part II)

Hi everyone I am still in the process of upstreaming some improvements to the embed bitcode option. If you want more background, you can read the previous RFC (http://lists.llvm.org/pipermail/llvm-dev/2016-February/094851.html

r271701 - Test commit

Author: twoh Date: Fri Jun 3 13:27:39 2016 New Revision: 271701 URL: http://llvm.org/viewvc/llvm-project?rev=271701&view=rev Log: Test commit Modified: cfe/trunk/include/clang/Basic/Builtins.def Modified: cfe/trunk/include/clang/Basic/Builtins.def URL: http://llvm.org/viewvc/llvm-project

Re: [PATCH] D20602: Add .rgba syntax extension to ext_vector_type types

pirama added a comment. Friendly ping... http://reviews.llvm.org/D20602 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D20964: [clang-tidy] Add modernize-use-emplace

Eugene.Zelenko added a comment. In http://reviews.llvm.org/D20964#448525, @Prazek wrote: > In http://reviews.llvm.org/D20964#448455, @Eugene.Zelenko wrote: > > > I think will be good idea to try this check with LLVM STL too. > > > You mean llvm::SmallVector stuff? No, I meant to build example w

Re: [PATCH] D20964: [clang-tidy] Add modernize-use-emplace

Prazek added a comment. In http://reviews.llvm.org/D20964#448455, @Eugene.Zelenko wrote: > I think will be good idea to try this check with LLVM STL too. You mean llvm::SmallVector stuff? http://reviews.llvm.org/D20964 ___ cfe-commits mailing lis

Re: [PATCH] D20964: [clang-tidy] Add modernize-use-emplace

Prazek updated this revision to Diff 59580. Prazek marked 4 inline comments as done. Prazek added a comment. - Review fixes http://reviews.llvm.org/D20964 Files: clang-tidy/modernize/CMakeLists.txt clang-tidy/modernize/ModernizeTidyModule.cpp clang-tidy/modernize/UseEmplaceCheck.cpp cla

RE: r271413 - Fixup list of available extensions

No worries! Thanks! Anastasia -Original Message- From: Jan Vesely [mailto:jv...@scarletmail.rutgers.edu] On Behalf Of Jan Vesely Sent: 03 June 2016 18:35 To: Anastasia Stulova; cfe-commits@lists.llvm.org Cc: nd Subject: Re: r271413 - Fixup list of available extensions of course. sorry ab

Re: r271413 - Fixup list of available extensions

of course. sorry about that. I didn't realize the patches had different title from phabricator. Jan On Fri, 2016-06-03 at 17:14 +, Anastasia Stulova wrote: > Hi Jan, > > In the future could you please keep "[OpenCL]" prefix for all commit > messages related to OpenCL. :) > > Thanks, > Anast

RE: r269670 - [OpenCL] Add supported OpenCL extensions to target info.

Hi Jeroen, I am not quite sure what your question is about in (a). Regarding (b) please see example in test/SemaOpenCL/extensions.cl from the commit r269670. Please note that SPIR target supports all extensions but disables them by default (see commit r269670). The latter logic hasn't been mod

Re: Don't pass --build-id by default.

r271692, thanks. Cheers, Rafael On 3 June 2016 at 10:19, Ed Maste wrote: > On 2 June 2016 at 21:19, Hal Finkel via cfe-commits > wrote: >> - Original Message - >>> From: "Rafael Espíndola" >>> To: "Hal Finkel" >>> Cc: "cfe-commits cfe" >>> Sent: Thursday, June 2, 2016 7:06:26 PM >>>

r271692 - Don't pass --build-id to ld by default.

Author: rafael Date: Fri Jun 3 12:26:16 2016 New Revision: 271692 URL: http://llvm.org/viewvc/llvm-project?rev=271692&view=rev Log: Don't pass --build-id to ld by default. We now have a cmake option to change the default: ENABLE_LINKER_BUILD_ID. The reason is that build-id is fairly expensive,

Re: [PATCH] D20338: [PCH] Fixed overridden files always invalidating preamble even when unchanged

cameron314 updated the summary for this revision. cameron314 removed rL LLVM as the repository for this revision. cameron314 updated this revision to Diff 59577. cameron314 added a comment. It took some modifications to the ASTUnit to support a virtual file system with a PCH parse/reparse (prelim

Re: Don't pass --build-id by default.

On 2 June 2016 at 21:19, Hal Finkel via cfe-commits wrote: > - Original Message - >> From: "Rafael Espíndola" >> To: "Hal Finkel" >> Cc: "cfe-commits cfe" >> Sent: Thursday, June 2, 2016 7:06:26 PM >> Subject: Re: Don't pass --build-id by default. >> >> > This is going to break a lot of

Re: [PATCH] D20964: [clang-tidy] Add modernize-use-emplace

Eugene.Zelenko added a subscriber: Eugene.Zelenko. Eugene.Zelenko added a comment. I think will be good idea to try this check with LLVM STL too. Comment at: docs/clang-tidy/checks/modernize-use-emplace.rst:47 @@ +46,3 @@ + +In this case the calls of push_back won't be replaced.

RE: r271413 - Fixup list of available extensions

Hi Jan, In the future could you please keep "[OpenCL]" prefix for all commit messages related to OpenCL. :) Thanks, Anastasia -Original Message- From: cfe-commits [mailto:cfe-commits-boun...@lists.llvm.org] On Behalf Of Jan Vesely via cfe-commits Sent: 01 June 2016 19:05 To: cfe-commi

Re: r271427 - [driver][arm] add armv7 and thumb include and lib paths

Fixed in r271689. On Fri, Jun 3, 2016 at 9:55 AM, Ismail Donmez wrote: > This is due to lib vs lib64. Will fix. Sorry for the wrong diagnosis :) > > On Thu, Jun 2, 2016 at 8:24 PM, Chih-hung Hsieh wrote: >> Ismail, >> I saw that android-ndk-standalone.cpp has already -stdlib=libstdc++. >> Could

r271689 - Fix test for LLVM_LIBDIR_SUFFIX=64 case.

Author: ismail Date: Fri Jun 3 12:06:52 2016 New Revision: 271689 URL: http://llvm.org/viewvc/llvm-project?rev=271689&view=rev Log: Fix test for LLVM_LIBDIR_SUFFIX=64 case. Modified: cfe/trunk/test/Driver/android-ndk-standalone.cpp Modified: cfe/trunk/test/Driver/android-ndk-standalone.cpp

Re: [PATCH] D20681: Add target-specific pre-linking passes to Clang

Anastasia added a comment. Do you think we could add any test for this change? Comment at: lib/CodeGen/BackendUtil.cpp:657 @@ -627,2 +656,3 @@ + bool EmitAssemblyHelper::AddEmitPasses(BackendAction Action, Remove the empty line please Commen

r271688 - Obj-C: Fix assert-on-invalid (PR27822)

Author: hans Date: Fri Jun 3 11:59:13 2016 New Revision: 271688 URL: http://llvm.org/viewvc/llvm-project?rev=271688&view=rev Log: Obj-C: Fix assert-on-invalid (PR27822) Clang would assert when isObjCInstancetype() was called on a tok::annot_cxxscope token. Modified: cfe/trunk/include/clang/

Re: [PATCH] D20964: [clang-tidy] Add modernize-use-emplace

sbarzowski added inline comments. Comment at: clang-tidy/modernize/UseEmplaceCheck.cpp:40 @@ +39,3 @@ + // passed pointer because smart pointer won't be constructed + // (and destructed) as in push_back case. + auto isCtorOfSmartPtr = hasDeclaration(cxxConstructorDecl(

Re: [PATCH] D20964: [clang-tidy] Add modernize-use-emplace

Prazek marked 2 inline comments as done. Comment at: clang-tidy/modernize/UseEmplaceCheck.cpp:40 @@ +39,3 @@ + // passed pointer because smart pointer won't be constructed + // (and destructed) as in push_back case. + auto isCtorOfSmartPtr = hasDeclaration(cxxConstructorDecl( -

Re: [PATCH] D20090: [OPENCL] Fix wrongly vla error for OpenCL array.

Anastasia accepted this revision. Anastasia added a comment. LGTM! Thanks! http://reviews.llvm.org/D20090 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D20444: [OpenCL] Include opencl-c.h by default as a clang module

Anastasia added inline comments. Comment at: include/clang/Basic/DiagnosticSemaKinds.td:7347 @@ -7346,3 +7346,3 @@ def err_blocks_disable : Error<"blocks support disabled - compile with -fblocks" - " or pick a deployment target that supports them">; + " or %select{pick a deplo

Re: [PATCH] D20964: [clang-tidy] Add modernize-use-emplace

sbarzowski added inline comments. Comment at: clang-tidy/modernize/UseEmplaceCheck.cpp:40 @@ +39,3 @@ + // passed pointer because smart pointer won't be constructed + // (and destructed) as in push_back case. + auto isCtorOfSmartPtr = hasDeclaration(cxxConstructorDecl(

Re: [PATCH] D20964: [clang-tidy] Add modernize-use-emplace

Prazek updated this revision to Diff 59568. Prazek added a comment. - Post review fix http://reviews.llvm.org/D20964 Files: clang-tidy/modernize/CMakeLists.txt clang-tidy/modernize/ModernizeTidyModule.cpp clang-tidy/modernize/UseEmplaceCheck.cpp clang-tidy/modernize/UseEmplaceCheck.h

Re: [PATCH] D20964: [clang-tidy] Add modernize-use-emplace

Prazek added inline comments. Comment at: clang-tidy/modernize/UseEmplaceCheck.cpp:31 @@ +30,3 @@ +on(hasType(cxxRecordDecl(hasName(VectorName) + .bind("call"); + ok, std::list works for me. I just don't want to spend much time

Re: [PATCH] D20053: [clang-tidy] Add misc-unnecessary-mutable check.

etienneb added inline comments. Comment at: clang-tidy/misc/UnnecessaryMutableCheck.cpp:23 @@ +22,3 @@ + +// Matcher checking if the declaration is non-macro existent mutable which is +// not dependent on context. add an anonymous namespace around this declaration

Re: [PATCH] D20964: [clang-tidy] Add modernize-use-emplace

sbarzowski added inline comments. Comment at: clang-tidy/modernize/UseEmplaceCheck.cpp:30 @@ +29,3 @@ + cxxMemberCallExpr(hasDeclaration(functionDecl(hasName(PushBackName))), +on(hasType(cxxRecordDecl(hasName(VectorName) + .bind("call"); -

Re: [PATCH] D20963: [mips] The P5600 does not support N32/N64 since it's a 32-bit CPU.

atanasyan accepted this revision. atanasyan added a comment. This revision is now accepted and ready to land. LGTM http://reviews.llvm.org/D20963 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/

Re: [PATCH] D20961: [mips] Replace almost all Arch checks in MipsTargetInfo with ABI checks. NFC.

atanasyan accepted this revision. atanasyan added a comment. This revision is now accepted and ready to land. LGTM http://reviews.llvm.org/D20961 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/

Re: [PATCH] D20964: [clang-tidy] Add modernize-use-emplace

dblaikie added a subscriber: dblaikie. dblaikie added a comment. Could you describe in more detail (ideally in the original patch review summary/description) what this transformation does? Under what conditions does it suggest emplace, etc. http://reviews.llvm.org/D20964 ___

Re: [PATCH] D20964: [clang-tidy] Add modernize-use-emplace

Prazek marked 2 inline comments as done. Prazek added a comment. http://reviews.llvm.org/D20964 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D20964: [clang-tidy] Add modernize-use-emplace

Prazek marked 4 inline comments as done. Comment at: clang-tidy/modernize/UseEmplaceCheck.cpp:29 @@ +28,3 @@ + auto callPushBack = + cxxMemberCallExpr(hasDeclaration(functionDecl(hasName(PushBackName))), +on(hasType(cxxRecordDecl(hasName(VectorName)))

Re: [PATCH] D20964: [clang-tidy] Add modernize-use-emplace

Prazek updated this revision to Diff 59562. Prazek added a comment. Learning how to use arc http://reviews.llvm.org/D20964 Files: clang-tidy/modernize/CMakeLists.txt clang-tidy/modernize/ModernizeTidyModule.cpp clang-tidy/modernize/UseEmplaceCheck.cpp clang-tidy/modernize/UseEmplaceChec

Re: [PATCH] D20964: [clang-tidy] Add modernize-use-emplace

Prazek updated this revision to Diff 59561. Prazek marked an inline comment as done. Prazek added a comment. Fixed stuff http://reviews.llvm.org/D20964 Files: clang-tidy/modernize/UseEmplaceCheck.cpp docs/ReleaseNotes.rst docs/clang-tidy/checks/modernize-use-emplace.rst Index: docs/clang

  1   2   >