r246422 - [OPENMP 4.0] Codegen for array sections.

Author: abataev Date: Mon Aug 31 02:32:19 2015 New Revision: 246422 URL: http://llvm.org/viewvc/llvm-project?rev=246422=rev Log: [OPENMP 4.0] Codegen for array sections. Added codegen for array section in 'depend' clause of 'task' directive. It emits to pointers, one for the begin of array

Re: [PATCH] D12512: [libcxxabi] Manually align pointers in __cxa_allocate_exception - Fixes PR24604

compnerd added a comment. While I can see the argument you raise, the saving grace here is that this is buried in libc++abi. The only real piece of libc++abi of real use to users is `__cxa_demangle` (which is only relevant for Windows, and Darwin where you have two level namespaces). For

r246524 - Use a more appropriate way of writing 1.

Author: rsmith Date: Mon Aug 31 21:02:09 2015 New Revision: 246524 URL: http://llvm.org/viewvc/llvm-project?rev=246524=rev Log: Use a more appropriate way of writing 1. Modified: cfe/trunk/lib/Serialization/ASTReader.cpp Modified: cfe/trunk/lib/Serialization/ASTReader.cpp URL:

Re: [PATCH] D12512: [libcxxabi] Manually align pointers in __cxa_allocate_exception - Fixes PR24604

EricWF added a comment. In http://reviews.llvm.org/D12512#236988, @majnemer wrote: > In http://reviews.llvm.org/D12512#236987, @EricWF wrote: > > > In http://reviews.llvm.org/D12512#236984, @majnemer wrote: > > > > > Wouldn't this change be problematic if you threw to code which was > > >

Re: [PATCH] D12511: [LTO] Pass -no-zero-initialized-in-bss on linker command line

ahatanak added a comment. If it's important to be able to compile one file with -fno-zero-initialized-in-bss and another without the option, we could add a bit to GlobalVariable that indicates it shouldn't be go into the bss section. Is that what you are suggesting? If we are going to take

[libcxx] r246445 - Remove unused code. NFC

Author: marshall Date: Mon Aug 31 09:43:41 2015 New Revision: 246445 URL: http://llvm.org/viewvc/llvm-project?rev=246445=rev Log: Remove unused code. NFC Modified: libcxx/trunk/include/type_traits Modified: libcxx/trunk/include/type_traits URL:

[clang-tools-extra] r246446 - [clang-tidy] misc-assert-side-effect: support assert macros defined through other macros

Author: alexfh Date: Mon Aug 31 09:47:14 2015 New Revision: 246446 URL: http://llvm.org/viewvc/llvm-project?rev=246446=rev Log: [clang-tidy] misc-assert-side-effect: support assert macros defined through other macros Modified:

Re: r246384 - [OpenMP] Make the filetered clause iterator a real iterator and type safe.

Any chance this would be improved/further simplified by basing it on the adapter_iterator helper in LLVM's STL Utilities? On Sun, Aug 30, 2015 at 8:12 AM, Benjamin Kramer via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Author: d0k > Date: Sun Aug 30 10:12:28 2015 > New Revision: 246384 >

Re: [clang-tools-extra] r246444 - Using an early return as it is more clear; NFC.

Thanks! Could you do the same for the other files changed in that revision? On Mon, Aug 31, 2015 at 4:23 PM, Aaron Ballman via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Author: aaronballman > Date: Mon Aug 31 09:23:21 2015 > New Revision: 246444 > > URL:

Re: [clang-tools-extra] r246444 - Using an early return as it is more clear; NFC.

On Mon, Aug 31, 2015 at 11:18 AM, Alexander Kornienko wrote: > Thanks! Could you do the same for the other files changed in that revision? Can do. ~Aaron > > > On Mon, Aug 31, 2015 at 4:23 PM, Aaron Ballman via cfe-commits > wrote: >> >> Author:

[clang-tools-extra] r246447 - Using an early return as it is more clear; NFC.

Author: aaronballman Date: Mon Aug 31 10:28:57 2015 New Revision: 246447 URL: http://llvm.org/viewvc/llvm-project?rev=246447=rev Log: Using an early return as it is more clear; NFC. Modified: clang-tools-extra/trunk/clang-tidy/misc/InaccurateEraseCheck.cpp

Re: [PATCH] D12446: [PATCH] Enable clang-tidy misc-static-assert for C11

alexfh added inline comments. Comment at: test/clang-tidy/misc-static-assert-c99.c:2 @@ +1,3 @@ +// RUN: %python %S/check_clang_tidy.py %s misc-static-assert %t -- -std=c99 + +void abort() {} aaron.ballman wrote: > I am not certain how to accomplish this with the

