r244843 - Wdeprecated: LambdaScopeInfos need to be copied, so make the ScopeInfo hierarchy safely copyable

2015-08-12 Thread David Blaikie via cfe-commits
Author: dblaikie Date: Wed Aug 12 18:59:02 2015 New Revision: 244843 URL: http://llvm.org/viewvc/llvm-project?rev=244843&view=rev Log: Wdeprecated: LambdaScopeInfos need to be copied, so make the ScopeInfo hierarchy safely copyable Making the base class's copy ctor protected and the derived clas

Re: [PATCH] D10356: scan-build: Add --analyzer-target option

2015-08-12 Thread Honggyu Kim via cfe-commits
honggyu.kim accepted this revision. honggyu.kim added a reviewer: honggyu.kim. honggyu.kim added a comment. This revision is now accepted and ready to land. In http://reviews.llvm.org/D10356#219772, @krememek wrote: > Committed r244400 Thanks for accepting it! http://reviews.llvm.org/D10356

Re: r220305 - Driver: Move crash report command mangling into Command::Print

2015-08-12 Thread Justin Bogner via cfe-commits
On Wed, Aug 12, 2015 at 3:09 PM, David Blaikie wrote: > On Tue, Oct 21, 2014 at 10:24 AM, Justin Bogner > wrote: >> Author: bogner >> Date: Tue Oct 21 12:24:44 2014 >> New Revision: 220305 >> >> URL: http://llvm.org/viewvc/llvm-project?rev=220305&view=rev >> Log: >> Driver: Move crash report comm

[PATCH] D12000: Bugfix - Clang handles __builtin_object_size in wrong way

2015-08-12 Thread George Burgess IV via cfe-commits
george.burgess.iv created this revision. george.burgess.iv added a subscriber: cfe-commits. Attached is a fix for https://llvm.org/bugs/show_bug.cgi?id=15212 . Summary: Tighten up some of the results for __builtin_object_size(ptr, type) when given type == 1 or type == 3 + fixed a bug where we wo

Re: [PATCH] D11808: Driver: Fix include and lib dirs when not using gcc under mingw

2015-08-12 Thread Martell Malone via cfe-commits
martell updated this revision to Diff 32007. martell added a comment. Updated to address yaron's comments. Okay for me to merge now ? http://reviews.llvm.org/D11808 Files: lib/Driver/MinGWToolChain.cpp test/Driver/Inputs/mingw_clang_tree/mingw32/i686-w64-mingw32/include/.keep test/Driver

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

2015-08-12 Thread Hal Finkel via cfe-commits
- Original Message - > From: "Dan Albert via cfe-commits" > To: "Marshall Clow" > Cc: "Joerg Sonnenberger" , "cfe-commits" > > Sent: Wednesday, August 12, 2015 6:03:30 PM > Subject: Re: [PATCH] D11948: Add some macros to abstract marking of > parameters as "not null", and use them in >

Re: [PATCH] D11968: Add sanitizer blacklists to the rules generated with -M/-MM/-MD/-MMD.

2015-08-12 Thread Ivan Krasin via cfe-commits
krasin added a comment. Thank you, Peter. I will commit once I have restored my password (the email to Chris is sent) http://reviews.llvm.org/D11968 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listi

[PATCH] D12002: Initial WebAssembly support in clang

2015-08-12 Thread Dan Gohman via cfe-commits
sunfish created this revision. sunfish added subscribers: cfe-commits, jfb. sunfish set the repository for this revision to rL LLVM. This patch adds initial WebAssembly support in clang. The WebAssembly target is currently experimental. Repository: rL LLVM http://reviews.llvm.org/D12002 File

r244852 - Add SourceManager::dump() to dump the current set of SLocEntries.

2015-08-12 Thread Richard Smith via cfe-commits
Author: rsmith Date: Wed Aug 12 19:45:11 2015 New Revision: 244852 URL: http://llvm.org/viewvc/llvm-project?rev=244852&view=rev Log: Add SourceManager::dump() to dump the current set of SLocEntries. Modified: cfe/trunk/include/clang/Basic/SourceManager.h cfe/trunk/lib/Basic/SourceManager.

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

2015-08-12 Thread Sean Silva via cfe-commits
On Wed, Aug 12, 2015 at 2:43 PM, Richard Smith wrote: > On Wed, Aug 12, 2015 at 12:08 AM, Sean Silva > wrote: > >> silvas added a subscriber: silvas. >> >> >> Comment at: lib/Parse/Parser.cpp:2003 >> @@ +2002,3 @@ >> +Diag(Tok, diag::err_unexpected_module_start); >> +//

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

2015-08-12 Thread Sean Silva via cfe-commits
On Wed, Aug 12, 2015 at 6:00 PM, Sean Silva wrote: > > > On Wed, Aug 12, 2015 at 2:43 PM, Richard Smith > wrote: > >> On Wed, Aug 12, 2015 at 12:08 AM, Sean Silva >> wrote: >> >>> silvas added a subscriber: silvas. >>> >>> >>> Comment at: lib/Parse/Parser.cpp:2003 >>> @@ +2002,

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

