Re: [PATCH] D13497: Silence warning from lit about missing clang-interpreter

richard.barton.arm abandoned this revision. richard.barton.arm added a comment. Adandoning and starting again with cfe-commits as subscriber. http://reviews.llvm.org/D13497 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.or

Re: [PATCH] D9888: [OPENMP] Driver support for OpenMP offloading

Hahnfeld added a comment. Currently trying to test, but 1. Offloading to the same target isn't supported (`x86_64-unknown-linux-gnu` as host and device) - this was working with `clang-omp` The produced IR isn't showing any calls to the target library and on linkage it complains: undefined r

[PATCH] D13498: Silence spurious warning from lit about missing clang-interpreter

richard.barton.arm created this revision. richard.barton.arm added reviewers: ddunbar, rafael. richard.barton.arm added a subscriber: cfe-commits. Clang's lit.cfg checks for the presence of a few test binaries, including the clang-interpreter. This binary is only built if building the clang examp

[PATCH] D13499: Convert test to take input from stdin rather than reading a file

richard.barton.arm created this revision. richard.barton.arm added a reviewer: ABataev. richard.barton.arm added a subscriber: cfe-commits. Herald added a subscriber: aemerson. This test takes input from a file rather than from stdin and then goes on to check for a label "bar". This could match s

Re: [PATCH] D13496: Convert test to take input from stdin rather than reading a file

richard.barton.arm abandoned this revision. richard.barton.arm added a comment. Abandoning as I forgot to add cfe-commits. http://reviews.llvm.org/D13496 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/l

[PATCH] D13500: Improve helpfulness of assert message when number of diagnostics overflows number of available enum values.

richard.barton.arm created this revision. richard.barton.arm added a reviewer: rafael. richard.barton.arm added a subscriber: cfe-commits. There are two asserts that can trigger if the number of diagnostic messages in a kind overflows the number of available enums values. The one that fires if you

Re: [PATCH] D13495: Improve the helpfulness of an assert message

richard.barton.arm abandoned this revision. richard.barton.arm added a comment. Abandoning as cfe-commits was not on subscribers http://reviews.llvm.org/D13495 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mai

r249525 - [VFS] Also drop '.' when adding files to an in-memory FS.

Author: d0k Date: Wed Oct 7 03:32:50 2015 New Revision: 249525 URL: http://llvm.org/viewvc/llvm-project?rev=249525&view=rev Log: [VFS] Also drop '.' when adding files to an in-memory FS. Otherwise we won't be able to find them later. Modified: cfe/trunk/lib/Basic/VirtualFileSystem.cpp c

[clang-tools-extra] r249526 - [VFS] Switch clang-tidy tests to use an in-memory fs.

Author: d0k Date: Wed Oct 7 03:35:23 2015 New Revision: 249526 URL: http://llvm.org/viewvc/llvm-project?rev=249526&view=rev Log: [VFS] Switch clang-tidy tests to use an in-memory fs. Again, this is both cleaner and completely removes any depedency on the host file system. Modified: clang-to

[PATCH] D13504: Prevent modernize-use-auto from emitting a warning when 'auto' was already being used.

angelgarcia created this revision. angelgarcia added reviewers: klimek, bkramer. angelgarcia added subscribers: alexfh, cfe-commits. This fixes https://llvm.org/bugs/show_bug.cgi?id=25082 . http://reviews.llvm.org/D13504 Files: clang-tidy/modernize/UseAutoCheck.cpp test/clang-tidy/modernize-

Re: [PATCH] D13504: Prevent modernize-use-auto from emitting a warning when 'auto' was already being used.

alexfh added inline comments. Comment at: test/clang-tidy/modernize-use-auto-new.cpp:100 @@ -98,1 +99,3 @@ + // Don't warn when 'auto' is already being used. + auto aut = new MyType(); } Please add tests for `auto *` and `const auto *`. http://reviews.llvm.or

r249532 - [VFS] Refactor VFSFromYAML a bit.

Author: d0k Date: Wed Oct 7 05:05:44 2015 New Revision: 249532 URL: http://llvm.org/viewvc/llvm-project?rev=249532&view=rev Log: [VFS] Refactor VFSFromYAML a bit. - Rename it to RedirectingFileSystem. This is what it does, YAML is just a serialization format for it. - Consistently use unique_p

Re: [PATCH] D13504: Prevent modernize-use-auto from emitting a warning when 'auto' was already being used.

angelgarcia updated this revision to Diff 36724. angelgarcia added a comment. Good point. Solved. http://reviews.llvm.org/D13504 Files: clang-tidy/modernize/UseAutoCheck.cpp test/clang-tidy/modernize-use-auto-new.cpp Index: test/clang-tidy/modernize-use-auto-new.cpp ===