Re: [PATCH] D12446: [PATCH] Enable clang-tidy misc-static-assert for C11

klimek added a subscriber: klimek. Comment at: test/clang-tidy/misc-static-assert-c99.c:2 @@ +1,3 @@ +// RUN: %python %S/check_clang_tidy.py %s misc-static-assert %t -- -std=c99 + +void abort() {} aaron.ballman wrote: > I am not certain how to accomplish this

Re: [PATCH] D12446: [PATCH] Enable clang-tidy misc-static-assert for C11

klimek added inline comments. Comment at: test/clang-tidy/misc-static-assert-c99.c:1 @@ +1,2 @@ +// RUN: %python %S/check_clang_tidy.py %s misc-static-assert %t -- -std=c99 + alexfh wrote: > Instead of duplicating the test, you could add a second run line in the

Re: [PATCH] D12446: [PATCH] Enable clang-tidy misc-static-assert for C11

aaron.ballman added inline comments. Comment at: test/clang-tidy/misc-static-assert-c99.c:2 @@ +1,3 @@ +// RUN: %python %S/check_clang_tidy.py %s misc-static-assert %t -- -std=c99 + +void abort() {} alexfh wrote: > klimek wrote: > > aaron.ballman wrote: > > > I

Re: [PATCH] D12458: [clang] add __builtin_unpredictable and convert to metadata

kparzysz added a comment. LGTM Comment at: lib/CodeGen/CodeGenFunction.cpp:1238 @@ -1217,3 +1237,3 @@ } - Builder.CreateCondBr(CondV, TrueBlock, FalseBlock, Weights); + Builder.CreateCondBr(CondV, TrueBlock, FalseBlock, Weights, Unpredictable); } spatel

Re: [PATCH] D12473: [clang-tidy] Add old style function check

alexfh added a comment. A high-level comment: It seems that the scope of the check is artificially made too narrow. The check could actually look at any POD variables declared unnecessarily far from their initialization and usages. And here the value of the check would also be much higher, if

Re: [PATCH] D12407: [clang-format-vs] Add an option to reformat source code when file is saved to disk

hans added a comment. I also don't know anything about how this plugin works, I just build it :) I'm hoping Manuel can take a look. http://reviews.llvm.org/D12407 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

r246452 - [OpenMP] base specific_clause_iterator on iterator_adaptor_base.

Author: d0k Date: Mon Aug 31 11:45:35 2015 New Revision: 246452 URL: http://llvm.org/viewvc/llvm-project?rev=246452=rev Log: [OpenMP] base specific_clause_iterator on iterator_adaptor_base. Removes some boilerplate code. No functionality change intended. Modified:

Re: [PATCH] D12407: [clang-format-vs] Add an option to reformat source code when file is saved to disk

I'll need to try to build / install it. I'll take a couple of days, I'm currently swamped. Hope that's ok. On Mon, Aug 31, 2015 at 6:25 PM Hans Wennborg wrote: > hans added a comment. > > I also don't know anything about how this plugin works, I just build it :) > I'm hoping

Re: [PATCH] D12407: [clang-format-vs] Add an option to reformat source code when file is saved to disk

klimek added a subscriber: klimek. klimek added a comment. I'll need to try to build / install it. I'll take a couple of days, I'm currently swamped. Hope that's ok. http://reviews.llvm.org/D12407 ___ cfe-commits mailing list

Re: r246384 - [OpenMP] Make the filetered clause iterator a real iterator and type safe.

On Mon, Aug 31, 2015 at 5:12 PM, David Blaikie wrote: > Any chance this would be improved/further simplified by basing it on the > adapter_iterator helper in LLVM's STL Utilities? > Fair enough, r246452. I was a bit nervous because this iterator overrides operator++ but

Re: r246384 - [OpenMP] Make the filetered clause iterator a real iterator and type safe.

On Mon, Aug 31, 2015 at 9:48 AM, Benjamin Kramer wrote: > > > On Mon, Aug 31, 2015 at 5:12 PM, David Blaikie wrote: > >> Any chance this would be improved/further simplified by basing it on the >> adapter_iterator helper in LLVM's STL Utilities? >> > >

Re: [PATCH][Solaris] Clang/Driver, stop hardcoding GCC paths in crt/ld.so lookup

Do you have a version with the last suggestions? I lgtmed it conditional on it. Do you need someone to commit it for you? On Aug 23, 2015 7:07 PM, "Rafael Espíndola" wrote: > SolarisScanLibDirForGCCTriple should start with a lower case. Starting > it with "scan"

