Re: [PATCH] D24500: [clang-tidy] Bugfix for readability-redundant-control-flow check

alexfh accepted this revision. alexfh added a comment. This revision is now accepted and ready to land. LG. If there's a related buganizer item, please mention it in the commit message. https://reviews.llvm.org/D24500 ___ cfe-commits mailing list

Re: [PATCH] D24224: [clang-rename] Merge rename-{ at | all } and optimise USRFindingAction.

omtcyfz updated this revision to Diff 71334. omtcyfz marked an inline comment as done. omtcyfz added a comment. Nit. https://reviews.llvm.org/D24224 Files: clang-rename/USRFindingAction.cpp clang-rename/USRFindingAction.h clang-rename/tool/ClangRename.cpp docs/clang-rename.rst

Re: r281452 - Revert "[modules] When merging one definition into another, propagate the list of re-exporting modules from the discarded definition to the retained definition."

This revision caused internal build breakage, but unfortunately I'm not sure what exactly in this revision caused the breakage. I have reported the issue to @rsmith - the revision author. On Wed, Sep 14, 2016 at 3:10 PM Nico Weber wrote: > When reverting something, please

Re: [PATCH] D24518: Correct assert text in DeclGroup::getSingleDecl()

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

Re: [PATCH] D24500: [clang-tidy] Bugfix for readability-redundant-control-flow check

malcolm.parsons added a comment. I didn't report a bug for this issue, and there isn't an existing one. https://reviews.llvm.org/D24500 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

Re: [PATCH] D24515: Supports adding insertion around non-insertion replacements.

djasper added inline comments. Comment at: include/clang/Tooling/Core/Replacement.h:172 @@ +171,3 @@ + /// - They do not overlap. + /// - One replacement is insertion, and the other is a replacement with + /// length > 0, and the insertion is adjecent to but not

Re: [PATCH] D24515: Supports adding insertion around non-insertion replacements.

djasper accepted this revision. djasper added a comment. This revision is now accepted and ready to land. Looks good. Comment at: include/clang/Tooling/Core/Replacement.h:172 @@ -169,1 +171,3 @@ + /// - aren't both inserts at the same code location. + /// Note: two

r281457 - Supports adding insertion around non-insertion replacements.

Author: ioeric Date: Wed Sep 14 08:04:51 2016 New Revision: 281457 URL: http://llvm.org/viewvc/llvm-project?rev=281457=rev Log: Supports adding insertion around non-insertion replacements. Summary: Extend `tooling::Replacements::add()` to support adding order-independent replacements. Two

Re: [PATCH] D24515: Supports adding insertion around non-insertion replacements.

This revision was automatically updated to reflect the committed changes. Closed by commit rL281457: Supports adding insertion around non-insertion replacements. (authored by ioeric). Changed prior to commit: https://reviews.llvm.org/D24515?vs=71335=71337#toc Repository: rL LLVM

Re: [clang-tools-extra] r281453 - [clang-tidy] Add check 'misc-use-after-move'

> > While I'm excited to see this go in anywhere, I have to say I'm a bit sad > it isn't going in as a warning and instead inside clang-tidy. This has been > a much requested warning from Clang for many years. > > Is there a concise description of why this design was chosen? Are there > specific

Re: [PATCH] D24005: [compiler-rt cmake] Support overriding llvm-config query results

This revision was automatically updated to reflect the committed changes. Closed by commit rL281461: [cmake] Support overriding llvm-config query results (authored by mgorny). Changed prior to commit: https://reviews.llvm.org/D24005?vs=71040=71348#toc Repository: rL LLVM

Re: [PATCH] D24561: [clang-tidy] Add dependency on clangAnalysis to clangTidyMiscModule

mboehme added a comment. Thanks! Repository: rL LLVM https://reviews.llvm.org/D24561 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D24550: [clang-tidy] Make test for misc-use-after-move pass under Windows

mboehme created this revision. mboehme added a reviewer: alexfh. mboehme added a subscriber: cfe-commits. Adds -fno-delayed-template-parsing https://reviews.llvm.org/D24550 Files: test/clang-tidy/misc-use-after-move.cpp Index: test/clang-tidy/misc-use-after-move.cpp

[clang-tools-extra] r281460 - [clang-tidy] Add dependency on clangAnalysis to clangTidyMiscModule

