r272253 - clang/test/CodeGenCXX/debug-info-method.cpp: Tweak for thiscall, for targeting Win32 x86.

Author: chapuni Date: Thu Jun 9 05:06:13 2016 New Revision: 272253 URL: http://llvm.org/viewvc/llvm-project?rev=272253=rev Log: clang/test/CodeGenCXX/debug-info-method.cpp: Tweak for thiscall, for targeting Win32 x86. Modified: cfe/trunk/test/CodeGenCXX/debug-info-method.cpp Modified:

Re: [PATCH] D18081: Make sizeof and alignof a CXCursor_UnaryExpr

ogoffart added a comment. Ping 2. http://reviews.llvm.org/D18081 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D21179: Add mrrc/mrrc2 intrinsics and update existing mcrr/mcrr2 intrinsics to accept a single uint64 type instead of 2 uint32 types

rs created this revision. rs added reviewers: t.p.northover, rengolin. rs added a subscriber: cfe-commits. Patch adds support for the intrinsics mrrc/mrrc2 and updates the existing mcrr/mcrr2 intrinsic definition to accept a single uint64 type as the input value instead of 2 uint32's as I think

Re: [PATCH] D21179: Add mrrc/mrrc2 intrinsics and update existing mcrr/mcrr2 intrinsics to accept a single uint64 type instead of 2 uint32 types

rs added a comment. LLVM part of the patch is here http://reviews.llvm.org/D21178 http://reviews.llvm.org/D21179 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D21173: [X86] _MM_ALIGN16 attribute support for non-windows targets

zvi created this revision. zvi added reviewers: aaboud, mkuper, echristo, cfe-commits. zvi set the repository for this revision to rL LLVM. zvi added a project: clang-c. Herald added a subscriber: mehdi_amini. This patch adds support for the _MM_ALIGN16 attribute on non-windows targets. This

Re: r271918 - Add a release note about the --build-id change.

On Mon, Jun 6, 2016 at 11:23 AM, Rafael Espindola via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Author: rafael > Date: Mon Jun 6 13:23:11 2016 > New Revision: 271918 > > URL: http://llvm.org/viewvc/llvm-project?rev=271918=rev > Log: > Add a release note about the --build-id change. > >

Re: [PATCH] D19763: Functions declared in a scope should not hide previous declaration in earlier scopes

ogoffart added a comment. Ping. This would be required for http://reviews.llvm.org/D19764 http://reviews.llvm.org/D19763 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D21176: Mark invalid RecordDecls as completed.