Re: [PATCH] D12435: [Concepts] Add diagnostic; invalid specifier on function or variable concept declaration

nwilson updated this revision to Diff 33587. nwilson added a comment. Change the wording of diagnostic message and quote the invalid specifier. Add 'friend' as an invalid specifier and diagnose function concepts when 'friend' is specified. Reorder the list of invalid specifiers and

Re: [PATCH] D12462: [PATCH] [clang-tidy] Add inconsistent declaration parameter name check

piotrdz added a comment. @alexfh: Thanks for reviewing my code. The amount of comments makes it pretty clear that it's my first contribution here :-). But no worries, I'll address each issue. > This check seems pretty similar to the check implemented in >

Re: [PATCH] D12407: [clang-format-vs] Add an option to reformat source code when file is saved to disk

berenm added a comment. Sure, no worries. http://reviews.llvm.org/D12407 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r246455 - [CUDA] fix codegen for __nvvm_atom_min/max_gen_u*

Author: jingyue Date: Mon Aug 31 12:25:51 2015 New Revision: 246455 URL: http://llvm.org/viewvc/llvm-project?rev=246455=rev Log: [CUDA] fix codegen for __nvvm_atom_min/max_gen_u* Summary: Clang should emit "atomicrmw umin/umax" instead of "atomicrmw min/max". Reviewers: eliben, tra

Re: [PATCH][Solaris] Clang/Driver, stop hardcoding GCC paths in crt/ld.so lookup

Oops, missed this. Here is the updated patch. And yes please, commit the patch for me. Cheers, Xan On Mon, Aug 31, 2015 at 01:07:35PM -0400, Rafael Espíndola wrote: > Do you have a version with the last suggestions? I lgtmed it conditional on > it. Do you need someone to commit it for you? >

r246468 - Pull the target attribute parsing out of CGCall and onto TargetInfo.

Author: echristo Date: Mon Aug 31 13:39:22 2015 New Revision: 246468 URL: http://llvm.org/viewvc/llvm-project?rev=246468=rev Log: Pull the target attribute parsing out of CGCall and onto TargetInfo. Also: - Add a typedef to make working with the result easier. - Update callers to use the new

r246469 - [MS ABI] Correctly mangle classes without names for linkage purposes

Author: majnemer Date: Mon Aug 31 13:48:39 2015 New Revision: 246469 URL: http://llvm.org/viewvc/llvm-project?rev=246469=rev Log: [MS ABI] Correctly mangle classes without names for linkage purposes A class without a name for linkage purposes gets a name along the lines of where foo is either

Re: [PATCH] D12359: New warning -Wnonconst-parameter when a pointer parameter can be const

danielmarjamaki added a comment. In http://reviews.llvm.org/D12359#236334, @aaron.ballman wrote: > I have concerns about this being a frontend warning. The true positive rate > seems rather high given the benign nature of the diagnostic, and the false > negative rate is quite high. This seems

Re: [PATCH] D8178: [libcxx] Fix PR22806 - Tuple incorrectly selects copy/move constructor when storing nested tuple-like types.

EricWF added a comment. In http://reviews.llvm.org/D8178#236180, @EricWF wrote: > I think this patch is actually pretty close to correct but thinking about > tuple hurts my brain. > > @mclow.lists can you start reviewing this when you have time. @mclow.lists Nevermind I think I have a better

Re: [PATCH] D11832: [Patch] [Analyzer] false positive: Potential leak connected with memcpy (PR 22954)

xazax.hun added a comment. Hi! With this patch committed I noticed a regression in the static analyzer. I analyzed openssl-1.0.0d (using the test suite in utils/analyzer/SATestBuild.py). I got the following assertion error: (lldb) bt - thread #1: tid = 0xa1fcb, 0x7fff943e50ae

r246473 - Stop hardcoding GCC paths in crt/ld.so lookup.

Author: rafael Date: Mon Aug 31 14:17:51 2015 New Revision: 246473 URL: http://llvm.org/viewvc/llvm-project?rev=246473=rev Log: Stop hardcoding GCC paths in crt/ld.so lookup. This patch refactors the code to use the GCC installation detector (modified so that it works in Solaris), and uses

Re: [PATCH] D12359: New warning -Wnonconst-parameter when a pointer parameter can be const

On Mon, Aug 31, 2015 at 2:58 PM, Daniel Marjamäki wrote: > danielmarjamaki added a comment. > > In http://reviews.llvm.org/D12359#236334, @aaron.ballman wrote: > >> I have concerns about this being a frontend warning. The true positive rate >> seems rather high