Author: mboehme Date: Wed Sep 14 08:33:11 2016 New Revision: 281460 URL: http://llvm.org/viewvc/llvm-project?rev=281460=rev Log: [clang-tidy] Add dependency on clangAnalysis to clangTidyMiscModule Summary: This is needed for the recently submitted misc-use-after-move check (rL281453). For some

Re: [PATCH] D24561: [clang-tidy] Add dependency on clangAnalysis to clangTidyMiscModule

This revision was automatically updated to reflect the committed changes. Closed by commit rL281460: [clang-tidy] Add dependency on clangAnalysis to clangTidyMiscModule (authored by mboehme). Changed prior to commit: https://reviews.llvm.org/D24561?vs=71345=71346#toc Repository: rL LLVM

Re: [PATCH] D24005: [compiler-rt cmake] Support overriding llvm-config query results

mgorny added a comment. Thanks again. I've left them as-is and committed. Repository: rL LLVM https://reviews.llvm.org/D24005 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D24550: [clang-tidy] Make test for misc-use-after-move pass under Windows

mboehme added a comment. Single-line trivial change, will submit without review. Please let me know if you have any objections. https://reviews.llvm.org/D24550 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

Re: [clang-tools-extra] r281453 - [clang-tidy] Add check 'misc-use-after-move'

On Wed, Sep 14, 2016 at 3:38 AM Martin Bohme via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Author: mboehme > Date: Wed Sep 14 05:29:32 2016 > New Revision: 281453 > > URL: http://llvm.org/viewvc/llvm-project?rev=281453=rev > Log: > [clang-tidy] Add check 'misc-use-after-move' > >

Re: [PATCH] D24550: [clang-tidy] Make test for misc-use-after-move pass under Windows

This revision was automatically updated to reflect the committed changes. Closed by commit rL281455: [clang-tidy] Make test for misc-use-after-move pass under Windows (authored by mboehme). Changed prior to commit: https://reviews.llvm.org/D24550?vs=71318=71319#toc Repository: rL LLVM

[clang-tools-extra] r281455 - [clang-tidy] Make test for misc-use-after-move pass under Windows

Author: mboehme Date: Wed Sep 14 07:22:35 2016 New Revision: 281455 URL: http://llvm.org/viewvc/llvm-project?rev=281455=rev Log: [clang-tidy] Make test for misc-use-after-move pass under Windows Summary: Adds -fno-delayed-template-parsing Reviewers: alexfh Subscribers: cfe-commits

Re: [PATCH] D24339: [clang-tidy] Add check 'readability-redundant-member-init'

alexfh added a comment. In https://reviews.llvm.org/D24339#537488, @malcolm.parsons wrote: > How do I add FixIt hints? > They should be simple removals, but how do I decide whether to remove the > following comma, preceding comma or preceding colon? Just remove the entry and leave the comma

[clang-tools-extra] r281459 - reverting r281456

Author: omtcyfz Date: Wed Sep 14 08:23:14 2016 New Revision: 281459 URL: http://llvm.org/viewvc/llvm-project?rev=281459=rev Log: reverting r281456 Modified: clang-tools-extra/trunk/clang-rename/USRFindingAction.cpp clang-tools-extra/trunk/clang-rename/USRFindingAction.h

Re: [PATCH] D24561: [clang-tidy] Add dependency on clangAnalysis to clangTidyMiscModule

mboehme added a comment. Trivial build fix, will submit without review. https://reviews.llvm.org/D24561 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D24561: [clang-tidy] Add dependency on clangAnalysis to clangTidyMiscModule

mboehme created this revision. mboehme added a subscriber: cfe-commits. Herald added subscribers: mgorny, beanz. This is needed for the recently submitted misc-use-after-move check (rL281453). For some reason, this still built under Linux, but it caused the PPC build bot to fail.

Re: [PATCH] D24224: [clang-rename] Merge rename-{ at | all } and optimise USRFindingAction.

This revision was automatically updated to reflect the committed changes. Closed by commit rL281456: [clang-rename] Merge rename-{at|all} & optimize. (authored by omtcyfz). Changed prior to commit: https://reviews.llvm.org/D24224?vs=71334=71336#toc Repository: rL LLVM

Re: r281452 - Revert "[modules] When merging one definition into another, propagate the list of re-exporting modules from the discarded definition to the retained definition."

When reverting something, please say why in the commit message. On Sep 14, 2016 6:13 AM, "Eric Liu via cfe-commits" < cfe-commits@lists.llvm.org> wrote: > Author: ioeric > Date: Wed Sep 14 05:05:10 2016 > New Revision: 281452 > > URL: http://llvm.org/viewvc/llvm-project?rev=281452=rev > Log: >