cfe-commits@lists.llvm.org

Author: abataev Date: Wed Oct 7 05:22:08 2015 New Revision: 249534 URL: http://llvm.org/viewvc/llvm-project?rev=249534&view=rev Log: Fix crash in codegen on casting to `bool &`. Currently codegen crashes trying to emit casting to bool &. It happens because bool type is converted to i1 and later

cfe-commits@lists.llvm.org

This revision was automatically updated to reflect the committed changes. Closed by commit rL249534: Fix crash in codegen on casting to `bool &`. (authored by ABataev). Changed prior to commit: http://reviews.llvm.org/D13325?vs=36179&id=36725#toc Repository: rL LLVM http://reviews.llvm.org/

r249535 - Make the test take input from stdin to prevent matching characters in a file path

Author: rbarton Date: Wed Oct 7 05:33:36 2015 New Revision: 249535 URL: http://llvm.org/viewvc/llvm-project?rev=249535&view=rev Log: Make the test take input from stdin to prevent matching characters in a file path Modified: cfe/trunk/test/OpenMP/for_simd_codegen.cpp Modified: cfe/trunk/te

Re: [PATCH] D13368: [clang-tidy] add check cppcoreguidelines-pro-type-static-cast-downcast

I think, it would be more consistent to have clang-tidy insert a link to the check documentation (on llvm.org) into each warning. The documentation pages already contain links to the relevant rules. We could even version documentation pages, if we think it's important (but afaiu, w

Re: [PATCH] D13498: Silence spurious warning from lit about missing clang-interpreter

rengolin added a subscriber: rengolin. rengolin accepted this revision. rengolin added a reviewer: rengolin. rengolin added a comment. This revision is now accepted and ready to land. Makes sense. LGTM. Thanks! http://reviews.llvm.org/D13498 ___ cfe

r249538 - Silence warning about not being able to find clang-interpreter

Author: rbarton Date: Wed Oct 7 06:14:25 2015 New Revision: 249538 URL: http://llvm.org/viewvc/llvm-project?rev=249538&view=rev Log: Silence warning about not being able to find clang-interpreter This binary is only built with the examples project, so only require it then. Modified: cfe/tru

Re: [PATCH] D13398: [clang-tidy] add check cppcoreguidelines-pro-type-const-cast

On Tue, Oct 6, 2015 at 11:07 PM, Alexander Kornienko wrote: > alexfh added a comment. > > It looks like potentially we're going to have tens of checks in this module. > I wonder whether we should start organizing the checks somehow right away. > For example, create a directory (and a namespace)

[clang-tools-extra] r249540 - Add checker for the C++ Core Guidelines: cppcoreguidelines-pro-type-const-cast.

Author: aaronballman Date: Wed Oct 7 07:24:57 2015 New Revision: 249540 URL: http://llvm.org/viewvc/llvm-project?rev=249540&view=rev Log: Add checker for the C++ Core Guidelines: cppcoreguidelines-pro-type-const-cast. Patch by Matthias Gehre! Added: clang-tools-extra/trunk/clang-tidy/cppco

Re: [PATCH] D13398: [clang-tidy] add check cppcoreguidelines-pro-type-const-cast

aaron.ballman closed this revision. aaron.ballman added a comment. Thanks for the patch! I've commit in r249540. Given the number of checkers that you're working on, I would recommend getting commit privileges if you'd like them. http://reviews.llvm.org/D13398 __

Re: [PATCH] D13368: [clang-tidy] add check cppcoreguidelines-pro-type-static-cast-downcast

aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. On Tue, Oct 6, 2015 at 5:46 PM, Matthias Gehre wrote: > mgehre marked an inline comment as done. > mgehre added a comment. > > I cannot think of any way to improve the usefu

Re: [PATCH] D13311: [clang-tidy] Add check cppcoreguidelines-pro-bounds-pointer-arithmetic

aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. On Tue, Oct 6, 2015 at 5:31 PM, Matthias Gehre wrote: > mgehre marked 2 inline comments as done. > > > Comment at: > test/clang-tidy/cppcoreguidelines-pro-

Re: [PATCH] D13482: Revised Initial patch for PS4 toolchain

pgousseau added a comment. For some reasons the patch file I get from the "download raw diff link" does not contain the diff for the added .keep files? But the .keep files show up in phabricator, so I guess it is a phabricator issue? Repository: rL LLVM http://reviews.llvm.org/D13482 ___

Re: r249392 - clang-format: Make IncludeCategories configurable in .clang-format file.