erikjv created this revision. erikjv added a reviewer: rsmith. erikjv added a subscriber: cfe-commits. Sema::ActOnTag creates TagDecls for records. However, if those record declarations are invalid, and the parser is in C++ mode, it would silently drop the TagDecl (and leave it as

Re: [PATCH] D21162: [CUDA] Implement __shfl* intrinsics in clang headers.

jholewinski added a comment. Looks reasonable to me. http://reviews.llvm.org/D21162 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D20821: Fix a few issues while skipping function bodies

ogoffart added a comment. The idea is that when we see a ") {" or "} {" in the ctor-initializers, (optionally with "..."), it is necessarily the start of the body. Unless there might be lambda expressions within a template aregument, as in: A::A() : Base<[](){return 42; }()>(){} But

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

On Thu, Jun 9, 2016 at 7:26 AM, David Majnemer via cfe-commits wrote: > 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

[PATCH] D21175: [include-fixer] do not index friend function declaration.

ioeric created this revision. ioeric added a reviewer: bkramer. ioeric added a subscriber: cfe-commits. we want to exclude friend declaration, but the `DeclContext` of a friend function declaration is not the class in which it is declared, so we need to explicitly check if the parent is a

Re: [PATCH] D21173: [X86] _MM_ALIGN16 attribute support for non-windows targets

On Thursday, June 9, 2016, Zvi Rackover via cfe-commits < cfe-commits@lists.llvm.org> wrote: > zvi created this revision. > zvi added reviewers: aaboud, mkuper, echristo, cfe-commits. > zvi set the repository for this revision to rL LLVM. > zvi added a project: clang-c. > Herald added a

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

It would mean that the instantiation of the class template gained a field which should be impossible. On Thursday, June 9, 2016, Kim Gräsman wrote: > On Thu, Jun 9, 2016 at 7:26 AM, David Majnemer via cfe-commits > > wrote: > >

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

teemperor updated this revision to Diff 60176. teemperor added a comment. Reduced executable size bloat. Made postorder and preorder mutually exclusive and postorder also uses the normal Visit* methods for callbacks. http://reviews.llvm.org/D20382 Files:

Re: [PATCH] D18081: Make sizeof and alignof a CXCursor_UnaryExpr

bkramer accepted this revision. bkramer added a comment. This revision is now accepted and ready to land. lg http://reviews.llvm.org/D18081 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

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

teemperor added a comment. Agreed. The new patch is now reusing the Visit methods so that the executable size stays the same. The downside is that you can no longer create a Visitor that is both post- and preorder traversing, but I don't think there is any major use case for that.

[libcxx] r272263 - Two more issues w/patches

Author: marshall Date: Thu Jun 9 09:50:38 2016 New Revision: 272263 URL: http://llvm.org/viewvc/llvm-project?rev=272263=rev Log: Two more issues w/patches Modified: libcxx/trunk/www/upcoming_meeting.html Modified: libcxx/trunk/www/upcoming_meeting.html URL:

[PATCH] D21185: [clang-tidy] Add performance-emplace-into-containers

vsk created this revision. vsk added reviewers: aaron.ballman, alexfh. vsk added a subscriber: cfe-commits. Introduce a check which suggests when it might be helpful to use "emplace" methods. The initial version only works with std::vector and push_back (e.g `V.push_back(T(1, 2))` ->

Re: r272253 - clang/test/CodeGenCXX/debug-info-method.cpp: Tweak for thiscall, for targeting Win32 x86.

Reid - is this intended fallout? (seems plausible, but just checking) Is MinGW a good analogy for any of this work? (does it produce DWARF? Does it use the Windows ABI? Does it emit Calling Convention attributes?) On Thu, Jun 9, 2016 at 3:06 AM, NAKAMURA Takumi via cfe-commits <

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

On Thu, Jun 9, 2016 at 4:51 PM, David Majnemer wrote: > It would mean that the instantiation of the class template gained a field > which should be impossible. OK, so assert(Lookup.size() > 0) always holds? Makes sense, thanks. - Kim

Re: [A fix related to closing C++ header file descriptors on windows]

Hi Manuel, I had forgotten this point and I see your mail only now... Thanks for your interest. I did not evaluate this issue with a newer release of clang (issue seen in 3.6.2) However, it seems that other people still have this kind of issue for several months.

Re: [PATCH] D21162: [CUDA] Implement __shfl* intrinsics in clang headers.

jlebar added a comment. (Art, I would appreciate a second set of eyes on this one, as the last time I did this -- with ldg -- I messed up pretty badly.) http://reviews.llvm.org/D21162 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

Re: [clang-tools-extra] r272188 - clang-rename: implement renaming of classes inside dynamic_cast

Hi Miklos, This revision broke tests on one of builders: http://lab.llvm.org:8011/builders/llvm-clang-lld-x86_64-scei-ps4-ubuntu-fast/builds/14332 Please have a look at this? Thanks Galina On Wed, Jun 8, 2016 at 11:38 AM, Miklos Vajna via cfe-commits < cfe-commits@lists.llvm.org> wrote: >

Re: [PATCH] D21162: [CUDA] Implement __shfl* intrinsics in clang headers.

jlebar added a comment. Thank you for the reviews, Justin! http://reviews.llvm.org/D21162 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D21187: Allow use of lambda expressions in array bounds

ahatanak created this revision. ahatanak added a subscriber: cfe-commits. clang currently errors out when a lambda expression is used to compute the size of an array even though clang supports variable-length arrays as a C99 extension. For example, $ cat vla1.cpp ``` int foo3(); struct S1 {

Re: [clang-tools-extra] r272188 - clang-rename: implement renaming of classes inside dynamic_cast

Hi Galina, On Thu, Jun 09, 2016 at 10:15:27AM -0700, Galina Kistanova wrote: > This revision broke tests on one of builders: > http://lab.llvm.org:8011/builders/llvm-clang-lld-x86_64-scei-ps4-ubuntu-fast/builds/14332 Please accept if it's

Re: [PATCH] D21185: [clang-tidy] Add performance-emplace-into-containers

vsk added a reviewer: flx. vsk updated this revision to Diff 60194. vsk added a comment. - Fix handling of push_back(X(/* comment */)). - Don't try to emit a warning when the callee comes from a macro expansion. - Get rid of a weird/wrong comment about checking for "default constructors". As a

r272280 - Revert "[CMake] Fix an issue building out-of-tree introduced in r272200"

Author: cbieneman Date: Thu Jun 9 12:24:16 2016 New Revision: 272280 URL: http://llvm.org/viewvc/llvm-project?rev=272280=rev Log: Revert "[CMake] Fix an issue building out-of-tree introduced in r272200" This reverts r272275. This actually wasn't the right way to fix the problem. The correct

[PATCH] D21186: clang-rename: mark the DynamicCastExpr test unsupported on ps4

vmiklos created this revision. vmiklos added a reviewer: klimek. vmiklos added a subscriber: cfe-commits. It has no RTTI, so the test would always fail in a non-interesting way. http://reviews.llvm.org/D21186 Files: test/clang-rename/DynamicCastExpr.cpp Index:

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

vmiklos added a comment. Possible solution posted as http://reviews.llvm.org/D21186. Repository: rL LLVM http://reviews.llvm.org/D21120 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

r272275 - [CMake] Fix an issue building out-of-tree introduced in r272200

Author: cbieneman Date: Thu Jun 9 11:21:10 2016 New Revision: 272275 URL: http://llvm.org/viewvc/llvm-project?rev=272275=rev Log: [CMake] Fix an issue building out-of-tree introduced in r272200 The out-of-tree build needs to read LLVM_TOOLS_INSTALL_DIR out of TOOLS_BINARY_DIR because

Re: [PATCH] D20389: NVPTX: Add supported CL features

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

r272299 - [CUDA] Implement __shfl* intrinsics in clang headers.

Author: jlebar Date: Thu Jun 9 15:04:57 2016 New Revision: 272299 URL: http://llvm.org/viewvc/llvm-project?rev=272299=rev Log: [CUDA] Implement __shfl* intrinsics in clang headers. Summary: Clang changes to make use of the LLVM intrinsics added in D21160. Reviewers: tra Subscribers:

Re: [PATCH] D20933: Preallocate ExplodedNode hash table

bcraig updated this revision to Diff 60177. bcraig added a comment. Capping the pre-reserve space http://reviews.llvm.org/D20933 Files: include/clang/StaticAnalyzer/Core/PathSensitive/ExplodedGraph.h lib/StaticAnalyzer/Core/CoreEngine.cpp Index: lib/StaticAnalyzer/Core/CoreEngine.cpp

Re: [PATCH] D21162: [CUDA] Implement __shfl* intrinsics in clang headers.

This revision was automatically updated to reflect the committed changes. Closed by commit rL272299: [CUDA] Implement __shfl* intrinsics in clang headers. (authored by jlebar). Changed prior to commit: http://reviews.llvm.org/D21162?vs=60223=60230#toc Repository: rL LLVM

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

vsk added a subscriber: vsk. vsk added a comment. @Eugene.Zelenko thanks for pointing this out, I had totally missed this patch! Once we get this reviewed I'm willing to abandon my version. Some comments inline -- Comment at: clang-tidy/modernize/UseEmplaceCheck.cpp:26 @@

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

eric_niebler added a comment. > Before this goes in again, I want to double check that this doesn't affect > compile time on darwin + frameworks. @bruno, you're not likely to find a difference for darwin + frameworks since the frameworks headers like `Cocoa/Cocoa.h` don't exist on-disk with

Re: [PATCH] D21054: CodeGen: Update Clang to use the new type metadata.

eugenis accepted this revision. eugenis added a reviewer: eugenis. eugenis added a comment. This revision is now accepted and ready to land. LGTM http://reviews.llvm.org/D21054 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

Re: [PATCH] D20933: Preallocate ExplodedNode hash table

bcraig added a comment. I got better valgrind numbers (symbols are important). Before: 106,131,538 After: 106,657,666 Diff: 526,128 larger. Note that this is sampled peaks for heap usage. They may not be accurate. Regardless, the peak usage increased by less than .5%.

Re: [PATCH] D20490: [Parser] Fix a crash on invalid where a delayed TypoExpr was corrected twice

rsmith added inline comments. Comment at: lib/Parse/ParseExpr.cpp:450-452 @@ -449,1 +449,5 @@ + +// In this case, ActOnBinOp performed the CorrectDelayedTyposInExpr check. +if (!getLangOpts().CPlusPlus) + continue; } else { The

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

rsmith added inline comments. Comment at: lib/Basic/FileManager.cpp:389 @@ -383,2 +388,3 @@ UFE->File.reset(); + UFE->IsVirtual = true; return UFE; Yes. The `IsValid` flag is just supposed to mean that this file has actually been added to the

r272310 - Revert "[Temporary] Add an ExprWithCleanups for each C++ MaterializeTemporaryExpr."

Author: timshen Date: Thu Jun 9 16:13:39 2016 New Revision: 272310 URL: http://llvm.org/viewvc/llvm-project?rev=272310=rev Log: Revert "[Temporary] Add an ExprWithCleanups for each C++ MaterializeTemporaryExpr." This reverts r272296, since there are clang-tidy failures that appear to be caused

Re: [PATCH] D20821: Fix a few issues while skipping function bodies

rsmith added a comment. Please call `Parser::ConsumeAndStoreFunctionPrologue` rather than trying to reinvent it. You're still getting a number of cases wrong that it handles properly. You also need to handle the case where the code completion token appears within the constructor

[PATCH] D21198: Add a RenderScript language type

pirama created this revision. pirama added a reviewer: rsmith. pirama added subscribers: srhines, cfe-commits. Add RenderScript language type and associate it with ".rs" extensions. Test that the driver passes "-x renderscript" to the frontend for ".rs" files. (Also add '.rs' to the list of

Re: [PATCH] D21187: Allow use of lambda expressions in array bounds

rsmith added a subscriber: rsmith. Comment at: lib/Sema/SemaExpr.cpp:12825 @@ -12824,1 +12824,3 @@ + // BlockContext. + } else if (!Rec.IsArrayBound) { // C++1y [expr.const]p2: This isn't correct; you still need to produce the diagnostic even

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

Anastasia added inline comments. Comment at: test/Headers/opencl-c-header.cl:70 @@ +69,3 @@ +// RUN: %clang_cc1 -triple spir-unknown-unknown -emit-llvm -o - -finclude-default-header -fmodules -fimplicit-module-maps -fmodules-cache-path=%t %s | FileCheck %s +// RUN: %clang_cc1

Re: [PATCH] D20347: Add support to clang-cl driver for /GS switch

hans added a comment. Is this waiting for anything more now that http://reviews.llvm.org/D20346 has landed? http://reviews.llvm.org/D20347 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

Fwd: Java Interface for clang

Hello everybody, I want to use clang and its facilities to parse c and c++ . but I need to use it in java. In my searches , saw jClang or Clang bindings for java, checking the date of pages returned to me was before 2015. so I know what is the status of jclang or is there any solution to use

Re: [PATCH] D20347: Add support to clang-cl driver for /GS switch

thakis added a comment. probably at least the "the XOR with RSP/EBP/ESP" bit still (and maybe EH function upgrades instead of bailing) Comment at: lib/Driver/Tools.cpp:9990 @@ +9989,3 @@ + /*default=*/false)) +CmdArgs.push_back("/GS-"); +

Re: [PATCH] D21185: [clang-tidy] Add performance-emplace-into-containers

Eugene.Zelenko added a subscriber: Eugene.Zelenko. Eugene.Zelenko added a comment. See also http://reviews.llvm.org/D20964. I think modernize is better place for such check. http://reviews.llvm.org/D21185 ___ cfe-commits mailing list

Re: [PATCH] D21185: [clang-tidy] Add performance-emplace-into-containers

vsk updated this revision to Diff 60206. vsk added a comment. - Fix the diagnostic message. Suggested by Erik Pilkington. http://reviews.llvm.org/D21185 Files: clang-tidy/performance/CMakeLists.txt clang-tidy/performance/EmplaceCheck.cpp clang-tidy/performance/EmplaceCheck.h

RE: [PATCH] D21173: [X86] _MM_ALIGN16 attribute support for non-windows targets

Thanks the suggestion, David. I did not realize __attribute__ is supported by Windows targets. Zvi From: David Majnemer [mailto:david.majne...@gmail.com] Sent: Thursday, June 09, 2016 17:48 To: reviews+d21173+public+9a6e31402e430...@reviews.llvm.org; Rackover, Zvi Cc:

Re: [PATCH] D21173: [X86] _MM_ALIGN16 attribute support for non-windows targets

zvi updated this revision to Diff 60217. zvi added a comment. Following David Majnemer's suggestion on cfe-commits: I would just use the __attribute__ spelling, no need for two definitions Repository: rL LLVM http://reviews.llvm.org/D21173 Files: lib/Headers/xmmintrin.h

Re: [PATCH] D20328: [libcxx] Externally threaded libc++ variant

rmaprath added a comment. @mclow.lists, @EricWF: Gentle (and shameless) ping! http://reviews.llvm.org/D20328 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D19854: Define Contiki OS toolchain

eugenis added a comment. This needs a driver test. http://reviews.llvm.org/D19854 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D20490: [Parser] Fix a crash on invalid where a delayed TypoExpr was corrected twice

erik.pilkington added a comment. Pong!! http://reviews.llvm.org/D20490 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Java Interface for clang

Hello everybody, I want to use clang and its facilities to parse c and c++ . but I need to use it in java. In my searches , saw jClang or Clang bindings for java, checking the date of pages returned to me was before 2015. so I know what is the status of jclang or is there any solution to use

Re: [PATCH] D20132: [libclang] Add clang_getAllSkippedRanges function

cameron314 added a comment. Can someone have a look at this now that there's a test? http://reviews.llvm.org/D20132 ___ 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. There are alternative implementation in http://reviews.llvm.org/D21185. Will be good idea to how one which take the best from both :-) http://reviews.llvm.org/D20964 ___ cfe-commits mailing list

Re: [PATCH] D21162: [CUDA] Implement __shfl* intrinsics in clang headers.

jlebar added inline comments. Comment at: lib/Headers/__clang_cuda_intrinsics.h:77-80 @@ +76,6 @@ +_Static_assert(sizeof(__tmp) == sizeof(__in)); \ +memcpy(&__tmp, &__in, sizeof(__in)); \ +__tmp =

Re: [PATCH] D20498: [Temporary] Add an ExprWithCleanups for each C++ MaterializeTemporaryExpr

This revision was automatically updated to reflect the committed changes. Closed by commit rL272296: [Temporary] Add an ExprWithCleanups for each C++ MaterializeTemporaryExpr. (authored by timshen). Changed prior to commit: http://reviews.llvm.org/D20498?vs=58694=60227#toc Repository: rL

r272296 - [Temporary] Add an ExprWithCleanups for each C++ MaterializeTemporaryExpr.

Author: timshen Date: Thu Jun 9 14:54:46 2016 New Revision: 272296 URL: http://llvm.org/viewvc/llvm-project?rev=272296=rev Log: [Temporary] Add an ExprWithCleanups for each C++ MaterializeTemporaryExpr. These ExprWithCleanups are added for holding a RunCleanupsScope not for destructor calls;

Re: [PATCH] D20389: NVPTX: Add supported CL features

jvesely updated this revision to Diff 60226. jvesely added a comment. Test all known extensions against expected nvptx outcome (add negative tests) Repository: rL LLVM http://reviews.llvm.org/D20389 Files: lib/Basic/Targets.cpp test/Misc/nvptx.languageOptsOpenCL.cl Index:

[libcxx] r272288 - Make the comparison objects that we pass in for various tests look more like actual comparison objects. No functional change.

Author: marshall Date: Thu Jun 9 13:34:38 2016 New Revision: 272288 URL: http://llvm.org/viewvc/llvm-project?rev=272288=rev Log: Make the comparison objects that we pass in for various tests look more like actual comparison objects. No functional change. Modified:

Re: [PATCH] D21113: Add support for case-insensitive header lookup

hans updated this revision to Diff 60220. hans added a comment. Adding the version that caches directory contents. This has the problem the it doesn't play with modules, the rewriter, and possibly others, because they write to the file system without any way for the vfs to find out. I've tried

Re: [PATCH] D20933: Preallocate ExplodedNode hash table

zaks.anna added a comment. > If the underlying allocator that does a poor job at reusing freed memory, > then trivial > functions will use about 1 MB more than before, then free the memory > immediately. You could probably flag some of those functions, especially the ones that do not

Re: [PATCH] D21162: [CUDA] Implement __shfl* intrinsics in clang headers.

jlebar updated this revision to Diff 60223. jlebar marked 2 inline comments as done. jlebar added a comment. Update after tra's review. http://reviews.llvm.org/D21162 Files: include/clang/Basic/BuiltinsNVPTX.def lib/Headers/__clang_cuda_intrinsics.h

Re: [PATCH] D21162: [CUDA] Implement __shfl* intrinsics in clang headers.

tra added inline comments. Comment at: lib/Headers/__clang_cuda_intrinsics.h:77-80 @@ +76,6 @@ +_Static_assert(sizeof(__tmp) == sizeof(__in)); \ +memcpy(&__tmp, &__in, sizeof(__in)); \ +__tmp =

Re: [PATCH] D20933: Preallocate ExplodedNode hash table

zaks.anna added a comment. Ah, right, please, add a comment explaining what we are doing and why. http://reviews.llvm.org/D20933 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D21204: clang-format: [JS] post-fix non-null assertion operator.

mprobst created this revision. mprobst added a reviewer: djasper. mprobst added a subscriber: cfe-commits. Herald added a subscriber: klimek. Do not insert whitespace preceding the "!" postfix operator. This is an incomplete fix, but should cover common usage. http://reviews.llvm.org/D21204

Re: [PATCH] D21198: RenderScript support in the Frontend

pirama added a comment. Oops, this update merged changes from http://reviews.llvm.org/D21199 as well. Let me clean up and upload a new patch. http://reviews.llvm.org/D21198 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

r272323 - [CMake] Version is aways greater than 3

Author: cbieneman Date: Thu Jun 9 17:38:40 2016 New Revision: 272323 URL: http://llvm.org/viewvc/llvm-project?rev=272323=rev Log: [CMake] Version is aways greater than 3 We don't need any checks for this code anymore. Since CMake version is always greater than 3 we can always generate the

Re: [PATCH] D21206: clang-format: [JS] recognized named functions in AnnotatingParser.

djasper accepted this revision. djasper added a comment. This revision is now accepted and ready to land. The change looks good, but can you add a before/after example to change description? http://reviews.llvm.org/D21206 ___ cfe-commits mailing

r272324 - [CMake] Cleaning up CMake version checks in ExternalProject calls

Author: cbieneman Date: Thu Jun 9 17:38:42 2016 New Revision: 272324 URL: http://llvm.org/viewvc/llvm-project?rev=272324=rev Log: [CMake] Cleaning up CMake version checks in ExternalProject calls Now that we're on CMake 3.4.3 all the ExternalProject features we use are supported everywhere, so

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

jmgao updated this revision to Diff 60294. jmgao added a comment. Add test. http://reviews.llvm.org/D21163 Files: lib/Driver/Driver.cpp test/Driver/Inputs/android_triple_version/bin/arm-linux-androideabi-ld test/Driver/Inputs/android_triple_version/bin/arm-linux-androideabi-ld.exe

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

vsk added inline comments. Comment at: clang-tidy/modernize/UseEmplaceCheck.cpp:41 @@ +40,3 @@ + // (and destructed) as in push_back case. + auto isCtorOfSmartPtr = hasDeclaration(cxxConstructorDecl( + ofClass(hasAnyName("std::shared_ptr", "std::unique_ptr",

Re: [PATCH] D21198: RenderScript support in the Frontend

pirama updated this revision to Diff 60277. pirama added a comment. Reorder IK_RenderScript before IK_AST like Richard had requested. http://reviews.llvm.org/D21198 Files: include/clang/Basic/Attr.td include/clang/Basic/LangOptions.def include/clang/Frontend/FrontendOptions.h

Re: [PATCH] D21198: RenderScript support in the Frontend

This revision was automatically updated to reflect the committed changes. Closed by commit rL272342: RenderScript support in the Frontend (authored by pirama). Changed prior to commit: http://reviews.llvm.org/D21198?vs=60277=60279#toc Repository: rL LLVM http://reviews.llvm.org/D21198

r272342 - RenderScript support in the Frontend

Author: pirama Date: Thu Jun 9 18:34:20 2016 New Revision: 272342 URL: http://llvm.org/viewvc/llvm-project?rev=272342=rev Log: RenderScript support in the Frontend Summary: Create a new Frontend LangOpt to specify the renderscript language. It is enabled by the "-x renderscript" option from

[PATCH] D21212: Add documentation for RenderScript changes

pirama created this revision. pirama added a reviewer: rsmith. pirama added subscribers: srhines, cfe-commits. Herald added subscribers: danalbert, tberghammer. - Document the new 'kernel' attribute - Mention RenderScript support in the Release Notes. http://reviews.llvm.org/D21212 Files:

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

rsmith added a comment. Thanks for the updates, LGTM (@bruno, did you get the performance numbers you wanted?) http://reviews.llvm.org/D19843 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

Re: [PATCH] D19260: [analyzer][scan-build-py] subprocess output handling reviewed in clang module

dcoughlin accepted this revision. dcoughlin added a comment. This revision is now accepted and ready to land. This looks good to me. I've run this on a suite of open source projects and it reported no changes in the reference results. Thanks for simplifying and adding the extra tests!

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

eric_niebler added inline comments. Comment at: lib/Lex/PPDirectives.cpp:33 @@ -28,2 +32,2 @@ #include "llvm/Support/Path.h" #include "llvm/Support/SaveAndRestore.h" rsmith wrote: > eric_niebler wrote: > > You mean, instead of the `StringSet` below? Looks like

Re: [PATCH] D21198: RenderScript support in the Frontend

pirama added a comment. Thanks for the review. Docs update is in http://reviews.llvm.org/D21212 Repository: rL LLVM http://reviews.llvm.org/D21198 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

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

eric_niebler updated this revision to Diff 60281. eric_niebler added a comment. Replace `StringSet` with `StringSwitch`, ASCII range ends at 0x7f not 0xff, miscellaneous formatting tweaks. http://reviews.llvm.org/D19843 Files: include/clang/Basic/DiagnosticLexKinds.td

r272350 - Add doxygen comments to mmintrin.h's intrinsics.

Author: kromanova Date: Thu Jun 9 19:10:40 2016 New Revision: 272350 URL: http://llvm.org/viewvc/llvm-project?rev=272350=rev Log: Add doxygen comments to mmintrin.h's intrinsics. The doxygen comments are automatically generated based on Sony's intrinsics docu ment. I got an OK from Eric

r272357 - Remove CXXConstructExpr::getFoundDecl(); it turned out to not be useful.

Author: rsmith Date: Thu Jun 9 19:58:19 2016 New Revision: 272357 URL: http://llvm.org/viewvc/llvm-project?rev=272357=rev Log: Remove CXXConstructExpr::getFoundDecl(); it turned out to not be useful. Modified: cfe/trunk/include/clang/AST/ExprCXX.h cfe/trunk/include/clang/Sema/Sema.h

[PATCH] D21199: Add a RenderScript language type

pirama created this revision. pirama added a reviewer: rsmith. pirama added subscribers: srhines, cfe-commits. Add RenderScript language type and associate it with ".rs" extensions. Test that the driver passes "-x renderscript" to the frontend for ".rs" files. (Also add '.rs' to the list of

r272312 - [CMake] Cleaning up CMake feature gating on 2.8.12

Author: cbieneman Date: Thu Jun 9 16:29:55 2016 New Revision: 272312 URL: http://llvm.org/viewvc/llvm-project?rev=272312=rev Log: [CMake] Cleaning up CMake feature gating on 2.8.12 CMake 2.8.12 introduced interface libraries and some related policies. This removes the conditional block because

Re: [PATCH] D21198: RenderScript support in the Frontend

rsmith accepted this revision. rsmith added a comment. This revision is now accepted and ready to land. Please make sure that any relevant parts of our documentation are also updated. Given how small and self-contained this is, that we have a reasonable expectation of good support and

r272318 - Fix a crash in the AST dumper.

Author: rtrieu Date: Thu Jun 9 17:03:04 2016 New Revision: 272318 URL: http://llvm.org/viewvc/llvm-project?rev=272318=rev Log: Fix a crash in the AST dumper. Boxed expressions in a template context may have a null method decl. If so, don't try to access the selector. Modified:

Re: [PATCH] D20388: AMDGPU: Fix supported CL features

jvesely marked an inline comment as done. jvesely added a comment. Repository: rL LLVM http://reviews.llvm.org/D20388 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D20388: AMDGPU: Fix supported CL features

jvesely updated this revision to Diff 60259. jvesely added a comment. tests all extensions against expected outcome (add negative tests) enable cl_khr_icd (works ok with mesa) Repository: rL LLVM http://reviews.llvm.org/D20388 Files: lib/Basic/Targets.cpp

r272325 - Redirect unused output in test to /dev/null

Author: rtrieu Date: Thu Jun 9 17:39:20 2016 New Revision: 272325 URL: http://llvm.org/viewvc/llvm-project?rev=272325=rev Log: Redirect unused output in test to /dev/null Discard unused output so when the test fails, it only prints information that is helpful about the failure. No functional

Re: [PATCH] D20933: Preallocate ExplodedNode hash table

zaks.anna accepted this revision. zaks.anna added a comment. This revision is now accepted and ready to land. LGTM. Thanks! http://reviews.llvm.org/D20933 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

Re: Does anyone need these zorg modules?

Hello, Last call for the next builder modules: ChrootSetup.py DragonEggBuilder.py KLEEBuilder.py ScriptedBuilder.py gccSuiteBuilder.py I am going to remove them. If anyone have plans for any of them please speak up! Thanks Galina ___ cfe-commits

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

cameron314 added inline comments. Comment at: lib/Basic/FileManager.cpp:389 @@ -383,2 +388,3 @@ UFE->File.reset(); + UFE->IsVirtual = true; return UFE; rsmith wrote: > Yes. The `IsValid` flag is just supposed to mean that this file has actually > been

Re: [PATCH] D21199: Add a RenderScript language type

This revision was automatically updated to reflect the committed changes. Closed by commit rL272317: Add a RenderScript language type (authored by pirama). Changed prior to commit: http://reviews.llvm.org/D21199?vs=60238=60249#toc Repository: rL LLVM http://reviews.llvm.org/D21199 Files:

r272317 - Add a RenderScript language type

Author: pirama Date: Thu Jun 9 16:57:40 2016 New Revision: 272317 URL: http://llvm.org/viewvc/llvm-project?rev=272317=rev Log: Add a RenderScript language type Summary: Add RenderScript language type and associate it with ".rs" extensions. Test that the driver passes "-x renderscript" to the

  1   2   >