Re: [PATCH] D24515: Supports adding insertion around non-insertion replacements.

ioeric updated this revision to Diff 71335. ioeric added a comment. - Updated comment. https://reviews.llvm.org/D24515 Files: include/clang/Tooling/Core/Replacement.h lib/Tooling/Core/Replacement.cpp unittests/Tooling/RefactoringTest.cpp Index: unittests/Tooling/RefactoringTest.cpp

Re: [PATCH] D24561: [clang-tidy] Add dependency on clangAnalysis to clangTidyMiscModule

aaron.ballman added a subscriber: aaron.ballman. aaron.ballman accepted this revision. aaron.ballman added a reviewer: aaron.ballman. aaron.ballman added a comment. This revision is now accepted and ready to land. LGTM -- for these sort of post-commit fixes that are trivial and make failing bots

Re: [PATCH] D24515: Supports adding insertion around non-insertion replacements.

ioeric updated this revision to Diff 71332. ioeric marked an inline comment as done. ioeric added a comment. - Addressed comment. https://reviews.llvm.org/D24515 Files: include/clang/Tooling/Core/Replacement.h lib/Tooling/Core/Replacement.cpp unittests/Tooling/RefactoringTest.cpp Index:

[clang-tools-extra] r281456 - [clang-rename] Merge rename-{at|all} & optimize.

Author: omtcyfz Date: Wed Sep 14 08:00:36 2016 New Revision: 281456 URL: http://llvm.org/viewvc/llvm-project?rev=281456=rev Log: [clang-rename] Merge rename-{at|all} & optimize. Having both rename-at and rename-all both seems confusing and introduces unneeded difficulties. Allowing to use both

Re: [PATCH] D24224: [clang-rename] Merge rename-{ at | all } and optimise USRFindingAction.