I am now getting sphinx warnings from this commit: /opt/llvm/build-llvm/src/llvm/tools/clang/docs/ClangFormatStyleOptions.rst:159: ERROR: Error in "code-block" directive: maximum 1 argument(s) allowed, 3 supplied. .. code-block:: c++ someLongFunction(argument1, argument2); /o

r249542 - clang-format: Hopefully fix code blocks in docs.

Author: djasper Date: Wed Oct 7 08:02:45 2015 New Revision: 249542 URL: http://llvm.org/viewvc/llvm-project?rev=249542&view=rev Log: clang-format: Hopefully fix code blocks in docs. Otherwise I will have to install sphinx ;).. Modified: cfe/trunk/docs/ClangFormatStyleOptions.rst cfe/tru

Re: r249392 - clang-format: Make IncludeCategories configurable in .clang-format file.

Should be fixed now. Sorry about that. On Wed, Oct 7, 2015 at 3:00 PM, Aaron Ballman wrote: > I am now getting sphinx warnings from this commit: > > > /opt/llvm/build-llvm/src/llvm/tools/clang/docs/ClangFormatStyleOptions.rst:159: > ERROR: Error in "code-block" directive: > maximum 1 argument(s)

Re: r249392 - clang-format: Make IncludeCategories configurable in .clang-format file.

On Wed, Oct 7, 2015 at 9:07 AM, Daniel Jasper wrote: > Should be fixed now. Sorry about that. No worries, thank you for the quick fix! ~Aaron > > On Wed, Oct 7, 2015 at 3:00 PM, Aaron Ballman > wrote: >> >> I am now getting sphinx warnings from this commit: >> >> >> /opt/llvm/build-llvm/src/ll

[PATCH] D13510: [PATCH] Support C++ Core Guidelines copy assignment restrictions

aaron.ballman created this revision. aaron.ballman added reviewers: Eugene.Zelenko, alexfh, mgehre, sbenza. aaron.ballman added a subscriber: cfe-commits. Eugene pointed out to me that our existing misc-assign-operator-signature check was almost perfectly implementing the C++ Core Guideline guida

Re: [PATCH] D13304: Avoid inlining in throw statement

junbuml added a comment. Is there any comment on this change? Note that with change, I observed about 6% performance improvement in spec2006/xalancbmk. I will address any comment to get this in. http://reviews.llvm.org/D13304 ___ cfe-commits maili

Re: [PATCH] D12358: [Analyzer] Handling constant bound loops

seaneveson updated this revision to Diff 36739. seaneveson added a comment. Updated to latest revision. Change patch to widen all loops which do not exit. Changed to widen on block entrance so the guard condition is accounted for (thanks for the suggestion). Updated tests to reflect changes. ht

[PATCH] D13513: [clang-format] Stop alignment sequences on open braces and parens when aligning assignments.

berenm created this revision. berenm added a reviewer: djasper. berenm added a subscriber: cfe-commits. Herald added a subscriber: klimek. This was done correctly when aligning the declarations, but not when aligning assignments. FIXME: The code between assignments and declarations alignment is

Re: [PATCH] D12358: [Analyzer] Handling constant bound loops

seaneveson marked 4 inline comments as done. seaneveson added a comment. There are some issues which haven't been resolved yet: - There is a loss of precision for loops that need to be executed exactly maxBlockVisitOnPath times, as the loop body is executed with the widened state **instead** of

Re: [PATCH] D13513: [clang-format] Stop alignment sequences on open braces and parens when aligning assignments.

djasper accepted this revision. djasper added a comment. This revision is now accepted and ready to land. Looks good, thank you. Comment at: unittests/Format/FormatTest.cpp:8657 @@ +8656,3 @@ + "};\n" + "int i = 0;\n" + "auto v = type{\

r249552 - [clang-format] Stop alignment sequences on open braces and parens when

Author: djasper Date: Wed Oct 7 10:03:26 2015 New Revision: 249552 URL: http://llvm.org/viewvc/llvm-project?rev=249552&view=rev Log: [clang-format] Stop alignment sequences on open braces and parens when aligning assignments. This was done correctly when aligning the declarations, but not when a

Re: [PATCH] D13221: Make CompilerInvocation's use of the debug options more understandable.

dougk marked an inline comment as done. dougk added a comment. James, you're right, 'gdwarf-2' followed by 'line-tables-only' works, but the opposite order doesn't. So it halfway works, which is better than not working at all. Comment at: lib/Driver/Tools.cpp:2353 @@ +2352,3 @@

Re: [PATCH] D13513: [clang-format] Stop alignment sequences on open braces and parens when aligning assignments.

djasper closed this revision. djasper added a comment. Submitted as r249552. http://reviews.llvm.org/D13513 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D13513: [clang-format] Stop alignment sequences on open braces and parens when aligning assignments.

