Re: [PATCH] D13081: [clang-tidy] Implement FixitHints for identifier references in IdentifierNamingCheck

berenm updated this revision to Diff 35466. berenm added a comment. Fix incorrect line removal in unit test. http://reviews.llvm.org/D13081 Files: clang-tidy/readability/IdentifierNamingCheck.cpp test/clang-tidy/readability-identifier-naming.cpp Index: test/clang-tidy/readability-identifie

Re: Implict casts disappeared from syntactic init list expressions in C++

Ping^2... Il 29/08/2015 10:01, Abramo Bagnara ha scritto: > Il 28/08/2015 23:27, Richard Smith ha scritto: >> On Tue, Aug 25, 2015 at 10:27 AM, Abramo Bagnara >> mailto:abramo.bagn...@bugseng.com>> wrote: >> >> Comparing the result of InitListExpr::getSyntacticForm between r224986 >> and r

Re: [PATCH] D13081: [clang-tidy] Implement FixitHints for identifier references in IdentifierNamingCheck

berenm changed the visibility of this Differential Revision from "berenm (Beren Minor)" to "Public (No Login Required)". berenm updated this revision to Diff 35465. berenm added a comment. Remove remaining commented code. http://reviews.llvm.org/D13081 Files: clang-tidy/readability/Identifie

Re: [PATCH] D12774: createUniqueFile() is documented to create the file in the temporary directory unless it's supplied an absolute path.Make sure the output filepath supplied to createUniqueFile() in

dirty updated this revision to Diff 35463. dirty added a comment. - Update patch with feedback from vsk: better handle errors from make_absolute(). Updating D12774: createUniqueFile() is documented to create the file in the temporary directory unless it's supplied an absolute path.

Re: [PATCH] D12379: Fix the bugs in the mapDiagnosticRanges (Still in progress)