alexfh accepted this revision. alexfh added a comment. This revision is now accepted and ready to land. LG with a nit. Comment at: clang-rename/USRFindingAction.cpp:182 @@ -161,1 +181,3 @@ +return false; } else { +unsigned CouldNotFindSymbolNamed =

[libcxx] r281475 - [libcxx] Enable building and testing of libcxx with ThreadSanitizer on OS X

Author: kuba.brecka Date: Wed Sep 14 09:12:50 2016 New Revision: 281475 URL: http://llvm.org/viewvc/llvm-project?rev=281475=rev Log: [libcxx] Enable building and testing of libcxx with ThreadSanitizer on OS X This patch enables building and testing libcxx under ThreadSanitizer on OS X. CMake

[libcxx] r281476 - [libcxx] Fix a typo in test/libcxx/test/target_info.py that prevents running tests on Darwin with sanitizers

Author: kuba.brecka Date: Wed Sep 14 09:13:50 2016 New Revision: 281476 URL: http://llvm.org/viewvc/llvm-project?rev=281476=rev Log: [libcxx] Fix a typo in test/libcxx/test/target_info.py that prevents running tests on Darwin with sanitizers Differential Revision:

r281487 - CodeGen: simplify the logic a slight bit

Author: compnerd Date: Wed Sep 14 10:17:46 2016 New Revision: 281487 URL: http://llvm.org/viewvc/llvm-project?rev=281487=rev Log: CodeGen: simplify the logic a slight bit Move the definition of `getTriple()` into the header. It would just call `getTarget().getTriple()`. Inline the definition

[PATCH] D24562: [libcxx] Recover no-exceptions XFAILs

rmaprath created this revision. rmaprath added reviewers: EricWF, mclow.lists. rmaprath added a subscriber: cfe-commits. When we added support for the no-exceptions build of `libc++`, all the tests that used exceptions got a blanket XFAIL for the no-exceptions variety. Previously, we tried to

r281469 - Fix documentation of MemberExpr::getMemberDecl

Author: sberg Date: Wed Sep 14 09:03:50 2016 New Revision: 281469 URL: http://llvm.org/viewvc/llvm-project?rev=281469=rev Log: Fix documentation of MemberExpr::getMemberDecl Differential Revision: https://reviews.llvm.org/D23907 Modified: cfe/trunk/include/clang/AST/Expr.h Modified:

[PATCH] D24567: [clang-rename] Merge rename-{at|all} & optimise.

omtcyfz created this revision. omtcyfz added reviewers: alexfh, vmiklos. omtcyfz added a subscriber: cfe-commits. Having both rename-at and rename-all both seems confusing and introduces unneeded difficulties. After merging rename-at and rename-all maintaining main function wrappers and custom

Re: [PATCH] D24245: [ARM] ARM-specific attributes should be accepted for big-endian

rovka added a subscriber: rovka. rovka accepted this revision. rovka added a reviewer: rovka. rovka added a comment. This revision is now accepted and ready to land. Looks like a sensible thing to do. Repository: rL LLVM https://reviews.llvm.org/D24245

[libcxx] r281477 - [libcxx] Add a TSan regression test for a data race in call_once

Author: kuba.brecka Date: Wed Sep 14 09:15:42 2016 New Revision: 281477 URL: http://llvm.org/viewvc/llvm-project?rev=281477=rev Log: [libcxx] Add a TSan regression test for a data race in call_once Differential Revision: https://reviews.llvm.org/D24297 Added:

Re: [PATCH] D24562: [libcxx] Recover no-exceptions XFAILs

rmaprath updated this revision to Diff 71349. rmaprath added a comment. (Added more context) https://reviews.llvm.org/D24562 Files: test/std/re/re.alg/re.alg.search/grep.pass.cpp Index: test/std/re/re.alg/re.alg.search/grep.pass.cpp

Re: [PATCH] D23907: Fix documentation of MemberExpr::getMemberDecl

This revision was automatically updated to reflect the committed changes. Closed by commit rL281469: Fix documentation of MemberExpr::getMemberDecl (authored by sberg). Changed prior to commit: https://reviews.llvm.org/D23907?vs=69317=71357#toc Repository: rL LLVM

Re: [PATCH] D24562: [libcxx] Recover no-exceptions XFAILs

rmaprath added a comment. In https://reviews.llvm.org/D24562#542630, @mclow.lists wrote: > What an awful test. I wonder who wrote such a steaming pile. Probably me. I think I've thrown quite a lot of XFAILs like this in the past, just to get the no-exceptions build passing. Now I've got to

Re: [PATCH] D21506: [analyzer] Block in critical section

zdtorok added a comment. You are right NoQ, thank you for the remark, I've upload a new diff containing a fix for the integer underflow case. Repository: rL LLVM https://reviews.llvm.org/D21506 ___ cfe-commits mailing list

[clang-tools-extra] r281510 - [Release notes] Mention readability-container-size-empty improvements.

Author: eugenezelenko Date: Wed Sep 14 12:41:51 2016 New Revision: 281510 URL: http://llvm.org/viewvc/llvm-project?rev=281510=rev Log: [Release notes] Mention readability-container-size-empty improvements. Differential revision: https://reviews.llvm.org/D24526 Modified:

Re: [PATCH] D24526: [Release notes] Mention readability-container-size-empty improvements

This revision was automatically updated to reflect the committed changes. Closed by commit rL281510: [Release notes] Mention readability-container-size-empty improvements. (authored by eugenezelenko). Changed prior to commit: https://reviews.llvm.org/D24526?vs=71250=71394#toc Repository: rL

[PATCH] D24571: [CUDA] Disallow overloading destructors.

jlebar created this revision. jlebar added a reviewer: rsmith. jlebar added subscribers: tra, cfe-commits. We'd attempted to allow this, but turns out we were doing a very bad job. :) Making this work properly would be a giant change in clang. For example, we'd need to make

[PATCH] D24572: [clang-tidy] Clean up code after applying replacements.

alexfh created this revision. alexfh added reviewers: ioeric, hokein. alexfh added a subscriber: cfe-commits. Remove empty namespaces and initializer list commas / colons in affected ranges. A strawman patch: proper options for enabling the cleanup and specifying the format style are needed.

[PATCH] D24574: clang-format: [JS] ASI insertion after boolean literals.

mprobst created this revision. mprobst added a reviewer: djasper. mprobst added a subscriber: cfe-commits. Herald added a subscriber: klimek. Before when a semicolon was missing after a boolean literal: a = true return 1; clang-format would parse this as one line and format as: a =

Re: [PATCH] D24562: [libcxx] Recover no-exceptions XFAILs

mclow.lists added a comment. What an awful test. I wonder who wrote such a steaming pile. Probably me. https://reviews.llvm.org/D24562 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

r281509 - Convert finite to builtin

Author: dehao Date: Wed Sep 14 12:34:14 2016 New Revision: 281509 URL: http://llvm.org/viewvc/llvm-project?rev=281509=rev Log: Convert finite to builtin Summary: This patch converts finite/__finite to builtin functions so that it will be inlined by compiler. Reviewers: hfinkel, davidxl,

Re: [PATCH] D24467: Fix an error after D21678

vbyakovlcl added inline comments. Comment at: llvm/tools/clang/lib/Sema/SemaExpr.cpp:8733 @@ -8731,2 +8732,3 @@ if (!IsCompAssign) { -LHS = S.UsualUnaryConversions(LHS.get()); +if (S.LangOpts.OpenCL || S.LangOpts.ZVector) + LHS =

Re: [PATCH] D21506: [analyzer] Block in critical section

zdtorok updated this revision to Diff 71375. zdtorok added a comment. Herald added subscribers: mgorny, beanz. Fixed possible integer-underflow in case of unexpected unlocking function. Also added test for this. Repository: rL LLVM https://reviews.llvm.org/D21506 Files:

Re: [PATCH] D22346: [Clang-tidy] CERT-MSC50-CPP (std:rand() )

aaron.ballman added inline comments. Comment at: clang-tidy/cert/LimitedRandomnessCheck.cpp:22-23 @@ +21,4 @@ + Finder->addMatcher( + declRefExpr(hasDeclaration(functionDecl(namedDecl(hasName("::rand")), + parameterCountIs(0

Re: [PATCH] D24526: [Release notes] Mention readability-container-size-empty improvements

Eugene.Zelenko added inline comments. Comment at: docs/ReleaseNotes.rst:94 @@ +93,3 @@ + `_ check + supports arbitrary containers with ``empty()`` and ``size()`` methods. +

Re: [PATCH] D24513: [AMDGPU] Expose flat work group size, register and wave control attributes

kzhuravl updated this revision to Diff 71382. kzhuravl added a comment. Fix minor typos https://reviews.llvm.org/D24513 Files: include/clang/Basic/Attr.td include/clang/Basic/AttrDocs.td include/clang/Basic/DiagnosticSemaKinds.td lib/CodeGen/TargetInfo.cpp lib/Sema/SemaDeclAttr.cpp

[PATCH] D24573: [CUDA] Do a better job at detecting wrong-side calls.

jlebar created this revision. jlebar added a reviewer: rsmith. jlebar added subscribers: tra, cfe-commits. Move CheckCUDACall from ActOnCallExpr and BuildDeclRefExpr to DiagnoseUseOfDecl. This lets us catch some edge cases we were missing, specifically around class operators. This necessitates

Re: [PATCH] D24513: [AMDGPU] Expose flat work group size, register and wave control attributes

aaron.ballman requested changes to this revision. This revision now requires changes to proceed. Comment at: include/clang/Basic/Attr.td:1054-1056 @@ -1058,3 +1053,5 @@ +// // FIXME: This should be for OpenCLKernelFunction, but is not to // workaround needing to see kernel

r281516 - [analyzer] scan-build-py: Remove relative path hack for SATestsBuild.py

Author: dcoughlin Date: Wed Sep 14 13:14:11 2016 New Revision: 281516 URL: http://llvm.org/viewvc/llvm-project?rev=281516=rev Log: [analyzer] scan-build-py: Remove relative path hack for SATestsBuild.py Remove the relative path hack in scan-build-py that converts a fully qualified directory name

Re: [PATCH] D24470: [analyzer] scan-build-py: Remove relative path hack for SATestsBuild.py

This revision was automatically updated to reflect the committed changes. Closed by commit rL281516: [analyzer] scan-build-py: Remove relative path hack for SATestsBuild.py (authored by dcoughlin). Changed prior to commit: https://reviews.llvm.org/D24470?vs=71046=71401#toc Repository: rL

Re: [PATCH] D21506: [analyzer] Block in critical section

zaks.anna accepted this revision. zaks.anna added a comment. This revision is now accepted and ready to land. LGTM! Thanks! Future steps: How do we plan to bring this checker out of alpha? Have you evaluated it on large codebases? Repository: rL LLVM https://reviews.llvm.org/D21506

Re: [PATCH] D24372: [libcxx] Sprinkle constexpr over compressed_pair

static_cast(t) also works as a constexpr C++11 forward. On Sep 13, 2016 4:07 PM, "Keno Fischer" wrote: > loladiro added inline comments. > > > Comment at: include/memory:2137 > @@ -2132,3 +2136,3 @@ > > -_LIBCPP_INLINE_VISIBILITY > +

Re: [PATCH] D24574: clang-format: [JS] ASI insertion after boolean literals.

djasper accepted this revision. djasper added a comment. This revision is now accepted and ready to land. Interesting.. Would have expected true/false to be literals as well, but ok :). https://reviews.llvm.org/D24574 ___ cfe-commits mailing list

Re: [PATCH] D24467: Fix an error after D21678

vbyakovlcl added inline comments. Comment at: llvm/tools/clang/lib/Sema/SemaExpr.cpp:8790 @@ -8774,3 +8789,3 @@ S.Context.getExtVectorType(RHSEleType, LHSVecTy->getNumElements()); RHS = S.ImpCastExprToType(RHS.get(), VecTy, CK_VectorSplat); }

Re: [PATCH] D24518: Correct assert text in DeclGroup::getSingleDecl()

brtaylor92 added a comment. I don't have commit access. @alexfh could you please land the patch? https://reviews.llvm.org/D24518 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D24467: Fix an error after D21678

ahatanak added inline comments. Comment at: llvm/tools/clang/lib/Sema/SemaExpr.cpp:8751 @@ -8747,3 +8750,3 @@ // OpenCL v1.1 s6.3.j says that the operands need to be integers. if (!LHSEleType->isIntegerType()) { Should we mention that any vectors used as

Re: [PATCH] D23820: Do not warn about format strings that are indexed string literals.

srhines added a comment. I will take care of submitting this. Thanks for the reviews everybody! https://reviews.llvm.org/D23820 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D23820: Do not warn about format strings that are indexed string literals.

meikeb marked an inline comment as done. meikeb added a comment. Thanks for reviewing my patch! It would be great if someone could submit this patch for me. https://reviews.llvm.org/D23820 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

Re: [PATCH] D22452: [libcxx] Fix last_write_time tests for filesystems that don't support negative and very large times.

EricWF added a comment. Thanks. I'll update this patch today. https://reviews.llvm.org/D22452 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D23820: Do not warn about format strings that are indexed string literals.

This revision was automatically updated to reflect the committed changes. Closed by commit rL281527: Do not warn about format strings that are indexed string literals. (authored by srhines). Changed prior to commit: https://reviews.llvm.org/D23820?vs=71286=71416#toc Repository: rL LLVM

r281527 - Do not warn about format strings that are indexed string literals.

Author: srhines Date: Wed Sep 14 15:05:20 2016 New Revision: 281527 URL: http://llvm.org/viewvc/llvm-project?rev=281527=rev Log: Do not warn about format strings that are indexed string literals. Summary: The warning for a format string not being a sting literal and therefore being potentially

Re: [PATCH] D24330: Add some MS aliases for existing intrinsics

agutowski marked an inline comment as done. agutowski added a comment. https://reviews.llvm.org/D24330 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r281525 - Correct assert text in DeclGroup::getSingleDecl()

Author: omtcyfz Date: Wed Sep 14 14:59:26 2016 New Revision: 281525 URL: http://llvm.org/viewvc/llvm-project?rev=281525=rev Log: Correct assert text in DeclGroup::getSingleDecl() Assert text for getSingleDecl() is inaccurate. Appears to have been copy pasted from getDeclGroup(). Patch by Ben

Re: [PATCH] D24518: Correct assert text in DeclGroup::getSingleDecl()

This revision was automatically updated to reflect the committed changes. Closed by commit rL281525: Correct assert text in DeclGroup::getSingleDecl() (authored by omtcyfz). Changed prior to commit: https://reviews.llvm.org/D24518?vs=71188=71414#toc Repository: rL LLVM

[PATCH] D24579: Revert "Do not warn about format strings that are indexed string literals."

srhines created this revision. srhines added a subscriber: cfe-commits. This reverts r281527 because I messed up the attribution. https://reviews.llvm.org/D24579 Files: lib/Sema/SemaChecking.cpp test/Sema/format-strings.c Index: test/Sema/format-strings.c

Re: [PATCH] D23921: Remove va_start diagnostic false positive with enumerations

dim added a comment. FWIW, I have already imported this fix into FreeBSD two weeks ago: https://svnweb.freebsd.org/changeset/base/304960 It has been working fine for us. https://reviews.llvm.org/D23921 ___ cfe-commits mailing list

[PATCH] D24581: [CUDA] Add test checking our ability to take a function pointer to a __global__ function on the host side.

jlebar created this revision. jlebar added a reviewer: tra. jlebar added a subscriber: cfe-commits. This functionality is used by Thrust. https://reviews.llvm.org/D24581 Files: clang/test/SemaCUDA/reference-to-kernel-fn.cu Index: clang/test/SemaCUDA/reference-to-kernel-fn.cu

Re: [PATCH] D24581: [CUDA] Add test checking our ability to take a function pointer to a __global__ function on the host side.

tra accepted this revision. tra added a comment. This revision is now accepted and ready to land. LGTM. https://reviews.llvm.org/D24581 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D24590: [test-suite] [CUDA] Update README.

jlebar created this revision. jlebar added a reviewer: tra. jlebar added a subscriber: cfe-commits. Fix typo (s/Externals/External/), and mention cuda-tests-simple. https://reviews.llvm.org/D24590 Files: External/CUDA/README Index: External/CUDA/README

Re: [PATCH] D24508: PR28752: Do not instantiate var decls which are not visible.

rsmith added a comment. I think you'll also need to update the `ASTDeclReader` to merge `VarDecl` definitions together if it reads a definition and there already is one in the AST. I note that right now `Sema::AddInitializerToDecl` permits multiple definitions of a `VarDecl`, which doesn't

Re: [PATCH] D24579: Revert "Do not warn about format strings that are indexed string literals."

This revision was automatically updated to reflect the committed changes. Closed by commit rL281530: Revert "Do not warn about format strings that are indexed string literals." (authored by srhines). Changed prior to commit: https://reviews.llvm.org/D24579?vs=71417=71419#toc Repository: rL

[PATCH] D24584: Do not warn about format strings that are indexed string literals.

meikeb created this revision. meikeb added a reviewer: rsmith. meikeb added subscribers: cfe-commits, srhines. The warning for a format string not being a sting literal and therefore being potentially insecure is overly strict for indecies into sting literals. This fix checks if the index into

Re: [PATCH] D24588: [CUDA] Make __clang_cuda_cmath.h compatible with libc++.

tra accepted this revision. tra added a comment. This revision is now accepted and ready to land. LGTM. https://reviews.llvm.org/D24588 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

r281544 - [CUDA] Make __clang_cuda_cmath.h compatible with libc++.

Author: jlebar Date: Wed Sep 14 16:50:14 2016 New Revision: 281544 URL: http://llvm.org/viewvc/llvm-project?rev=281544=rev Log: [CUDA] Make __clang_cuda_cmath.h compatible with libc++. Summary: We need to add a bunch more "using"s, which weren't necessary with libstdc++. Once this is in I can

Re: [PATCH] D24581: [CUDA] Add test checking our ability to take a function pointer to a __global__ function on the host side.

This revision was automatically updated to reflect the committed changes. Closed by commit rL281543: [CUDA] Add test checking our ability to take a function pointer to a… (authored by jlebar). Changed prior to commit: https://reviews.llvm.org/D24581?vs=71423=71446#toc Repository: rL LLVM

r281543 - [CUDA] Add test checking our ability to take a function pointer to a __global__ function on the host side.

Author: jlebar Date: Wed Sep 14 16:50:11 2016 New Revision: 281543 URL: http://llvm.org/viewvc/llvm-project?rev=281543=rev Log: [CUDA] Add test checking our ability to take a function pointer to a __global__ function on the host side. Summary: This functionality is used by Thrust. Reviewers:

Re: [PATCH] D24513: [AMDGPU] Expose flat work group size, register and wave control attributes

kzhuravl updated the summary for this revision. kzhuravl updated this revision to Diff 71311. kzhuravl added a comment. Update docs in AttrDocs.td https://reviews.llvm.org/D24513 Files: include/clang/Basic/Attr.td include/clang/Basic/AttrDocs.td include/clang/Basic/DiagnosticSemaKinds.td

[clang-tools-extra] r281453 - [clang-tidy] Add check 'misc-use-after-move'

Author: mboehme Date: Wed Sep 14 05:29:32 2016 New Revision: 281453 URL: http://llvm.org/viewvc/llvm-project?rev=281453=rev Log: [clang-tidy] Add check 'misc-use-after-move' Summary: The check warns if an object is used after it has been moved, without an intervening reinitialization. See

Re: [libcxx] r280944 - Added 'inline' attribute to basic_string's destructor

FYI I had to revert this in r281562 because of: * http://llvm.org/PR30341 which breaks the LLVM build. * http://llvm.org/PR30391 which breaks ABI compatibility when libc++.so is compiled with GCC. I'm working on fixing these issues so that this patch may be recommitted. The GCC issue also made

[libclc] r281564 - math: Implement lgamma_r

Author: awatry Date: Wed Sep 14 19:17:28 2016 New Revision: 281564 URL: http://llvm.org/viewvc/llvm-project?rev=281564=rev Log: math: Implement lgamma_r Ported from the amd-builtins branch, which is itself based on the Sun Microsystems implementation. Signed-off-by: Aaron Watry

[libclc] r281563 - Add ADDR_SPACE parameter to _CLC_V_V_VP_VECTORIZE

Author: awatry Date: Wed Sep 14 19:17:22 2016 New Revision: 281563 URL: http://llvm.org/viewvc/llvm-project?rev=281563=rev Log: Add ADDR_SPACE parameter to _CLC_V_V_VP_VECTORIZE This macro is currently unused, but I plan to use it shortly. The previous form did casts of pointers without an

[libclc] r281566 - math: Implement tgamma

Author: awatry Date: Wed Sep 14 19:17:34 2016 New Revision: 281566 URL: http://llvm.org/viewvc/llvm-project?rev=281566=rev Log: math: Implement tgamma Signed-off-by: Aaron Watry Reviewed-by: Tom Stellard Added:

Re: [PATCH] D23765: Fix for clang PR 29087

twoh added a comment. Ping. https://reviews.llvm.org/D23765 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[libcxx] r281562 - Revert r280944 - Added 'inline' attribute to basic_string's destructor

Author: ericwf Date: Wed Sep 14 18:52:01 2016 New Revision: 281562 URL: http://llvm.org/viewvc/llvm-project?rev=281562=rev Log: Revert r280944 - Added 'inline' attribute to basic_string's destructor This patch causes a couple of issues: 1) It triggers http://llvm.org/PR30341. Although the bug

r281540 - Add some MS aliases for existing intrinsics

Author: agutowski Date: Wed Sep 14 16:19:43 2016 New Revision: 281540 URL: http://llvm.org/viewvc/llvm-project?rev=281540=rev Log: Add some MS aliases for existing intrinsics Reviewers: thakis, compnerd, majnemer, rsmith, rnk Subscribers: alexshap, cfe-commits Differential Revision:

Re: [PATCH] D24584: Do not warn about format strings that are indexed string literals.

meikeb added a comment. This is the same as https://reviews.llvm.org/D23820 besides that I added myself in the commit message as "Patch by". https://reviews.llvm.org/D23820 was reverted in https://reviews.llvm.org/D24579 because srhines' commit took authorship of this patch.

Re: [PATCH] D24513: [AMDGPU] Expose flat work group size, register and wave control attributes

kzhuravl updated this revision to Diff 71449. kzhuravl marked 15 inline comments as done. kzhuravl added a comment. Address review feedback https://reviews.llvm.org/D24513 Files: include/clang/Basic/Attr.td include/clang/Basic/AttrDocs.td include/clang/Basic/DiagnosticSemaKinds.td

Re: [PATCH] D24513: [AMDGPU] Expose flat work group size, register and wave control attributes

kzhuravl added inline comments. Comment at: include/clang/Basic/Attr.td:1067 @@ +1066,3 @@ + let Spellings = [GNU<"amdgpu_waves_per_eu">]; + let Args = [UnsignedArgument<"Min">, VariadicUnsignedArgument<"Max">]; + let Documentation = [AMDGPUWavesPerEUDocs];

Re: [PATCH] D22057: Prevent devirtualization of calls to un-instantiated functions.

Sunil_Srivastava updated this revision to Diff 71432. Sunil_Srivastava added a comment. This is an update to address points raised by Richard Smith: 1. The bit and the access functions have been renamed from MarkedForPendingInstantiation to InstantiationIsPending. 2. It closely, though not

r281440 - Document option '-rtlib' in clang's man page and help info

Author: hahnfeld Date: Wed Sep 14 00:52:21 2016 New Revision: 281440 URL: http://llvm.org/viewvc/llvm-project?rev=281440=rev Log: Document option '-rtlib' in clang's man page and help info This patch adds an entry for "-rtlib" in the output of `man clang` and `clang -help`. Patch by Lei Zhang!

Re: [PATCH] D24069: Document option '-rtlib' in clang's man page and help info

This revision was automatically updated to reflect the committed changes. Closed by commit rL281440: Document option '-rtlib' in clang's man page and help info (authored by Hahnfeld). Changed prior to commit: https://reviews.llvm.org/D24069?vs=69947=71298#toc Repository: rL LLVM

  1   2   >