Re: [PATCH] D22774: [MSVC] Add ARM support to intrin.h for MSVC compatibility

2016-08-06 Thread Martin Storsjö via cfe-commits
mstorsjo added a comment. Can someone commit this for me? https://reviews.llvm.org/D22774 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r277918 - Move helpers into anonymous namespaces. NFC.

2016-08-06 Thread Benjamin Kramer via cfe-commits
Author: d0k Date: Sat Aug 6 06:21:04 2016 New Revision: 277918 URL: http://llvm.org/viewvc/llvm-project?rev=277918&view=rev Log: Move helpers into anonymous namespaces. NFC. Modified: cfe/trunk/lib/Frontend/CompilerInvocation.cpp cfe/trunk/lib/Sema/SemaTemplate.cpp cfe/trunk/lib/Seri

r277917 - [StaticAnalyzer] Remove dead code.

2016-08-06 Thread Benjamin Kramer via cfe-commits
Author: d0k Date: Sat Aug 6 06:20:59 2016 New Revision: 277917 URL: http://llvm.org/viewvc/llvm-project?rev=277917&view=rev Log: [StaticAnalyzer] Remove dead code. Modified: cfe/trunk/include/clang/StaticAnalyzer/Core/PathSensitive/MemRegion.h cfe/trunk/lib/StaticAnalyzer/Core/MemRegion.

r277920 - [Sema] Make switch fully covered again.

2016-08-06 Thread Benjamin Kramer via cfe-commits
Author: d0k Date: Sat Aug 6 06:28:20 2016 New Revision: 277920 URL: http://llvm.org/viewvc/llvm-project?rev=277920&view=rev Log: [Sema] Make switch fully covered again. Modified: cfe/trunk/lib/Sema/SemaExpr.cpp Modified: cfe/trunk/lib/Sema/SemaExpr.cpp URL: http://llvm.org/viewvc/llvm-proj

r277923 - [ASTReader] Use real move semantics instead of emulating them in the copy ctor.

2016-08-06 Thread Benjamin Kramer via cfe-commits
Author: d0k Date: Sat Aug 6 07:45:16 2016 New Revision: 277923 URL: http://llvm.org/viewvc/llvm-project?rev=277923&view=rev Log: [ASTReader] Use real move semantics instead of emulating them in the copy ctor. No functionality change intended. Modified: cfe/trunk/lib/Serialization/ASTReaderD

[PATCH] D23228: Do not depend on unwind when building standalone

2016-08-06 Thread Petr Hosek via cfe-commits
phosek created this revision. phosek added a subscriber: cfe-commits. When libcxxabi is being built standalone, unwind dependency is not available, so do not use it even when LLVM unwinder is being requested. https://reviews.llvm.org/D23228 Files: test/CMakeLists.txt Index: test/CMakeLists.t

r277928 - Headers: Add ARM support to intrin.h for MSVC compatibility

2016-08-06 Thread Saleem Abdulrasool via cfe-commits
Author: compnerd Date: Sat Aug 6 12:58:24 2016 New Revision: 277928 URL: http://llvm.org/viewvc/llvm-project?rev=277928&view=rev Log: Headers: Add ARM support to intrin.h for MSVC compatibility This fixes compiling with headers from the Windows SDK for ARM, where the YieldProcessor function (in

Re: [PATCH] D22774: [MSVC] Add ARM support to intrin.h for MSVC compatibility

2016-08-06 Thread Saleem Abdulrasool via cfe-commits
compnerd closed this revision. compnerd added a comment. SVN r277928 https://reviews.llvm.org/D22774 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D23239: [CUDA] Add __device__ overloads for placement new and delete.

2016-08-06 Thread Justin Lebar via cfe-commits
jlebar created this revision. jlebar added a reviewer: tra. jlebar added a subscriber: cfe-commits. Previously these sort of worked because they didn't end up resulting in calls at the ptx layer. But I'm adding stricter checks that break placement new without these changes. https://reviews.llvm.

[PATCH] D23238: [CUDA] Rename CheckCUDATarget to IsAllowedCUDACall. NFC