2015-08-12 Thread Richard Smith via cfe-commits
On Wed, Aug 12, 2015 at 6:00 PM, Sean Silva wrote: > > > On Wed, Aug 12, 2015 at 2:43 PM, Richard Smith > wrote: > >> On Wed, Aug 12, 2015 at 12:08 AM, Sean Silva >> wrote: >> >>> silvas added a subscriber: silvas. >>> >>> >>> Comment at: lib/Parse/Parser.cpp:2003 >>> @@ +2002,

r244855 - Update for llvm api change.

2015-08-12 Thread Rafael Espindola via cfe-commits
Author: rafael Date: Wed Aug 12 20:07:06 2015 New Revision: 244855 URL: http://llvm.org/viewvc/llvm-project?rev=244855&view=rev Log: Update for llvm api change. Modified: cfe/trunk/tools/driver/driver.cpp Modified: cfe/trunk/tools/driver/driver.cpp URL: http://llvm.org/viewvc/llvm-project/c

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

2015-08-12 Thread JF Bastien via cfe-commits
jfb added a subscriber: rengolin. Comment at: include/clang/Basic/TargetCXXABI.h:166 @@ +165,3 @@ +case GenericMIPS: + // TODO: ARM-style pointers to member functions put the discriminator in + // the this adjustment, so they don't require functions to have any

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

2015-08-12 Thread Sean Silva via cfe-commits
On Wed, Aug 12, 2015 at 6:05 PM, Richard Smith wrote: > On Wed, Aug 12, 2015 at 6:00 PM, Sean Silva wrote: > >> >> >> On Wed, Aug 12, 2015 at 2:43 PM, Richard Smith >> wrote: >> >>> On Wed, Aug 12, 2015 at 12:08 AM, Sean Silva >>> wrote: >>> silvas added a subscriber: silvas. ==

Re: [PATCH] D7642: Introduce the idea of a minimum libc version

2015-08-12 Thread Saleem Abdulrasool via cfe-commits
compnerd added a comment. Okay, if thats the general pattern, then we can stick it in the triple. Though, that means that the use of the triple for determining the MS compatibility needs to be undone. If the version in the triple is the libc version, then it can differ from the MS compatibili

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

2015-08-12 Thread Dan Gohman via cfe-commits
sunfish planned changes to this revision. sunfish marked 3 inline comments as done. Comment at: lib/Basic/Targets.cpp:6935 @@ +6934,3 @@ +NoAsmVariants = true; +LongDoubleWidth = LongDoubleAlign = 64; +SuitableAlign = 128; jfb wrote: > That's already t

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

2015-08-12 Thread Dan Gohman via cfe-commits
sunfish updated this revision to Diff 32025. sunfish marked 3 inline comments as done. sunfish added a comment. Minor changes to address review comments. Repository: rL LLVM http://reviews.llvm.org/D12002 Files: include/clang/Basic/BuiltinsWebAssembly.def include/clang/Basic/TargetBuilti

r244867 - Add sanitizer blacklists to the rules generated with -M/-MM/-MD/-MMD.

2015-08-12 Thread Ivan Krasin via cfe-commits
Author: krasin Date: Wed Aug 12 23:04:37 2015 New Revision: 244867 URL: http://llvm.org/viewvc/llvm-project?rev=244867&view=rev Log: Add sanitizer blacklists to the rules generated with -M/-MM/-MD/-MMD. Summary: Clang sanitizers, such as AddressSanitizer, ThreadSanitizer, MemorySanitizer, Control

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

2015-08-12 Thread John McCall via cfe-commits
rjmccall added a comment. You only have one attempt to define the function here; I don't see the problem. Recall that I said to add a flag to getOrCreateLLVMFunction that says whether the caller intends to define the function. The rule should be that only callers that pass "true" should be al

Re: [PATCH] D11808: Driver: Fix include and lib dirs when not using gcc under mingw

2015-08-12 Thread Yaron Keren via cfe-commits
yaron.keren added a comment. Yes, go ahead. http://reviews.llvm.org/D11808 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r244869 - Disable failing check in bots from r244867.

2015-08-12 Thread Yaron Keren via cfe-commits
Author: yrnkrn Date: Thu Aug 13 01:12:49 2015 New Revision: 244869 URL: http://llvm.org/viewvc/llvm-project?rev=244869&view=rev Log: Disable failing check in bots from r244867. Modified: cfe/trunk/test/Frontend/dependency-gen.c Modified: cfe/trunk/test/Frontend/dependency-gen.c URL: http:/

Re: r244867 - Add sanitizer blacklists to the rules generated with -M/-MM/-MD/-MMD.

2015-08-12 Thread Yaron Keren via cfe-commits
Hi Ivan, CHECK-EIGHT is failing bots, see http://lab.llvm.org:8011/builders/clang-x86_64-ubuntu-gdb-75/builds/24306/steps/check-all/logs/FAIL%3A%20Clang%3A%3Adependency-gen.c I disabled it in r244869. Yaron 2015-08-13 7:04 GMT+03:00 Ivan Krasin via cfe-commits < cfe-commits@lists.llvm.org>:

<    1   2