berenm added inline comments. Comment at: unittests/Format/FormatTest.cpp:8657 @@ +8656,3 @@ + "};\n" + "int i = 0;\n" + "auto v = type{\n" djasper wrote: > So, it is kind of interesting that we (would) align with a lambd

Re: [PATCH] D12489: [clang-format] Fixed missing space between Obj-C for/in and a typecast

djasper closed this revision. djasper added a comment. Submitted as r249553. Sorry for the delay. http://reviews.llvm.org/D12489 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] r249555 - Loosening the restriction on variadic function definitions so that extern "C" function definitions are permissible.

Author: aaronballman Date: Wed Oct 7 10:14:10 2015 New Revision: 249555 URL: http://llvm.org/viewvc/llvm-project?rev=249555&view=rev Log: Loosening the restriction on variadic function definitions so that extern "C" function definitions are permissible. Modified: clang-tools-extra/trunk/cla

Re: [PATCH] D13446: [PATCH] Add checker discouraging definition of variadic function definitions in C++

On Tue, Oct 6, 2015 at 4:51 PM, Aaron Ballman wrote: > On Tue, Oct 6, 2015 at 4:49 PM, Joerg Sonnenberger via cfe-commits > wrote: >> On Tue, Oct 06, 2015 at 04:20:05PM -0400, Aaron Ballman via cfe-commits >> wrote: >>> On Tue, Oct 6, 2015 at 4:12 PM, Joerg Sonnenberger via cfe-commits >>> wrot

r249553 - clang-format: Fixed missing space between Obj-C for/in and a typecast.

Author: djasper Date: Wed Oct 7 10:09:08 2015 New Revision: 249553 URL: http://llvm.org/viewvc/llvm-project?rev=249553&view=rev Log: clang-format: Fixed missing space between Obj-C for/in and a typecast. Fixes this bug: https://llvm.org/bugs/show_bug.cgi?id=24504 TokenAnnotator::spaceRequiredBe

r249556 - [VFS] Port driver tool chains to VFS.

Author: d0k Date: Wed Oct 7 10:48:01 2015 New Revision: 249556 URL: http://llvm.org/viewvc/llvm-project?rev=249556&view=rev Log: [VFS] Port driver tool chains to VFS. There are still some loose ends here but it's sufficient so we can detect GCC headers that are inside of a VFS. Added: cfe/t

Re: [clang-tools-extra] r249399 - Add a new module for the C++ Core Guidelines, and the first checker for those guidelines: cppcoreguidelines-pro-type-reinterpret-cast.

On Tue, Oct 06, 2015 at 01:31:01PM -, Aaron Ballman via cfe-commits wrote: > Log: > Add a new module for the C++ Core Guidelines, and the first checker > for those guidelines: cppcoreguidelines-pro-type-reinterpret-cast. I wonder about the generality of this. Does it really make sense to flag

Re: [PATCH] D13221: Make CompilerInvocation's use of the debug options more understandable.

dblaikie added a subscriber: dblaikie. dblaikie accepted this revision. dblaikie added a reviewer: dblaikie. This revision is now accepted and ready to land. Comment at: test/Driver/clang-g-opts.c:33 @@ -22,5 +32,3 @@ -// CHECK-WITHOUT-G-NOT: "-g" -// CHECK-WITH-G: "-g" -// CHEC

Re: [PATCH] D13510: [PATCH] Support C++ Core Guidelines copy assignment restrictions

Eugene.Zelenko added a comment. I think it'll be fine to rename check without leaving traces of misc. Same thing happened with modernize-shrink-to-fit. http://reviews.llvm.org/D13510 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://li

Re: [PATCH] D13368: [clang-tidy] add check cppcoreguidelines-pro-type-static-cast-downcast

sbenza added inline comments. Comment at: test/clang-tidy/cppcoreguidelines-pro-type-static-cast-downcast.cpp:28 @@ +27,3 @@ + + auto P0 = static_cast(new Base()); + // CHECK-MESSAGES: :[[@LINE-1]]:13: warning: do not use static_cast to downcast from a base to a derived class

r249567 - clang-format: Add include sorting capabilities to sublime, emacs and

Author: djasper Date: Wed Oct 7 12:00:20 2015 New Revision: 249567 URL: http://llvm.org/viewvc/llvm-project?rev=249567&view=rev Log: clang-format: Add include sorting capabilities to sublime, emacs and clang-format-diff.py. Modified: cfe/trunk/tools/clang-format/clang-format-diff.py cfe/

[PATCH] D13516: Fix overlapping replacements in clang-tidy.

angelgarcia created this revision. angelgarcia added reviewers: klimek, bkramer. angelgarcia added subscribers: alexfh, cfe-commits. Prevent clang-tidy from applying fixes to errors that overlap with other errors' fixes, with one exception: if one fix is completely contained inside another one,

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