2016-08-06 Thread Justin Lebar via cfe-commits
jlebar created this revision. jlebar added a reviewer: tra. jlebar added a subscriber: cfe-commits. I want to reuse "CheckCUDAFoo" in a later patch. Also, I think IsAllowedCUDACall gets the point across more clearly. https://reviews.llvm.org/D23238 Files: clang/include/clang/Sema/Sema.h cla

[PATCH] D23240: [CUDA] Print a "previous-decl" note when calling an illegal member fn.

2016-08-06 Thread Justin Lebar via cfe-commits
jlebar created this revision. jlebar added a reviewer: tra. jlebar added a subscriber: cfe-commits. When we emit err_ref_bad_target, we should emit a "'method' declared here" note. We already do so in most places, just not in BuildCallToMemberFunction. https://reviews.llvm.org/D23240 Files: c

[PATCH] D23241: Add the notion of deferred diagnostics.

2016-08-06 Thread Justin Lebar via cfe-commits
jlebar created this revision. jlebar added a reviewer: rnk. jlebar added subscribers: tra, cfe-commits. This patch lets you create diagnostics that are emitted if and only if a particular FunctionDecl is codegen'ed. This is necessary for CUDA, where some constructs -- e.g. calls from host+device

[PATCH] D23242: [CUDA] Raise an error if a wrong-side call is codegen'ed.

2016-08-06 Thread Justin Lebar via cfe-commits
jlebar created this revision. jlebar added reviewers: tra, rnk. jlebar added a subscriber: cfe-commits. Some function calls in CUDA are allowed to appear in semantically-correct programs but are an error if they're ever codegen'ed. Specifically, a host+device function may call a host function, bu

r277935 - Add the new scan-build option (--show-description) in the 4.0 release notes

2016-08-06 Thread Sylvestre Ledru via cfe-commits
Author: sylvestre Date: Sat Aug 6 15:23:54 2016 New Revision: 277935 URL: http://llvm.org/viewvc/llvm-project?rev=277935&view=rev Log: Add the new scan-build option (--show-description) in the 4.0 release notes Modified: cfe/trunk/docs/ReleaseNotes.rst Modified: cfe/trunk/docs/ReleaseNotes.

Re: [PATCH] D23198: clang-rename rename-all: support reading old/newname pairs from a YAML file

2016-08-06 Thread Kirill Bobyrev via cfe-commits
omtcyfz added a comment. `.cpp.rename-at.yaml`? I understand it's done for the purpose of doing `-input %s.rename-at.yaml`, though I don't think getting proper names for them and either doing some string magic in tool invocation or just passing full name is wrong. Other than that my concerns s

Re: [PATCH] D23198: clang-rename rename-all: support reading old/newname pairs from a YAML file

2016-08-06 Thread Kirill Bobyrev via cfe-commits
omtcyfz added a comment. + I'll take a look again on Monday, too. May be too sleepy now :) https://reviews.llvm.org/D23198 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D23243: [clang-tidy] enhance modernize-use-bool-literals to check ternary operator

2016-08-06 Thread Kirill Bobyrev via cfe-commits
omtcyfz created this revision. omtcyfz added reviewers: alexfh, Eugene.Zelenko. omtcyfz added a subscriber: cfe-commits. Fixing bug https://llvm.org/bugs/show_bug.cgi?id=28854. Implicit cast with ternary operator makes AST look in a way in which it feels reasonable to add second matcher to detec

[PATCH] [clang-format] add AfterMultilineControlStatement

2016-08-06 Thread Rinat Ibragimov via cfe-commits
Hi. When indent is four spaces, "if" conditions spanning multiple lines could be difficult to discern from the "if" body. Indentation is the same, so they are visually blending. (See example below). One of the solutions is to place opening brace on a separate line when "if" condition spans multipl

Re: [PATCH] D23243: [clang-tidy] enhance modernize-use-bool-literals to check ternary operator

2016-08-06 Thread Piotr Padlewski via cfe-commits
Prazek added inline comments. Comment at: clang-tidy/modernize/UseBoolLiteralsCheck.cpp:38-43 @@ +37,8 @@ + unless(isInTemplateInstantiation(, + anyOf(hasTrueExpression(ignoringParenImpCasts( +integerLiteral().bind("trueBranchLiteral")