[PATCH] D25669: [Driver] Simplify ToolChain::GetCXXStdlibType (NFC)

2016-11-09 Thread Jonas Hahnfeld via cfe-commits
Hahnfeld added a comment. ping https://reviews.llvm.org/D25669 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D26354: Use -fsanitize-recover instead of -mllvm -msan-keep-going: clang.

2016-11-09 Thread Eugene Zelenko via cfe-commits
Eugene.Zelenko closed this revision. Eugene.Zelenko added a comment. Committed in https://reviews.llvm.org/rL286148. https://reviews.llvm.org/D26354 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D25522: Remove warnings from google-benchmarks in libcxx

2016-11-09 Thread Eugene Zelenko via cfe-commits
Eugene.Zelenko closed this revision. Eugene.Zelenko added a comment. Committed in https://reviews.llvm.org/rL284179. https://reviews.llvm.org/D25522 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D26461: Tread TSan LLVM flags to driver: add TSan controlling flags to clang.

2016-11-09 Thread Aleksey Shlyapnikov via cfe-commits
alekseyshl added inline comments. Comment at: test/Driver/fsanitize.c:288 +// RUN: %clang -target x86_64-linux-gnu -fsanitize=thread -fsanitize-thread-data-races -fno-sanitize-thread-data-races %s -### 2>&1 | FileCheck %s --check-prefix=CHECK-TSAN-DATA-RACES-OFF +//

[clang-tools-extra] r286424 - [clang-tidy] Do not issue fix for functions that are referenced outside of callExpr

2016-11-09 Thread Felix Berger via cfe-commits
Author: flx Date: Wed Nov 9 19:28:22 2016 New Revision: 286424 URL: http://llvm.org/viewvc/llvm-project?rev=286424=rev Log: [clang-tidy] Do not issue fix for functions that are referenced outside of callExpr Summary: Suppress fixes for functions that are referenced within the compilation unit

[PATCH] D26203: [ClangTidy - performance-unnecessary-value-param]: Do not issue fix for functions that are referenced outside of callExpr

2016-11-09 Thread Felix Berger via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL286424: [clang-tidy] Do not issue fix for functions that are referenced outside of… (authored by flx). Changed prior to commit: https://reviews.llvm.org/D26203?vs=77343=77430#toc Repository: rL LLVM

[PATCH] D25857: [tsan][clang] Introduce a function attribute to disable TSan checking at run time

2016-11-09 Thread Anna Zaks via cfe-commits
zaks.anna marked 4 inline comments as done. zaks.anna added inline comments. Comment at: test/CodeGen/sanitize-thread-no-checking-at-run-time.m:1 +// RUN: %clang_cc1 -triple x86_64-apple-darwin -x objective-c++ -emit-llvm -o - %s | FileCheck -check-prefix=WITHOUT %s +// RUN:

[PATCH] D25764: Add end location of loop to loop metadata.

2016-11-09 Thread John McCall via cfe-commits
rjmccall accepted this revision. rjmccall added a comment. This revision is now accepted and ready to land. Looks good, thanks! https://reviews.llvm.org/D25764 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D23096: [Sema] Pass CombineWithOuterScope = true to constructor of LocalInstantiationScope

2016-11-09 Thread Akira Hatanaka via cfe-commits
ahatanak added a comment. Do you think we can transform the Exprs for the uninstantiated default arguments in TreeTransform::TransformLambdaExpr? I guess it would be much simpler than trying to make getTemplateInstantiationArgs return the correct template arguments.

[PATCH] D26461: Tread TSan LLVM flags to driver: add TSan controlling flags to clang.

2016-11-09 Thread Aleksey Shlyapnikov via cfe-commits
alekseyshl updated this revision to Diff 77432. alekseyshl marked 2 inline comments as done. alekseyshl added a comment. - Rename new options to better reflect their functionality. https://reviews.llvm.org/D26461 Files: include/clang/Driver/Options.td include/clang/Driver/SanitizerArgs.h

[PATCH] D25857: [tsan][clang] Introduce a function attribute to disable TSan checking at run time

2016-11-09 Thread Anna Zaks via cfe-commits
zaks.anna updated this revision to Diff 77435. zaks.anna marked an inline comment as done. https://reviews.llvm.org/D25857 Files: lib/CodeGen/CodeGenFunction.cpp test/CodeGen/sanitize-thread-no-checking-at-run-time.m Index: test/CodeGen/sanitize-thread-no-checking-at-run-time.m

[PATCH] D26108: Add -Wnullability-completeness-on-arrays.

2016-11-09 Thread Akira Hatanaka via cfe-commits
ahatanak added inline comments. Comment at: lib/Sema/SemaType.cpp:3988 // Allow arrays of auto if we are a generic lambda parameter. // i.e. [](auto ()[5]) { return array[0]; }; OK if (AT && D.getContext() != Declarator::LambdaExprParameterContext) {

[PATCH] D26491: Define __ANDROID_API__ when specified as part of an Android target.

2016-11-09 Thread Stephen Hines via cfe-commits
srhines updated this revision to Diff 77446. srhines added a comment. Added bug link to commit message. https://reviews.llvm.org/D26491 Files: lib/Basic/Targets.cpp test/Preprocessor/init.c Index: test/Preprocessor/init.c

[PATCH] D26491: Define __ANDROID_API__ when specified as part of an Android target.

2016-11-09 Thread Stephen Hines via cfe-commits
srhines created this revision. srhines added subscribers: danalbert, eugenis, pirama, cfe-commits. Herald added a subscriber: tberghammer. This macro should be defined only when the user directly specifies an API level as part of an Android target. For any regular Android target, we leave this

[PATCH] D26423: [clang-move] Support template class.

2016-11-09 Thread Haojian Wu via cfe-commits
This revision was automatically updated to reflect the committed changes. hokein marked an inline comment as done. Closed by commit rL286427: [clang-move] Support template class. (authored by hokein). Changed prior to commit: https://reviews.llvm.org/D26423?vs=77292=77447#toc Repository: rL

[clang-tools-extra] r286427 - [clang-move] Support template class.

2016-11-09 Thread Haojian Wu via cfe-commits
Author: hokein Date: Wed Nov 9 23:33:26 2016 New Revision: 286427 URL: http://llvm.org/viewvc/llvm-project?rev=286427=rev Log: [clang-move] Support template class. Reviewers: ioeric Subscribers: cfe-commits Differential Revision: https://reviews.llvm.org/D26423 Added:

[PATCH] D26493: [clang-move] Make the output code look more pretty.

2016-11-09 Thread Haojian Wu via cfe-commits
hokein created this revision. hokein added a reviewer: ioeric. hokein added a subscriber: cfe-commits. Add decent blank line between declarations. https://reviews.llvm.org/D26493 Files: clang-move/ClangMove.cpp unittests/clang-move/ClangMoveTests.cpp Index:

<    1   2