spetrovic updated this revision to Diff 36763. http://reviews.llvm.org/D13351 Files: include/clang/Basic/TargetInfo.h lib/Basic/Targets.cpp lib/CodeGen/TargetInfo.cpp lib/Driver/Tools.cpp lib/Driver/Tools.h test/CodeGen/ppc-sfvarargs.c test/Driver/ppc-features.cpp Index: test/Drive

Re: [clang-tools-extra] r249399 - Add a new module for the C++ Core Guidelines, and the first checker for those guidelines: cppcoreguidelines-pro-type-reinterpret-cast.

On Wed, Oct 7, 2015 at 12:05 PM, Joerg Sonnenberger via cfe-commits wrote: > On Tue, Oct 06, 2015 at 01:31:01PM -, Aaron Ballman via cfe-commits wrote: >> Log: >> Add a new module for the C++ Core Guidelines, and the first checker >> for those guidelines: cppcoreguidelines-pro-type-reinterpret

Re: [PATCH] D12793: Three new overflow builtins with generic argument types

DavidEGrayson marked an inline comment as done. DavidEGrayson added a comment. Well, it has been another week. John, I don't have commit access, so can you commit this? Thanks! Let me know if the patch doesn't apply cleanly or any tests fail. http://reviews.llvm.org/D12793 __

Re: [PATCH] D13510: [PATCH] Support C++ Core Guidelines copy assignment restrictions

aaron.ballman added a comment. In http://reviews.llvm.org/D13510#261825, @Eugene.Zelenko wrote: > I think it'll be fine to rename check without leaving traces of misc. Same > thing happened with modernize-shrink-to-fit. I think the difference here is that many C++ Core Guideline checks are...

Re: [PATCH] D13221: Make CompilerInvocation's use of the debug options more understandable.

dougk marked an inline comment as done. dougk added a comment. http://reviews.llvm.org/D13221 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D13523: ASTMatchers: Keep AllCallbacks in a set instead of a vector