Re: [PATCH] D11815: Pass subtarget feature "force-align-stack"

ahatanak added a comment. In http://reviews.llvm.org/D11815#236368, @vkalintiris wrote: > In http://reviews.llvm.org/D11815#235394, @ahatanak wrote: > > > The cc1 option "-mstackrealign" now means "force stack realignment" rather > > than "allow stack realignment" and causes function attribute

Re: [PATCH] D12002: Initial WebAssembly support in clang

sunfish updated this revision to Diff 33611. sunfish marked 2 inline comments as done. sunfish added a comment. Herald added a subscriber: dschuff. Minor updates to keep the patch building and working as code changes around it. Also enabled -mthread-model. Repository: rL LLVM

Re: [PATCH] D11832: [Patch] [Analyzer] false positive: Potential leak connected with memcpy (PR 22954)

I also got crash with this commit: 0x00f52d4b in (anonymous namespace)::CStringChecker::InvalidateBuffer(clang::ento::CheckerContext&, llvm::IntrusiveRefCntPtr, clang::Expr const*, clang::ento::SVal, bool, clang::Expr const*) () at

Re: [PATCH] D12444: [Sema] Avoid crash on tag-type mismatch (Fixes PR24610)

majnemer added a comment. I think it'd be nice if we keep the assert but make `setTagNameForLinkagePurposes` early-return if `TagFromDeclSpec->isInvalidDecl()` returns true. How does that sound to you? http://reviews.llvm.org/D12444 ___

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

ksuther added a reviewer: djasper. ksuther added a comment. This is my first commit and I'm trying to figure out the system, hope I'm doing this right. http://reviews.llvm.org/D12489 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

Re: [PATCH] D12462: [PATCH] [clang-tidy] Add inconsistent declaration parameter name check