rtrieu added inline comments. Comment at: lib/Basic/SourceManager.cpp:1008 @@ -1008,1 +1007,3 @@ +bool SourceManager::isMacroArgExpansion(SourceLocation Loc, + SourceLocation *NewLoc) const { if (!Loc.isMacroID()) return false;

Re: [PATCH] D12774: createUniqueFile() is documented to create the file in the temporary directory unless it's supplied an absolute path.Make sure the output filepath supplied to createUniqueFile() in

vsk added a comment. Yes, that looks good. http://reviews.llvm.org/D12774 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D9924: Ignore report when the argument to malloc is assigned known value

hiraditya updated this revision to Diff 35461. hiraditya added a comment. Updated commit message and summary. http://reviews.llvm.org/D9924 Files: lib/StaticAnalyzer/Checkers/MallocOverflowSecurityChecker.cpp test/Analysis/malloc-overflow.c test/Analysis/malloc-overflow2.c Index: test/An

Re: [PATCH] D12996: Driver: support ARM/HF on a single toolchain

compnerd added a comment. Yeah, I realize that we will need that part of the change as well. However, I think that this is a sufficiently standalone piece that we can merge this first. http://reviews.llvm.org/D12996 ___ cfe-commits mailing list cf

Re: [PATCH] D13080: [libc++] ~unique_ptr() should not set stored pointer to null

thakis added a subscriber: thakis. thakis added a comment. mclow should review this, not me. http://reviews.llvm.org/D13080 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D12903: Add -fplugin=name.so option to the driver

rengolin added a comment. In http://reviews.llvm.org/D12903#249406, @thakis wrote: > Also also, this approach fundamentally doesn't work on Windows. I don't think it's supposed to, anyway. :) Repository: rL LLVM http://reviews.llvm.org/D12903

Re: [PATCH] D13080: [libc++] ~unique_ptr() should not set stored pointer to null

On Tue, Sep 22, 2015 at 5:26 PM, Daniel Cheng via cfe-commits < cfe-commits@lists.llvm.org> wrote: > dcheng added a comment. > > In http://reviews.llvm.org/D13080#251324, @dblaikie wrote: > > > I'd still be curious to see a stronger justification from the standard > > about this. > > > > I know y

Re: [PATCH] D13080: [libc++] ~unique_ptr() should not set stored pointer to null

dcheng added a comment. In http://reviews.llvm.org/D13080#251324, @dblaikie wrote: > I'd still be curious to see a stronger justification from the standard > about this. > > I know you mentioned/quoted the definition of ~unique_ptr in the standard - > but I'm not sure that is meant to imply obs

Re: r248234 - ms Intrin.h: Fix __movsw's and __stosw's inline asm.

r248355 might help. On Tue, Sep 22, 2015 at 8:23 PM, Nico Weber wrote: > It says "error: unable to create target: 'No available targets are > compatible with this triple, see -version for the available targets.'" I > guess you disable a bunch of targets on that bot? > > On Tue, Sep 22, 2015 at 6

r248355 - Make ms-intrin.cpp test require the x86 target.

Author: nico Date: Tue Sep 22 19:24:34 2015 New Revision: 248355 URL: http://llvm.org/viewvc/llvm-project?rev=248355&view=rev Log: Make ms-intrin.cpp test require the x86 target. Modified: cfe/trunk/test/Headers/ms-intrin.cpp Modified: cfe/trunk/test/Headers/ms-intrin.cpp URL: http://llvm.o

Re: [PATCH] D12417: Improvements to localizability checks for iOS / OS X

dcoughlin added a comment. This is causing tests to fail on the bots: http://lab.llvm.org:8011/builders/clang-x86_64-debian-fast/builds/31373 /home/llvmbb/llvm-build-dir/clang-x86_64-debian-fast/llvm.obj/Release+Asserts/bin/clang -cc1 -internal-isystem /home/llvmbb/llvm-build-dir/clang-x86_

Re: r248234 - ms Intrin.h: Fix __movsw's and __stosw's inline asm.

It says "error: unable to create target: 'No available targets are compatible with this triple, see -version for the available targets.'" I guess you disable a bunch of targets on that bot? On Tue, Sep 22, 2015 at 6:00 PM, Renato Golin via cfe-commits < cfe-commits@lists.llvm.org> wrote: > On 21

r248351 - Revert "[analyzer] Improve localizability checks for iOS / OS X."

Author: dcoughlin Date: Tue Sep 22 19:17:52 2015 New Revision: 248351 URL: http://llvm.org/viewvc/llvm-project?rev=248351&view=rev Log: Revert "[analyzer] Improve localizability checks for iOS / OS X." This reverts commit r248350. The pluralization checks are failing on some bots. Modified:

Re: [PATCH] D13067: [CUDA] __global__ functions should always be visible externally.

echristo accepted this revision. echristo added a comment. This revision is now accepted and ready to land. Seems to be what we were chatting about. Thanks! -eric http://reviews.llvm.org/D13067 ___ cfe-commits mailing list cfe-commits@lists.llvm.o

Re: [PATCH] D13080: [libc++] ~unique_ptr() should not set stored pointer to null

dblaikie added a subscriber: dblaikie. dblaikie added a comment. I'd still be curious to see a stronger justification from the standard about this. I know you mentioned/quoted the definition of ~unique_ptr in the standard - but I'm not sure that is meant to imply observable behavior during destru

Re: [PATCH] D12989: [CUDA] Added CUDA installation detector class.

echristo accepted this revision. echristo added a comment. This revision is now accepted and ready to land. LGTM. Thanks! -eric http://reviews.llvm.org/D12989 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/ma

[PATCH] D13080: [libc++] ~unique_ptr() should not set stored pointer to null

dcheng created this revision. dcheng added a reviewer: thakis. dcheng added a subscriber: cfe-commits. This better matches the behavior of MSVC and libstdc++: neither standard library sets the stored pointer to null when destroying the unique_ptr. http://reviews.llvm.org/D13080 Files: include/

Re: [PATCH] D12417: Improvements to localizability checks for iOS / OS X

This revision was automatically updated to reflect the committed changes. Closed by commit rL248350: [analyzer] Improve localizability checks for iOS / OS X. (authored by dcoughlin). Changed prior to commit: http://reviews.llvm.org/D12417?vs=35404&id=35445#toc Repository: rL LLVM http://rev

r248350 - [analyzer] Improve localizability checks for iOS / OS X.

Author: dcoughlin Date: Tue Sep 22 18:58:04 2015 New Revision: 248350 URL: http://llvm.org/viewvc/llvm-project?rev=248350&view=rev Log: [analyzer] Improve localizability checks for iOS / OS X. Various improvements to the localization checker: * Adjusted copy to be consistent with diagnostic text

Re: [PATCH] D12840: [cfe-dev] Enabling ThreadSanitizer on PPC64(BE/LE) plarforms

samsonov added a subscriber: samsonov. samsonov accepted this revision. samsonov added a reviewer: samsonov. samsonov added a comment. LGTM http://reviews.llvm.org/D12840 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/

Re: [PATCH] D9924: Ignore report when the argument to malloc is assigned known value

dcoughlin added a comment. Aditya, can you update the patch title and summary to a commit message so I can commit it? Thanks! http://reviews.llvm.org/D9924 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailma

Re: [PATCH] D12633: Implement ACLE 2.0 macros of chapters 6.6 and 6.7 for [ARM] and [Aarch64] targets

rengolin added a comment. In http://reviews.llvm.org/D12633#250612, @labrinea wrote: > __ARM_FP_FAST is defined to 1 if floating-point optimizations may occur such > that the computed results are different from those prescribed by the order of > operations according to the C standard. Examples

r248344 - Serialization: Let ASTWriter return the signature of the written module.

Author: adrian Date: Tue Sep 22 18:26:31 2015 New Revision: 248344 URL: http://llvm.org/viewvc/llvm-project?rev=248344&view=rev Log: Serialization: Let ASTWriter return the signature of the written module. NFC Modified: cfe/trunk/include/clang/Serialization/ASTWriter.h cfe/trunk/lib/Seri

r248345 - Module Debugging: Use the clang module signature as the module's dwo_id

Author: adrian Date: Tue Sep 22 18:26:43 2015 New Revision: 248345 URL: http://llvm.org/viewvc/llvm-project?rev=248345&view=rev Log: Module Debugging: Use the clang module signature as the module's dwo_id when building a module. Clang already records the module signature when building a skeleton C

Re: [PATCH] D9040: [analyzer] Make realloc(ptr, 0) handling equivalent to malloc(0).

This revision was automatically updated to reflect the committed changes. Closed by commit rL248336: [analyzer] Make realloc(ptr, 0) handling equivalent to malloc(0). (authored by dcoughlin). Changed prior to commit: http://reviews.llvm.org/D9040?vs=34583&id=35432#toc Repository: rL LLVM ht

r248336 - [analyzer] Make realloc(ptr, 0) handling equivalent to malloc(0).

Author: dcoughlin Date: Tue Sep 22 17:47:14 2015 New Revision: 248336 URL: http://llvm.org/viewvc/llvm-project?rev=248336&view=rev Log: [analyzer] Make realloc(ptr, 0) handling equivalent to malloc(0). Currently realloc(ptr, 0) is treated as free() which seems to be not correct. C standard (N1570

Re: [PATCH] D12251: Analyzer: Calculate field offset correctly

ismailp added a comment. Ping! http://reviews.llvm.org/D12251 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: r248234 - ms Intrin.h: Fix __movsw's and __stosw's inline asm.

On 21 September 2015 at 17:46, Nico Weber via cfe-commits wrote: > Author: nico > Date: Mon Sep 21 19:46:21 2015 > New Revision: 248234 > > URL: http://llvm.org/viewvc/llvm-project?rev=248234&view=rev > Log: > ms Intrin.h: Fix __movsw's and __stosw's inline asm. Hi Nico, This still seems to be b

[libcxx] r248329 - Fix Typo in GCC no RTTI detection. Fixes PR#24901. Thanks to Bernhard Rosenkraenzer for the report and the patch.

Author: marshall Date: Tue Sep 22 16:58:30 2015 New Revision: 248329 URL: http://llvm.org/viewvc/llvm-project?rev=248329&view=rev Log: Fix Typo in GCC no RTTI detection. Fixes PR#24901. Thanks to Bernhard Rosenkraenzer for the report and the patch. Modified: libcxx/trunk/include/__config Mo

Re: [PATCH] D12840: [cfe-dev] Enabling ThreadSanitizer on PPC64(BE/LE) plarforms

hfinkel added a subscriber: hfinkel. hfinkel accepted this revision. hfinkel added a reviewer: hfinkel. hfinkel added a comment. This revision is now accepted and ready to land. LGTM, although don't commit until any necessary backend/compiler-rt patches are in. http://reviews.llvm.org/D12840

Re: [PATCH] D13013: [ARM] Fix crash "-target arm -mcpu=generic", without "-march="

rengolin accepted this revision. rengolin added a comment. This revision is now accepted and ready to land. Apart from the style nitpick, LGTM. Thanks! Comment at: lib/Driver/Tools.cpp:6139 @@ +6138,3 @@ + ArchKind = llvm::ARM::parseCPUArch(Triple.getARMCPUForArch(Arch)); +

Re: [PATCH] D13051: Add placeholder __libcpp_relaxed_store() for when atomic builtins are not available.

dim added a comment. In http://reviews.llvm.org/D13051#250921, @jroelofs wrote: > In http://reviews.llvm.org/D13051#250910, @EricWF wrote: > > > @jroelofs @dim, could we fallback to the __sync_* builtins on arm? > The actual implementations of these __sync builtins should still come from somew

[PATCH] D13071: [PATCH] New checker for mismatched operator new/operator delete definitions

aaron.ballman created this revision. aaron.ballman added reviewers: klimek, alexfh. aaron.ballman added a subscriber: cfe-commits. When defining a free store function, it is almost invariably a bug to fail to declare the corresponding free store function. For instance, implementing operator new(

Re: r248293 - [CUDA] Add implicit __attribute__((used)) to all __global__ functions.

On Tue, Sep 22, 2015 at 11:49 AM, Richard Smith wrote: > It seems like the real problem here is that we're giving the template > instantiation the wrong linkage. It can be used from outside this llvm > module, so it should be weak_odr instead of linkonce_odr. > This indeed works much better. I've

[PATCH] D13067: [CUDA] __global__ functions should always be visible externally.

tra created this revision. tra added reviewers: rsmith, echristo. tra added a subscriber: cfe-commits. Adjusted __global__ functions with DiscardableODR linkage to use StrongODR linkage instead so they are visible externally. Replaces D11666 / r248293 http://reviews.llvm.org/D13067 Files: l

Re: [PATCH] D5102: [analyzer][Bugfix/improvement] Fix for PR16833

This revision was automatically updated to reflect the committed changes. Closed by commit rL248318: [analyzer] Create one state for a range switch case instead of multiple. (authored by dcoughlin). Changed prior to commit: http://reviews.llvm.org/D5102?vs=32664&id=35411#toc Repository: rL L

r248318 - [analyzer] Create one state for a range switch case instead of multiple.

Author: dcoughlin Date: Tue Sep 22 15:31:19 2015 New Revision: 248318 URL: http://llvm.org/viewvc/llvm-project?rev=248318&view=rev Log: [analyzer] Create one state for a range switch case instead of multiple. This fixes PR16833, in which the analyzer was using large amounts of memory for switch s

Re: [PATCH] D13051: Add placeholder __libcpp_relaxed_store() for when atomic builtins are not available.

On Tue, Sep 22, 2015 at 07:26:46PM +, Jonathan Roelofs via cfe-commits wrote: > jroelofs added a comment. > > In http://reviews.llvm.org/D13051#250910, @EricWF wrote: > > > @jroelofs @dim, could we fallback to the __sync_* builtins on arm? > > > @dim would need armv4-flavored implementatio

Re: [PATCH] D10881: [Sema] Catch a case when 'volatile' qualifier is dropped while binding

rjmccall added a comment. This is outside of my expertise, but I've asked Doug Gregor to take a look. http://reviews.llvm.org/D10881 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] __attribute__((enable_if)) and non-overloaded member functions

Hello, > Looks good to me! > > Do you have commit access, or would you like for me to commit it for you? I do not have commit access. It would be great if you can commit for me. Thanks, Ettore Speziale ___ cfe-commits mailing list cfe-commits@lists.ll

Re: [PATCH] D12417: Improvements to localizability checks for iOS / OS X

kulpreet added a subscriber: cfe-commits. kulpreet updated this revision to Diff 35404. kulpreet added a comment. Shortened description of PluralMisuseChecker in Checkers.td as per Anna's recommendation. http://reviews.llvm.org/D12417 Files: lib/StaticAnalyzer/Checkers/Checkers.td lib/Stat

Re: [PATCH] D12482: Analyzer: Teach analyzer how to handle TypeTraitExpr

This revision was automatically updated to reflect the committed changes. Closed by commit rL248314: Analyzer: Teach analyzer how to handle TypeTraitExpr (authored by ismailp). Changed prior to commit: http://reviews.llvm.org/D12482?vs=35384&id=35403#toc Repository: rL LLVM http://reviews.l

r248314 - Analyzer: Teach analyzer how to handle TypeTraitExpr

Author: ismailp Date: Tue Sep 22 14:33:15 2015 New Revision: 248314 URL: http://llvm.org/viewvc/llvm-project?rev=248314&view=rev Log: Analyzer: Teach analyzer how to handle TypeTraitExpr Summary: `TypeTraitExpr`s are not supported by the ExprEngine today. Analyzer creates a sink, and aborts the b

Re: [PATCH] D13051: Add placeholder __libcpp_relaxed_store() for when atomic builtins are not available.

jroelofs added a comment. In http://reviews.llvm.org/D13051#250910, @EricWF wrote: > @jroelofs @dim, could we fallback to the __sync_* builtins on arm? @dim would need armv4-flavored implementations of them in compiler-rt (if that's what he's using)... the existing ones use instructions that v

Re: r244063 - Add missing atomic libcall support.

On 05 Aug 2015, at 18:57, James Y Knight wrote: > Author: jyknight > Date: Wed Aug 5 11:57:36 2015 > New Revision: 244063 > > URL: http://llvm.org/viewvc/llvm-project?rev=244063&view=rev > Log: > Add missing atomic libcall support. > > Support for emitting libcalls for __atomic_fetch_nand and >

Re: [PATCH] D12362: [clang-format] Add support of consecutive declarations alignment

berenm added a comment. Ping? The unit tests should work fine, now that http://reviews.llvm.org/D12369 has been merged. http://reviews.llvm.org/D12362 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/li

Re: [PATCH] D13051: Add placeholder __libcpp_relaxed_store() for when atomic builtins are not available.

EricWF added a comment. @jroelofs @dim, could we fallback to the __sync_* builtins on arm? http://reviews.llvm.org/D13051 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[libcxx] r248313 - Add placeholder __libcpp_relaxed_store() for when atomic builtins are not available.

Author: dim Date: Tue Sep 22 13:55:37 2015 New Revision: 248313 URL: http://llvm.org/viewvc/llvm-project?rev=248313&view=rev Log: Add placeholder __libcpp_relaxed_store() for when atomic builtins are not available. Summary: In rL241532, atomic_support.h was added, which provides handling of atom

Re: [PATCH] D12689: [libc++][static linking] std streams are not initialized prior to their use in static object constructors

On Sep 11, 2015 4:23 AM, "Evgeny Astigeevich via cfe-commits" < cfe-commits@lists.llvm.org> wrote: > > eastig added a comment. > > In http://reviews.llvm.org/D12689#243592, @rsmith wrote: > > > Can we instead fix this in Clang by ensuring that libc++ is put at the right position in the static link

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

rjmccall added inline comments. Comment at: docs/LanguageExtensions.rst:1720 @@ -1712,1 +1719,3 @@ +being stored there, and the function returns 1. The behavior of these builtins +is well-defined for all argument values. DavidEGrayson wrote: > rjmccall wrote: >

Re: r248293 - [CUDA] Add implicit __attribute__((used)) to all __global__ functions.

It seems like the real problem here is that we're giving the template instantiation the wrong linkage. It can be used from outside this llvm module, so it should be weak_odr instead of linkonce_odr. On Sep 22, 2015 10:24 AM, "Artem Belevich via cfe-commits" < cfe-commits@lists.llvm.org> wrote: > A

Re: [PATCH] D13062: [libcxx] Fix PR24779 -- Prevent evaluation of std::is_default_constructible in clearly ill-formed tuple constructor.

EricWF added a comment. This probably isn't quite right. I need to think about it for a minute. http://reviews.llvm.org/D13062 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D12247: [libc++] remove possible trailing padding from aligned_storage

EricWF added a comment. Sure sorry for the delay. http://reviews.llvm.org/D12247 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D12247: [libc++] remove possible trailing padding from aligned_storage

EricWF closed this revision. EricWF added a comment. Committed as r248309. http://reviews.llvm.org/D12247 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[libcxx] r248309 - Remove possible trailing padding from aligned_storage. Patch from Yiran Wang

Author: ericwf Date: Tue Sep 22 13:37:03 2015 New Revision: 248309 URL: http://llvm.org/viewvc/llvm-project?rev=248309&view=rev Log: Remove possible trailing padding from aligned_storage. Patch from Yiran Wang Modified: libcxx/trunk/include/type_traits Modified: libcxx/trunk/include/type_tra

Re: [PATCH] D13051: Add placeholder __libcpp_relaxed_store() for when atomic builtins are not available.

EricWF accepted this revision. EricWF added a comment. This revision is now accepted and ready to land. Ah sorry I hadn't had coffee yet. Sorry for the breakage. I'll make sure this make sure this patch gets backported appropriately. LGTM thanks for the fix. http://reviews.llvm.org/D13051 _

[PATCH] D13062: [libcxx] Fix PR24779 -- Prevent evaluation of std::is_default_constructible in clearly ill-formed tuple constructor.

EricWF created this revision. EricWF added a reviewer: mclow.lists. EricWF added a subscriber: cfe-commits. This patch fixes PR24779 (https://llvm.org/bugs/show_bug.cgi?id=24779) http://reviews.llvm.org/D13062 Files: include/tuple test/std/utilities/tuple/tuple.tuple/tuple.cnstr/alloc_UType

Re: [PATCH] D13051: Add placeholder __libcpp_relaxed_store() for when atomic builtins are not available.

dim added a comment. In http://reviews.llvm.org/D13051#250836, @EricWF wrote: > Does anything actually need this? This interface was never really meant to be > complete, just want was needed. I need it at least for FreeBSD/arm, since it defaults to armv4, and it does not yet have a full set o

Re: [PATCH] D13051: Add placeholder __libcpp_relaxed_store() for when atomic builtins are not available.

On Tue, Sep 22, 2015 at 05:44:59PM +, Eric Fiselier via cfe-commits wrote: > EricWF added a comment. > > Does anything actually need this? This interface was never really meant to be > complete, just want was needed. > > > It's a shame that libc++ decided to reinvent the wheel here and not u

[libcxx] r248307 - The test I cnecked in to check the fix for PR#24890 failed (as expected) w/o the fix, but for the wrong reason. Now it fails for the right reason.

Author: marshall Date: Tue Sep 22 13:09:13 2015 New Revision: 248307 URL: http://llvm.org/viewvc/llvm-project?rev=248307&view=rev Log: The test I cnecked in to check the fix for PR#24890 failed (as expected) w/o the fix, but for the wrong reason. Now it fails for the right reason. Modified:

Re: [PATCH] D5102: [analyzer][Bugfix/improvement] Fix for PR16833

dcoughlin added a comment. I will commit. Thanks! http://reviews.llvm.org/D5102 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] __attribute__((enable_if)) and non-overloaded member functions

Looks good to me! Do you have commit access, or would you like for me to commit it for you? George On Tue, Sep 22, 2015 at 7:45 AM, Ettore Speziale wrote: > Hello, > > > It looks like the attached patch is the same as the original one? > > I’ve attache the wrong patch. Here is the right one: >

Re: [PATCH] D11240: Add basic #include sorting functionality to clang-format

klimek accepted this revision. klimek added a comment. This revision is now accepted and ready to land. Ok, I think this is now understandable enough for me to go in. http://reviews.llvm.org/D11240 ___ cfe-commits mailing list cfe-commits@lists.llvm

Re: [PATCH] D5102: [analyzer][Bugfix/improvement] Fix for PR16833

ygribov added a subscriber: ygribov. ygribov added a comment. Folks, could someone commit this for us? http://reviews.llvm.org/D5102 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D11240: Add basic #include sorting functionality to clang-format

djasper updated this revision to Diff 35395. djasper added a comment. Added comments and did some renaming. http://reviews.llvm.org/D11240 Files: include/clang/Format/Format.h include/clang/Tooling/Core/Replacement.h lib/Format/Format.cpp lib/Format/FormatToken.cpp lib/Tooling/Core/Re

[libcxx] r248305 - Check in the test for PR#24890 that I forgot in previous commit

Author: marshall Date: Tue Sep 22 12:57:41 2015 New Revision: 248305 URL: http://llvm.org/viewvc/llvm-project?rev=248305&view=rev Log: Check in the test for PR#24890 that I forgot in previous commit Modified: libcxx/trunk/test/std/utilities/utility/pairs/pairs.pair/swap.pass.cpp Modified: li

Re: [PATCH] D13051: Add placeholder __libcpp_relaxed_store() for when atomic builtins are not available.

jroelofs added a comment. In http://reviews.llvm.org/D13051#250836, @EricWF wrote: > Does armv4 need lib calls for atomic operations on "unsigned long"? IIRC, yes. http://reviews.llvm.org/D13051 ___ cfe-commits mailing list cfe-commits@lists.llvm

[libcxx] r248304 - Change pair::swap(pair&) to call ADL swap instead of iter_swap; this fixes an obscure bug having to do with overloaded operator&. Fixes PR#24890

Author: marshall Date: Tue Sep 22 12:50:11 2015 New Revision: 248304 URL: http://llvm.org/viewvc/llvm-project?rev=248304&view=rev Log: Change pair::swap(pair&) to call ADL swap instead of iter_swap; this fixes an obscure bug having to do with overloaded operator&. Fixes PR#24890 Modified: li

Re: [PATCH] D13051: Add placeholder __libcpp_relaxed_store() for when atomic builtins are not available.

EricWF added a comment. Does anything actually need this? This interface was never really meant to be complete, just want was needed. > It's a shame that libc++ decided to reinvent the wheel here and not use the > C11 atomics support GCC doesn't provide "_Atomic" and `` isn't always fully ava

Re: [PATCH] D12917: [CUDA] Allow parsing of host and device code simultaneously.

This revision was automatically updated to reflect the committed changes. tra marked an inline comment as done. Closed by commit rL248299: [CUDA] Allow parsing of host and device code simultaneously. (authored by tra). Changed prior to commit: http://reviews.llvm.org/D12917?vs=35031&id=35389#to

Re: [PATCH] D12893: Augmented CudaHostAction to carry device-side triple.

This revision was automatically updated to reflect the committed changes. Closed by commit rL248298: Augmented CudaHostAction to carry device-side triple. (authored by tra). Changed prior to commit: http://reviews.llvm.org/D12893?vs=34857&id=35388#toc Repository: rL LLVM http://reviews.llvm

Re: [PATCH] D12892: [CUDA] Minor cuda-related driver fixes.

This revision was automatically updated to reflect the committed changes. Closed by commit rL248297: [CUDA] Fixes minor cuda-related issues in the driver (authored by tra). Changed prior to commit: http://reviews.llvm.org/D12892?vs=34855&id=35386#toc Repository: rL LLVM http://reviews.llvm.

Re: [PATCH] D12453: [CUDA] Allow function overloads based on host/device attributes.

This revision was automatically updated to reflect the committed changes. Closed by commit rL248295: [CUDA] Allow function overloads in CUDA based on host/device attributes. (authored by tra). Changed prior to commit: http://reviews.llvm.org/D12453?vs=34368&id=35387#toc Repository: rL LLVM

r248299 - [CUDA] Allow parsing of host and device code simultaneously.

Author: tra Date: Tue Sep 22 12:23:22 2015 New Revision: 248299 URL: http://llvm.org/viewvc/llvm-project?rev=248299&view=rev Log: [CUDA] Allow parsing of host and device code simultaneously. * adds -aux-triple option to specify target triple * propagates aux target info to AST context and Prepr

Re: PATCH: Provide the compile commands of the JSON database in consistent order

In r248292, the added unit test fails without the changes. > On Sep 18, 2015, at 5:49 PM, Manuel Klimek via cfe-commits > wrote: > > LG in general; I think if we like the order to be deterministic we should try > to come up with a unit test so nobody regresses this in the future. > > On Fri,

r248298 - Augmented CudaHostAction to carry device-side triple.

Author: tra Date: Tue Sep 22 12:23:13 2015 New Revision: 248298 URL: http://llvm.org/viewvc/llvm-project?rev=248298&view=rev Log: Augmented CudaHostAction to carry device-side triple. Differential Revision: http://reviews.llvm.org/D12893 Modified: cfe/trunk/include/clang/Driver/Action.h

r248297 - [CUDA] Fixes minor cuda-related issues in the driver

Author: tra Date: Tue Sep 22 12:23:09 2015 New Revision: 248297 URL: http://llvm.org/viewvc/llvm-project?rev=248297&view=rev Log: [CUDA] Fixes minor cuda-related issues in the driver * Only the last of the --cuda-host-only/--cuda-device-only options has effect. * CudaHostAction always wraps host-

r248295 - [CUDA] Allow function overloads in CUDA based on host/device attributes.

Author: tra Date: Tue Sep 22 12:22:59 2015 New Revision: 248295 URL: http://llvm.org/viewvc/llvm-project?rev=248295&view=rev Log: [CUDA] Allow function overloads in CUDA based on host/device attributes. The patch makes it possible to parse CUDA files that contain host/device functions with identi

r248296 - [CUDA] Add appropriate host/device attribute to builtins.

Author: tra Date: Tue Sep 22 12:23:05 2015 New Revision: 248296 URL: http://llvm.org/viewvc/llvm-project?rev=248296&view=rev Log: [CUDA] Add appropriate host/device attribute to builtins. The changes are part of attribute-based CUDA function overloading (D12453) and as such are only enabled when

r248293 - [CUDA] Add implicit __attribute__((used)) to all __global__ functions.

Author: tra Date: Tue Sep 22 12:22:51 2015 New Revision: 248293 URL: http://llvm.org/viewvc/llvm-project?rev=248293&view=rev Log: [CUDA] Add implicit __attribute__((used)) to all __global__ functions. This makes sure that we emit kernels that were instantiated from the host code and which would n

r248292 - [tooling] Provide the compile commands of the JSON database in the order that they were provided in the JSON file.

Author: akirtzidis Date: Tue Sep 22 12:22:33 2015 New Revision: 248292 URL: http://llvm.org/viewvc/llvm-project?rev=248292&view=rev Log: [tooling] Provide the compile commands of the JSON database in the order that they were provided in the JSON file. This is useful for debugging of issues and r

Re: [PATCH] D12482: Analyzer: Teach analyzer how to handle TypeTraitExpr

ismailp marked an inline comment as done. ismailp added a comment. Thanks for reviewing. http://reviews.llvm.org/D12482 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D12482: Analyzer: Teach analyzer how to handle TypeTraitExpr

ismailp updated this revision to Diff 35384. ismailp added a comment. Addressed comments. http://reviews.llvm.org/D12482 Files: lib/StaticAnalyzer/Core/Environment.cpp lib/StaticAnalyzer/Core/ExprEngine.cpp lib/StaticAnalyzer/Core/SValBuilder.cpp test/Analysis/dtor.cpp Index: test/Anal

Re: [PATCH] D13052: Add NamingStyle option to modernize-loop-convert.

angelgarcia updated this revision to Diff 35380. angelgarcia marked 10 inline comments as done. angelgarcia added a comment. Use CamelCase on the existing tests and add a test for every other naming convention. http://reviews.llvm.org/D13052 Files: clang-tidy/modernize/LoopConvertCheck.cpp

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

DavidEGrayson added a comment. Thanks for looking at this patch, John! I disagreed with your comment about calculating the encompassing type, but I will incorporate everything else and make a new patch tonight or within a few days. Comment at: docs/LanguageExtensions.rst:1720

Re: [PATCH] D11240: Add basic #include sorting functionality to clang-format

klimek added a comment. Sending another batch of comments. Comment at: lib/Tooling/Core/Replacement.cpp:305-307 @@ +304,5 @@ + Replacements Result; + // Iterate over both sets and always add the next element (smallest total + // Offset) from either 'First' or 'Second'. Merge

Re: [PATCH] D13048: Fix for merging decls in pragma weak

aaron.ballman added a comment. Generally LGTM, but I would wait for Richard to review in this case. ~Aaron Comment at: lib/Sema/SemaDeclAttr.cpp:5201 @@ -5200,5 +5200,3 @@ if (FunctionDecl *FD = dyn_cast(ND)) { -FunctionDecl *NewFD; -// FIXME: Missing call to CheckFu

Re: r248276 - [mips] Added support for using the command line options -Wa, -msoft-float and -Wa, -mhard-float.

On Tue, Sep 22, 2015 at 01:52:33PM -, Daniel Sanders via cfe-commits wrote: > Author: dsanders > Date: Tue Sep 22 08:52:32 2015 > New Revision: 248276 > > URL: http://llvm.org/viewvc/llvm-project?rev=248276&view=rev > Log: > [mips] Added support for using the command line options -Wa,-msoft-fl

Re: [PATCH] D13048: Fix for merging decls in pragma weak

amusman added inline comments. Comment at: lib/Sema/SemaDeclAttr.cpp:5201 @@ -5200,5 +5200,3 @@ if (FunctionDecl *FD = dyn_cast(ND)) { -FunctionDecl *NewFD; -// FIXME: Missing call to CheckFunctionDeclaration(). // FIXME: Mangling? // FIXME: Is the qualifier i

Re: [PATCH] D12633: Implement ACLE 2.0 macros of chapters 6.6 and 6.7 for [ARM] and [Aarch64] targets

t.p.northover added a comment. I can't explain the Opts.C99 (fesetround is provided by conforming C++11 implementations too), but I think I was just trying to capture the fact that decent hosted platforms support fesetround but freestanding ones don't (not having any library). Supporting "C89

Re: [PATCH] D11240: Add basic #include sorting functionality to clang-format

djasper updated this revision to Diff 35375. djasper added a comment. Restructured a bit. Not as many comments just yet. Do you think this is an improvement? http://reviews.llvm.org/D11240 Files: include/clang/Format/Format.h include/clang/Tooling/Core/Replacement.h lib/Format/Format.cpp

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

ksuther added a comment. I don't, this and http://reviews.llvm.org/D12489 are my first patch submissions. http://reviews.llvm.org/D12489 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commi

Re: [PATCH] D12501: [clang-format] Obj-C dictionary literals: Fixed typecast getting put on a separate line from the key

ksuther added a comment. I do not. http://reviews.llvm.org/D12501 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] __attribute__((enable_if)) and non-overloaded member functions

Hello, > It looks like the attached patch is the same as the original one? I’ve attache the wrong patch. Here is the right one: enable-if.diff Description: Binary data Thanks, Ettore Speziale___ cfe-commits mailing list cfe-commits@lists.llvm.org ht

Re: [PATCH] D13057: [mips] Relax -mnan=2008 acceptance to permit MIPS32R2 and MIPS64R2.

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

[PATCH] D13057: [mips] Relax -mnan=2008 acceptance to permit MIPS32R2 and MIPS64R2.

dsanders created this revision. dsanders added a reviewer: atanasyan. dsanders added a subscriber: cfe-commits. Strictly speaking, the MIPS*R2 ISA's should not permit -mnan=2008 since this feature was added in MIPS*R3. However, other toolchains permit this and we should do the same. http://review

  1   2   >