djasper created this revision. djasper added a reviewer: klimek. djasper added a subscriber: cfe-commits. Herald added a subscriber: klimek. AllCallbacks is currently only used to call onStartOfTranslationUnit and onEndOfTranslationUnit on them. In this (and any other scenario I can come up with

[PATCH] D13525: [CodeGen] Attach function attributes to functions created in CGBlocks.cpp.

ahatanak created this revision. ahatanak added reviewers: dexonsmith, echristo. ahatanak added a subscriber: cfe-commits. This patch makes changes to attach function attributes to the following functions created in CGBlocks.cpp: __copy_helper_block_ __destroy_helper_block_ __Block_byref_object_c

Re: r248949 - Don't inherit availability information when implementing a protocol requirement.

Hi Doug, On Wed, Sep 30, 2015 at 2:27 PM, Douglas Gregor via cfe-commits wrote: > Author: dgregor > Date: Wed Sep 30 16:27:42 2015 > New Revision: 248949 > > URL: http://llvm.org/viewvc/llvm-project?rev=248949&view=rev > Log: > Don't inherit availability information when implementing a protocol

Re: [PATCH] D11328: [clang-tidy] new "throw-by-value-catch-by-reference" check for clang-tidy

aaron.ballman added a comment. This patch is looking much closer! Thank you for continuing to work on it. I found several mechanical changes that need to be made, like style, comments, formatting, isa<> followed by cast<>, etc. There is one logic issue regarding materialized temporaries that I

Re: [PATCH] D13440: [clang-tidy] Python script for easy check rename

mgehre added a subscriber: mgehre. Comment at: clang-tidy/rename_check.py:39 @@ +38,3 @@ + if len(sys.argv) != 4: +print('Usage: rename_check.py , e.g.\n') +print('rename_check.py misc awesome-functions new-awesome-function\n') Should be ``` Usage: renam

Re: [PATCH] D13440: [clang-tidy] Python script for easy check rename

alexfh added inline comments. Comment at: clang-tidy/rename_check.py:3 @@ +2,3 @@ +# +#===- add_new_check.py - clang-tidy check generator --*- python -*--===# +# Please update the script name and description in the comment. Comment at: c

Re: [PATCH] D13523: ASTMatchers: Keep AllCallbacks in a set instead of a vector

klimek accepted this revision. klimek added a comment. This revision is now accepted and ready to land. That makes a lot of sense. LG http://reviews.llvm.org/D13523 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bi

[libcxx] r249593 - While researching LWG#2244, I noticed we weren't testing that eofbit was being cleared. Now we are

Author: marshall Date: Wed Oct 7 14:41:24 2015 New Revision: 249593 URL: http://llvm.org/viewvc/llvm-project?rev=249593&view=rev Log: While researching LWG#2244, I noticed we weren't testing that eofbit was being cleared. Now we are Modified: libcxx/trunk/test/std/input.output/iostream.for

[libcxx] r249595 - Mark 2244 as 'Patch Ready', 2477 and 2487 as 'Complete'

Author: marshall Date: Wed Oct 7 14:45:14 2015 New Revision: 249595 URL: http://llvm.org/viewvc/llvm-project?rev=249595&view=rev Log: Mark 2244 as 'Patch Ready', 2477 and 2487 as 'Complete' Modified: libcxx/trunk/www/kona.html Modified: libcxx/trunk/www/kona.html URL: http://llvm.org/viewv

Re: [PATCH] D13523: ASTMatchers: Keep AllCallbacks in a set instead of a vector

djasper closed this revision. djasper added a comment. Submitted as r249598. http://reviews.llvm.org/D13523 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r249598 - ASTMatchers: Keep AllCallbacks in a set instead of a vector

Author: djasper Date: Wed Oct 7 14:56:12 2015 New Revision: 249598 URL: http://llvm.org/viewvc/llvm-project?rev=249598&view=rev Log: ASTMatchers: Keep AllCallbacks in a set instead of a vector AllCallbacks is currently only used to call onStartOfTranslationUnit and onEndOfTranslationUnit on them

Re: r249556 - [VFS] Port driver tool chains to VFS.

This change breaks cmake build: undefined reference to `clang::DiagnosticIDs::DiagnosticIDs()' /home/jvesely/llvm/tools/clang/unittests/Driver/ToolChainTest.cpp:33: undefined reference to `clang::DiagnosticsEngine::DiagnosticsEngine(llvm::IntrusiveRefCntPtr const&, clang::DiagnosticOptions*, clang:

Re: [PATCH] D13510: [PATCH] Support C++ Core Guidelines copy assignment restrictions

mgehre added a comment. The user will see [cppcoreguidelines-c-copy-assignment-signature] in warnings, but there is no docs/clang-tidy/checks/cppcoreguidelines-c-copy-assignment-signature.rst. The user won't know that he needs to look for misc-assign-operator-signature. Maybe create a link to

Re: [PATCH] D13304: Avoid inlining in throw statement

rsmith added a subscriber: rsmith. rsmith added a comment. This seems like something that's much better handled by the inliner. We should presumably treat all the code that leads up to the throw as being cold, not just the invocation of the throw-expression itself, and it seems like it should b

Re: [PATCH] D13510: [PATCH] Support C++ Core Guidelines copy assignment restrictions

sbenza added a comment. In http://reviews.llvm.org/D13510#261925, @aaron.ballman wrote: > In http://reviews.llvm.org/D13510#261825, @Eugene.Zelenko wrote: > > > I think it'll be fine to rename check without leaving traces of misc. Same > > thing happened with modernize-shrink-to-fit. > > > I thi

r249608 - Fix a shared CMake build by linking with libclangBasic.

Author: d0k Date: Wed Oct 7 15:19:25 2015 New Revision: 249608 URL: http://llvm.org/viewvc/llvm-project?rev=249608&view=rev Log: Fix a shared CMake build by linking with libclangBasic. Patch by Jan Vesely! Modified: cfe/trunk/unittests/Driver/CMakeLists.txt Modified: cfe/trunk/unittests/Dr

Re: r249556 - [VFS] Port driver tool chains to VFS.

On Wed, Oct 7, 2015 at 10:09 PM, Jan Vesely wrote: > > This change breaks cmake build: > undefined reference to `clang::DiagnosticIDs::DiagnosticIDs()' > /home/jvesely/llvm/tools/clang/unittests/Driver/ToolChainTest.cpp:33: > undefined reference to > `clang::DiagnosticsEngine::DiagnosticsEngine(

Re: [PATCH] D13510: [PATCH] Support C++ Core Guidelines copy assignment restrictions

On Wed, Oct 7, 2015 at 4:18 PM, Samuel Benzaquen wrote: > sbenza added a comment. > > In http://reviews.llvm.org/D13510#261925, @aaron.ballman wrote: > >> In http://reviews.llvm.org/D13510#261825, @Eugene.Zelenko wrote: >> >> > I think it'll be fine to rename check without leaving traces of misc.

Re: [PATCH] D13482: Revised Initial patch for PS4 toolchain

kromanova added a comment. Hi Pierre, I noticed the same issue. When I downloaded a patch from http://reviews.llvm.org/D11279, I had to manually add .keep files to make tests to pass. It's a Phabricator problem. The diff file that I uploaded to Phabricator contains .keep files, however the di

[clang-tools-extra] r249612 - Fixing links and reformatting code; NFC.

Author: aaronballman Date: Wed Oct 7 15:33:36 2015 New Revision: 249612 URL: http://llvm.org/viewvc/llvm-project?rev=249612&view=rev Log: Fixing links and reformatting code; NFC. Patch by Marek Kurdej! Modified: clang-tools-extra/trunk/clang-tidy/cppcoreguidelines/ProTypeConstCastCheck.cpp

Re: [PATCH] D13525: [CodeGen] Attach function attributes to functions created in CGBlocks.cpp.

> On 2015-Oct-07, at 11:35, Akira Hatanaka wrote: > > ahatanak created this revision. > ahatanak added reviewers: dexonsmith, echristo. > ahatanak added a subscriber: cfe-commits. > > This patch makes changes to attach function attributes to the following > functions created in CGBlocks.cpp: >

Re: [PATCH] D13399: [CMake] Bug 14109 - CMake build for compiler-rt should use just-built clang

beanz added a comment. Ping? http://reviews.llvm.org/D13399 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D11328: [clang-tidy] new "throw-by-value-catch-by-reference" check for clang-tidy

randomcppprogrammer updated this revision to Diff 36795. randomcppprogrammer marked 11 inline comments as done. randomcppprogrammer added a comment. Incorporated feedback from Aaron Ballmann main changes: - replaced isa followed by cast with dyn_cast - reworked comments - fixed typo in diagnosis

r249616 - [WinEH] Don't use lifetime markers for MS catch parameters

Author: rnk Date: Wed Oct 7 16:03:41 2015 New Revision: 249616 URL: http://llvm.org/viewvc/llvm-project?rev=249616&view=rev Log: [WinEH] Don't use lifetime markers for MS catch parameters We don't have a good place to put them. Our previous spot was causing us to optimize loads from the exceptio

Re: [PATCH] D13099: [Analyzer] Don’t invalidate CXXThis when conservatively evaluating const methods (PR 21606)

zaks.anna added a comment. > Again I'm not sure this is worth checking for, based on the assumption that a > reasonable const method won't modify the relevant object. What do people > think? These are heuristics. I think dealing with this case should not be a pre-requisite for the patch. We c

Fwd: [PATCH] D11328: [clang-tidy] new "throw-by-value-catch-by-reference" check for clang-tidy

Hi,I incorporated most of your changes. There are 2 open issues- on one location I could not follow your advice, the compiler refused to compile the code. I chose a different approach and hope you like it.- the second thing is this MaterializeTemporary advice that you gave. I don’t fully understand

Re: [PATCH] D13192: Fix incorrect parsing of arguments for nested functions

mclow.lists accepted this revision. mclow.lists added a comment. This revision is now accepted and ready to land. This looks fine to me. Thanks for the patch (and the test case). http://reviews.llvm.org/D13192 ___ cfe-commits mailing list cfe-commi

Re: [PATCH] D12747: Implement [depr.c.headers]

Marshall: ping, does the below satisfy your concerns about the direction here? On Wed, Sep 16, 2015 at 2:04 PM, Richard Smith wrote: > On Mon, Sep 14, 2015 at 7:07 AM, Marshall Clow > wrote: > >> mclow.lists added a comment. >> >> I have two concerns about this patch (w/o commenting on the actu

Re: [libcxx] r249475 - Remove unnecessary inline functions capturing the contents of C library macros.

On Tue, Oct 6, 2015 at 3:31 PM, Jonathan Roelofs wrote: > On 10/6/15 4:03 PM, Richard Smith via cfe-commits wrote: > >> Author: rsmith >> Date: Tue Oct 6 17:03:22 2015 >> New Revision: 249475 >> >> URL: http://llvm.org/viewvc/llvm-project?rev=249475&view=rev >> Log: >> Remove unnecessary inline

Re: [PATCH] D11328: [clang-tidy] new "throw-by-value-catch-by-reference" check for clang-tidy

aaron.ballman added a comment. On Wed, Oct 7, 2015 at 5:08 PM, Tobias Langner wrote: > Hi, > > I incorporated most of your changes. There are 2 open issues Thank you for your work on this! There are a few minor nitpicks still, but is definitely moving forward nicely. > - on one location

Re: [PATCH] D13357: [Concepts] Add diagnostic; specializations of variable and function concepts

rsmith added inline comments. Comment at: lib/Sema/SemaDecl.cpp:5902-5915 @@ -5901,1 +5901,16 @@ + + // C++ Concepts TS [dcl.spec.concept]p7: A program shall not declare [...] + // an explicit specialization, or a partial specialization of a concept + // definitio

Re: [PATCH] D13357: [Concepts] Add diagnostic; specializations of variable and function concepts

On Wed, Oct 7, 2015 at 2:59 PM, Richard Smith via cfe-commits < cfe-commits@lists.llvm.org> wrote: > rsmith added inline comments. > > > Comment at: lib/Sema/SemaDecl.cpp:5902-5915 > @@ -5901,1 +5901,16 @@ > + > + // C++ Concepts TS [dcl.spec.concept]p7: A program shall not >

Re: [PATCH] D13440: [clang-tidy] Python script for easy check rename

ClockMan marked 5 inline comments as done. Comment at: clang-tidy/rename_check.py:53 @@ +52,3 @@ + return newFileName + +def getListOfFiles(clang_tidy_path): Not sure what you mean... Repository: rL LLVM http://reviews.llvm.org/D13440

Re: [PATCH] D13440: [clang-tidy] Python script for easy check rename

ClockMan updated this revision to Diff 36801. ClockMan added a comment. Corrected review issues. Repository: rL LLVM http://reviews.llvm.org/D13440 Files: clang-tidy/rename_check.py Index: clang-tidy/rename_check.py === --- /

Re: [PATCH] D13525: [CodeGen] Attach function attributes to functions created in CGBlocks.cpp.

On Wed, Oct 7, 2015 at 1:48 PM, Duncan P. N. Exon Smith < dexonsm...@apple.com> wrote: > > > On 2015-Oct-07, at 11:35, Akira Hatanaka wrote: > > > > ahatanak created this revision. > > ahatanak added reviewers: dexonsmith, echristo. > > ahatanak added a subscriber: cfe-commits. > > > > This patch

Re: [PATCH] D13304: Avoid inlining in throw statement

junbuml added a comment. Thanks Richard for the comment. Initially, I intended to implement this in inliner by checking if a callsite is in exception handling regions. However, I decided not to implement this in inliner because this kind of check should be performed for all callsites if we imp

Re: [PATCH] D13344: Keep the IfStmt node even if the condition is invalid

rsmith added inline comments. Comment at: lib/Sema/SemaStmt.cpp:508-509 @@ -513,6 +507,4 @@ - DiagnoseUnusedExprResult(elseStmt); - return new (Context) IfStmt(Context, IfLoc, ConditionVar, ConditionExpr, thenStmt, ElseLoc, elseStmt); } --

Re: [PATCH] D12901: [Static Analyzer] Assertion "System is over constrained" after truncating 64 bits integers to 32 bits. (PR25078)

zaks.anna added a comment. I agree with Gabor. We should investigate how we can model the overflow on a cast correctly. http://reviews.llvm.org/D12901 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/lis

Re: [PATCH] D13304: Avoid inlining in throw statement

On Wed, Oct 7, 2015 at 3:10 PM, Jun Bum Lim via cfe-commits < cfe-commits@lists.llvm.org> wrote: > junbuml added a comment. > > Thanks Richard for the comment. > > Initially, I intended to implement this in inliner by checking if a > callsite is in exception handling regions. However, I decided no

Re: [PATCH] D13453: Always generate cmake config files

chapuni added a reviewer: jordan_rose. chapuni added a comment. LGTM, but I haven't tested. I'll wait for 3rd opinion for several hours. :) I thought it may be moved to an arbitrary subdirectory. http://reviews.llvm.org/D13453 ___ cfe-commits maili

Re: [PATCH] D13453: Always generate cmake config files

jordan_rose resigned from this revision. jordan_rose edited reviewers, added: beanz; removed: jordan_rose. jordan_rose added a comment. I just get things to build with CMake, and I'm not involved with Clang so much these days. This is more about controlling what gets installed, which should be c

Re: [PATCH] D13453: Always generate cmake config files

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

Re: [PATCH] D13453: Always generate cmake config files

chapuni added a comment. I am testing this with cmake-2.8.12. I remember some issues would be there. http://reviews.llvm.org/D13453 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D9888: [OPENMP] Driver support for OpenMP offloading

tra added a comment. In http://reviews.llvm.org/D9888#257904, @sfantao wrote: > This diff refactors the original patch and is rebased on top of the latests > offloading changes inserted for CUDA. > > Here I don't touch the CUDA support. I tried, however, to have the > implementation modular eno

r249639 - Make clang_Cursor_getMangling not mangle if the declaration isn't mangled

Author: ehsan Date: Wed Oct 7 19:01:20 2015 New Revision: 249639 URL: http://llvm.org/viewvc/llvm-project?rev=249639&view=rev Log: Make clang_Cursor_getMangling not mangle if the declaration isn't mangled Right now clang_Cursor_getMangling will attempt to mangle any declaration, even if the decl

  1   2   >