piotrdz marked 7 inline comments as done. Comment at: clang-tidy/readability/InconsistentDeclarationParameterNameCheck.cpp:23 @@ +22,3 @@ + +struct CheckResult { + CheckResult(bool HasInconsistentParams, alexfh wrote: > Maybe use `llvm::Optional`? Or, if you

r246479 - Revert r246345 until an assertion is fixed.

Author: xazax Date: Mon Aug 31 15:10:35 2015 New Revision: 246479 URL: http://llvm.org/viewvc/llvm-project?rev=246479=rev Log: Revert r246345 until an assertion is fixed. Removed: cfe/trunk/test/Analysis/pr22954.c Modified:

Re: [PATCH] D12462: [PATCH] [clang-tidy] Add inconsistent declaration parameter name check

piotrdz updated this revision to Diff 33619. piotrdz marked 3 inline comments as done. piotrdz added a comment. Applied fixes for most issues found in review. http://reviews.llvm.org/D12462 Files: clang-tidy/readability/CMakeLists.txt

Re: [PATCH] D12462: [PATCH] [clang-tidy] Add inconsistent declaration parameter name check

piotrdz marked an inline comment as done. piotrdz added a comment. Now this issue with templates is a bit difficult for me. I tried everything that seems to have made sense, but I still get output like this:

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

ksuther created this revision. ksuther added a reviewer: djasper. ksuther added a subscriber: cfe-commits. Herald added a subscriber: klimek. Fixes this bug: https://llvm.org/bugs/show_bug.cgi?id=22647 The following dictionary was getting formatted oddly: NSDictionary *query = @{ (__bridge

Re: [PATCH] D11832: [Patch] [Analyzer] false positive: Potential leak connected with memcpy (PR 22954)

xazax.hun added a comment. I reverted the commit until this assertion is fixed. Steps to reproduce: Download the following preprocessed file: F804743: clang_crash_7QnDaH.i Execute the analyzer on that one: clang -cc1 -analyze -analyzer-checker=core

[PATCH] D12502: [libcxx] Better constain tuples constructors -- Fix PR23256 and PR22806

EricWF created this revision. EricWF added reviewers: mclow.lists, eugenis, ldionne. EricWF added a subscriber: cfe-commits. This patch fixes the following bugs: * PR23256 tuple is_constructible attempts tuple to A conversion: * Fix this by disabling the UTypes... overloads using the

Re: [PATCH] D9085: Fix tuple to A conversion in SFINAE for tuple(_Up...) constructor

EricWF added a comment. Another possible fix up for review as http://reviews.llvm.org/D12502. Repository: rL LLVM http://reviews.llvm.org/D9085 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

Re: [PATCH] D8178: [libcxx] Fix PR22806 - Tuple incorrectly selects copy/move constructor when storing nested tuple-like types.

EricWF accepted this revision. EricWF added a reviewer: EricWF. EricWF added a comment. This revision is now accepted and ready to land. Abandoning for patch http://reviews.llvm.org/D12502. http://reviews.llvm.org/D8178 ___ cfe-commits mailing list

Re: [PATCH] D8178: [libcxx] Fix PR22806 - Tuple incorrectly selects copy/move constructor when storing nested tuple-like types.

EricWF abandoned this revision. EricWF added a comment. Actually abandoning. Sorry for the spam. http://reviews.llvm.org/D8178 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH][Solaris] Clang/Driver, stop hardcoding GCC paths in crt/ld.so lookup

r246473. On 31 August 2015 at 13:30, Xan López wrote: > Oops, missed this. Here is the updated patch. > > And yes please, commit the patch for me. > > Cheers, > > Xan > > On Mon, Aug 31, 2015 at 01:07:35PM -0400, Rafael Espíndola wrote: >> Do you have a version with the last

Re: [PATCH] D11948: Add some macros to abstract marking of parameters as "not null", and use them in

mclow.lists updated this revision to Diff 33624. mclow.lists added a comment. Fixed a typo in the patch for `strncmp`, and added a macro `_LIBCPP_HAS_NO_NONNULL` to allow people to disable these checks. Marked them as disabled for non-clang and non-gcc compilers.

Re: [PATCH] D11963: Create a __config_site file to capture configuration decisions.

EricWF added a comment. In http://reviews.llvm.org/D11963#236376, @jroelofs wrote: > In http://reviews.llvm.org/D11963#236045, @EricWF wrote: > > > So that's not what happens but that's because libc++ builds using the > > headers in the source directory. During the build all of the macros in

Re: [PATCH] D11740: ABI versioning macros for libc++

EricWF added a comment. FYI I'm focusing on getting http://reviews.llvm.org/D11963 in so that it stops blocking this patch. http://reviews.llvm.org/D11740 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

Re: [PATCH] D11740: ABI versioning macros for libc++

EricWF added a comment. FYI I'm focusing on getting http://reviews.llvm.org/D11963 in so that it stops blocking this patch. http://reviews.llvm.org/D11740 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

Re: [PATCH] D12087: always_inline codegen rewrite

rnk added a subscriber: rnk. Comment at: lib/CodeGen/CodeGenModule.cpp:469-470 @@ +468,4 @@ + llvm::LLVMContext = getModule().getContext(); + llvm::Function *StubFn = + llvm::Function::Create(FT, Fn->getLinkage(), Name, ()); + assert(StubFn->getName() == Name && "name

Re: [PATCH] D11963: Create a __config_site file to capture configuration decisions.

jroelofs added a comment. Without the context of this review, this strategy probably won't make any sense. So it's probably worth capturing these notes in a comment that describes why we're doing it that way. http://reviews.llvm.org/D11963 ___

Re: [PATCH] D12446: [PATCH] Enable clang-tidy misc-static-assert for C11

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

r246492 - Fix CHECK directives that weren't checking.

Author: hans Date: Mon Aug 31 16:48:52 2015 New Revision: 246492 URL: http://llvm.org/viewvc/llvm-project?rev=246492=rev Log: Fix CHECK directives that weren't checking. Modified: cfe/trunk/test/CodeGen/builtin-cpu-supports.c cfe/trunk/test/CodeGen/builtins-ppc-crypto.c

[clang-tools-extra] r246494 - Help the clang-tidy helper script to understand C files better.

Author: aaronballman Date: Mon Aug 31 16:53:55 2015 New Revision: 246494 URL: http://llvm.org/viewvc/llvm-project?rev=246494=rev Log: Help the clang-tidy helper script to understand C files better. Modified: clang-tools-extra/trunk/test/clang-tidy/check_clang_tidy.py Modified:

[clang-tools-extra] r246495 - Allow the static assert clang-tidy checker to run over C code.

Author: aaronballman Date: Mon Aug 31 16:54:42 2015 New Revision: 246495 URL: http://llvm.org/viewvc/llvm-project?rev=246495=rev Log: Allow the static assert clang-tidy checker to run over C code. Added: clang-tools-extra/trunk/test/clang-tidy/misc-static-assert.c Modified:

Re: [PATCH] D12301: [PATCH] New checker for UB in handler of a function-try-block

rsmith accepted this revision. rsmith added a comment. This revision is now accepted and ready to land. Couple of thoughts, but LGTM. Comment at: lib/Sema/SemaExprMember.cpp:889 @@ +888,3 @@ +S = S->getParent(); + } while (S != S->getFnParent()); + return false;

Re: [PATCH] D12002: Initial WebAssembly support in clang

sunfish updated this revision to Diff 33637. sunfish added a comment. Updated to address review comments: - added comments - simplified Triple validation code - removed -mcpu="native" code Repository: rL LLVM http://reviews.llvm.org/D12002 Files:

r246497 - [modules] Rework serialized DeclContext lookup table management. Instead of

Author: rsmith Date: Mon Aug 31 17:17:11 2015 New Revision: 246497 URL: http://llvm.org/viewvc/llvm-project?rev=246497=rev Log: [modules] Rework serialized DeclContext lookup table management. Instead of walking the loaded ModuleFiles looking for lookup tables for the context, store them all in

Re: [PATCH] D12471: Correct documentation for numSelectorArgs matcher

kastiglione added inline comments. Comment at: include/clang/ASTMatchers/ASTMatchers.h:2140-2153 @@ -2139,16 +2139,16 @@ /// \brief Matches when the selector has the specified number of arguments /// -/// matcher = objCMessageExpr(numSelectorArgs(1)); +/// matcher =

r246504 - [modules] Add some missing blockinfo records.

Author: rsmith Date: Mon Aug 31 17:43:10 2015 New Revision: 246504 URL: http://llvm.org/viewvc/llvm-project?rev=246504=rev Log: [modules] Add some missing blockinfo records. Modified: cfe/trunk/include/clang/Serialization/ASTBitCodes.h cfe/trunk/lib/Serialization/ASTWriter.cpp Modified:

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

eliben added inline comments. Comment at: include/clang/Sema/Sema.h:8599 @@ +8598,3 @@ +CFP_Never, // Invalid caller/callee combination. +CFP_LastResort, // same as CFP_Never or CFP_Fallback, depending on +// -fcuda-disable-target-call-checks

[PATCH] D12505: [X86] Set MaxVectorAlign for non-Darwin targets as well.

ab created this revision. ab added reviewers: echristo, rnk. ab added subscribers: alexr, dim, dschuff, tberghammer, cfe-commits. Follow-up to r246229: this isn't Darwin-specific, so let's enable it for all X86 targets. Quoting r160209, which introduced it: ``` Currently, if no aligned

Re: r246468 - Pull the target attribute parsing out of CGCall and onto TargetInfo.

This change makes Basic depend on AST. The immediate problem I'm hitting is clang-format (which uses Basic but not AST) failing to link on Windows [1], but I suspect this could cause problems elsewhere too. Is there some way to avoid this dependency? [1].

Re: [PATCH] D11361: [OpenMP] Target directive host codegen

sfantao updated this revision to Diff 33640. sfantao added a comment. Address last review comments. http://reviews.llvm.org/D11361 Files: lib/CodeGen/CGOpenMPRuntime.cpp lib/CodeGen/CGOpenMPRuntime.h lib/CodeGen/CGStmtOpenMP.cpp test/OpenMP/target_codegen.cpp Index:

Re: [PATCH] D11361: [OpenMP] Target directive host codegen

sfantao added inline comments. Comment at: lib/CodeGen/CGOpenMPRuntime.cpp:2991-3005 @@ +2990,17 @@ + +/// \brief Values for bit flags used to specify the mapping type for +/// offloading. +enum OpenMPOffloadMappingFlags { + /// \brief Allocate memory on the device and move data

Re: r246468 - Pull the target attribute parsing out of CGCall and onto TargetInfo.

Aha. This is what's causing Siva Chandra's problem. Hrm, I'll see what I can figure out. I'll revert until then or talk to Richard. -eric On Mon, Aug 31, 2015 at 4:02 PM Hans Wennborg wrote: > This change makes Basic depend on AST. > > The immediate problem I'm hitting is

r246508 - Revert "Pull the target attribute parsing out of CGCall and onto TargetInfo."

Author: echristo Date: Mon Aug 31 18:19:55 2015 New Revision: 246508 URL: http://llvm.org/viewvc/llvm-project?rev=246508=rev Log: Revert "Pull the target attribute parsing out of CGCall and onto TargetInfo." This reverts commit r246468 while we figure out what to do about Basic and AST.

Re: r246468 - Pull the target attribute parsing out of CGCall and onto TargetInfo.

On Mon, Aug 31, 2015 at 11:39 AM, Eric Christopher via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Author: echristo > Date: Mon Aug 31 13:39:22 2015 > New Revision: 246468 > > URL: http://llvm.org/viewvc/llvm-project?rev=246468=rev > Log: > Pull the target attribute parsing out of CGCall

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

tra added inline comments. Comment at: include/clang/Sema/Sema.h:8599 @@ +8598,3 @@ +CFP_Never, // Invalid caller/callee combination. +CFP_LastResort, // same as CFP_Never or CFP_Fallback, depending on +// -fcuda-disable-target-call-checks option

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

tra updated this revision to Diff 33654. tra marked 2 inline comments as done. tra added a comment. addressed eliben@'s comments. http://reviews.llvm.org/D12453 Files: include/clang/Basic/Builtins.h include/clang/Basic/LangOptions.def include/clang/Driver/CC1Options.td

Re: [PATCH] D11844: [Modules] More descriptive diagnostics for misplaced import directive

silvas added a comment. ping? http://reviews.llvm.org/D11844 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D12511: [LTO] Pass -no-zero-initialized-in-bss on linker command line

ahatanak created this revision. ahatanak added reviewers: echristo, dexonsmith. ahatanak added a subscriber: cfe-commits. Clang driver option -fno-zero-initialized-in-bss is used to prevent global variables that are initialized to zero from being placed in the bss section, but is currently

[libcxxabi] r246516 - Convert LIBCXXABI_BUILD_32_BITS to LLVM_BUILD_32_BITS.

Author: ericwf Date: Mon Aug 31 20:02:06 2015 New Revision: 246516 URL: http://llvm.org/viewvc/llvm-project?rev=246516=rev Log: Convert LIBCXXABI_BUILD_32_BITS to LLVM_BUILD_32_BITS. Modified: libcxxabi/trunk/test/CMakeLists.txt libcxxabi/trunk/test/lit.site.cfg.in Modified:

[PATCH] D12512: [libcxxabi] Manually align pointers in __cxa_allocate_exception - Fixes PR24604

EricWF created this revision. EricWF added reviewers: mclow.lists, danalbert, compnerd, majnemer. EricWF added a subscriber: cfe-commits. In 32 bit builds the __cxa_exception class has an alignment requirement greater than the maximal alignment supported by malloc. If this is the case we need to

r246520 - Fix path seperator for Windows.

Author: yrnkrn Date: Mon Aug 31 20:29:13 2015 New Revision: 246520 URL: http://llvm.org/viewvc/llvm-project?rev=246520=rev Log: Fix path seperator for Windows. Modified: cfe/trunk/test/Driver/solaris-ld.c Modified: cfe/trunk/test/Driver/solaris-ld.c URL:

Re: [PATCH] D12511: [LTO] Pass -no-zero-initialized-in-bss on linker command line

Looks like something that should be in the bitcode, no? What happens if one compile unit has it and another one doesn't? On Aug 31, 2015 8:50 PM, "Akira Hatanaka via cfe-commits" < cfe-commits@lists.llvm.org> wrote: > ahatanak created this revision. > ahatanak added reviewers: echristo,

Re: [PATCH] D12511: [LTO] Pass -no-zero-initialized-in-bss on linker command line

rafael added a subscriber: rafael. rafael added a comment. Looks like something that should be in the bitcode, no? What happens if one compile unit has it and another one doesn't? http://reviews.llvm.org/D12511 ___ cfe-commits mailing list

[libunwind] r246413 - [libunwind] Add support for OpenRISC 1000.

Author: whitequark Date: Mon Aug 31 00:26:37 2015 New Revision: 246413 URL: http://llvm.org/viewvc/llvm-project?rev=246413=rev Log: [libunwind] Add support for OpenRISC 1000. This patch makes no assumptions on ABI past the ABI defined in the OpenRISC 1000 spec except that the DWARF register

Re: [PATCH] D11976: [libclang] Return deduced type for auto type, not the one written in the source.

skalinichev added a comment. Yes, please submit it. http://reviews.llvm.org/D11976 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r246430 - AVX-512: Changed cnt parameter in valignq/d intrinsic from i8 to i32 according to the Intel Spec

Author: ibreger Date: Mon Aug 31 06:15:06 2015 New Revision: 246430 URL: http://llvm.org/viewvc/llvm-project?rev=246430=rev Log: AVX-512: Changed cnt parameter in valignq/d intrinsic from i8 to i32 according to the Intel Spec Differential Revision: http://reviews.llvm.org/D12274 Modified:

r246526 - Don't use fprintf to emit this diagnostic!

Author: rsmith Date: Mon Aug 31 21:35:58 2015 New Revision: 246526 URL: http://llvm.org/viewvc/llvm-project?rev=246526=rev Log: Don't use fprintf to emit this diagnostic! Added: cfe/trunk/test/PCH/datetime.c Modified: cfe/trunk/include/clang/Basic/DiagnosticSerializationKinds.td

Re: [PATCH] D12511: [LTO] Pass -no-zero-initialized-in-bss on linker command line

On 31 August 2015 at 22:45, Akira Hatanaka wrote: > ahatanak added a comment. > > If it's important to be able to compile one file with > -fno-zero-initialized-in-bss and another without the option, we could add a > bit to GlobalVariable that indicates it shouldn't be go

Re: [PATCH] D12471: Correct documentation for numSelectorArgs matcher

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

[clang-tools-extra] r246437 - [clang-tidy] Move misc-use-override and readability-shrink-to-fit to "modernize/"

Author: alexfh Date: Mon Aug 31 08:17:43 2015 New Revision: 246437 URL: http://llvm.org/viewvc/llvm-project?rev=246437=rev Log: [clang-tidy] Move misc-use-override and readability-shrink-to-fit to "modernize/" These checks are focusing on migrating the code from C++98/03 to C++11, so they

Re: [PATCH] D11815: Pass subtarget feature "force-align-stack"

vkalintiris added a subscriber: vkalintiris. vkalintiris added a comment. In http://reviews.llvm.org/D11815#235394, @ahatanak wrote: > The cc1 option "-mstackrealign" now means "force stack realignment" rather > than "allow stack realignment" and causes function attribute "stackrealign" > to

Re: [PATCH] D12359: New warning -Wnonconst-parameter when a pointer parameter can be const

danielmarjamaki updated this revision to Diff 33575. danielmarjamaki added a comment. Fixed 2 false positives found in python source code. http://reviews.llvm.org/D12359 Files: include/clang/AST/DeclBase.h include/clang/Basic/DiagnosticGroups.td include/clang/Basic/DiagnosticSemaKinds.td

Re: [PATCH] D12301: [PATCH] New checker for UB in handler of a function-try-block

aaron.ballman added a comment. In http://reviews.llvm.org/D12301#232100, @aaron.ballman wrote: > This patch completely reworks the way the warning is implemented by moving it > into the frontend instead of clang-tidy. Ping ~Aaron http://reviews.llvm.org/D12301

r246438 - PR17829: Proper diagnostic of mangled names conflicts

Author: asbokhan Date: Mon Aug 31 08:20:44 2015 New Revision: 246438 URL: http://llvm.org/viewvc/llvm-project?rev=246438=rev Log: PR17829: Proper diagnostic of mangled names conflicts Proper diagnostic and resolution of mangled names conflicts between C++ methods and C functions. This patch

Re: [PATCH] D11297: PR17829: Functions declared extern "C" with a name matching a mangled C++ function are allowed

andreybokhanko added a comment. In http://reviews.llvm.org/D11297#235525, @rjmccall wrote: > Yes, please make it an error. Done. John, thank you for all your patience and explanations! -- I understand that this particular review and patch author required more than the usual measure. :-( >

Re: [PATCH] D12446: [PATCH] Enable clang-tidy misc-static-assert for C11

aaron.ballman updated this revision to Diff 33579. aaron.ballman added a comment. No longer worrying about static_assert() vs _Static_assert(). Corrected Python script. http://reviews.llvm.org/D12446 Files: clang-tidy/misc/StaticAssertCheck.cpp test/clang-tidy/check_clang_tidy.py

Re: [PATCH] D12446: [PATCH] Enable clang-tidy misc-static-assert for C11

aaron.ballman marked an inline comment as done. Comment at: test/clang-tidy/misc-static-assert-c99.c:2 @@ +1,3 @@ +// RUN: %python %S/check_clang_tidy.py %s misc-static-assert %t -- -std=c99 + +void abort() {} I am not certain how to accomplish this with the

[clang-tools-extra] r246444 - Using an early return as it is more clear; NFC.

Author: aaronballman Date: Mon Aug 31 09:23:21 2015 New Revision: 246444 URL: http://llvm.org/viewvc/llvm-project?rev=246444=rev Log: Using an early return as it is more clear; NFC. Modified: clang-tools-extra/trunk/clang-tidy/misc/AssignOperatorSignatureCheck.cpp Modified:

Re: [PATCH] D12458: [clang] add __builtin_unpredictable and convert to metadata

kparzysz added inline comments. Comment at: lib/CodeGen/CodeGenFunction.cpp:1238 @@ -1217,3 +1237,3 @@ } - Builder.CreateCondBr(CondV, TrueBlock, FalseBlock, Weights); + Builder.CreateCondBr(CondV, TrueBlock, FalseBlock, Weights, Unpredictable); } Is the

  1   2   >