[PATCH] D67208: [WebAssembly] Add -fwasm-exceptions for wasm EH

2019-09-04 Thread Heejin Ahn via Phabricator via cfe-commits
aheejin updated this revision to Diff 218834. aheejin added a comment. - Remove a blank line Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67208/new/ https://reviews.llvm.org/D67208 Files: clang/include/clang/Basic/LangOptions.def clang/includ

[PATCH] D67208: [WebAssembly] Add -fwasm-exceptions for wasm EH

2019-09-04 Thread Heejin Ahn via Phabricator via cfe-commits
aheejin created this revision. aheejin added reviewers: dschuff, tlively. Herald added subscribers: cfe-commits, sunfish, jgravelle-google, sbc100, aprantl. Herald added a project: clang. This adds `-fwasm-exceptions` (in similar fashion with `-fdwarf-exceptions` or `-fsjlj-exceptions`) that turn

[PATCH] D67058: [clang][CodeGen] Add alias for cpu_dispatch function with IFunc & Fix resolver linkage type

2019-09-04 Thread Sr.Zhang via Phabricator via cfe-commits
zsrkmyn updated this revision to Diff 218830. zsrkmyn retitled this revision from "[clang][CodeGen] Add alias for cpu_dispatch function with IFunc" to "[clang][CodeGen] Add alias for cpu_dispatch function with IFunc & Fix resolver linkage type". CHANGES SINCE LAST ACTION https://reviews.llvm.o

[PATCH] D66198: AMDGPU: Add builtins for is_local/is_private

2019-09-04 Thread Matt Arsenault via Phabricator via cfe-commits
arsenm closed this revision. arsenm added a comment. r371010 CHANGES SINCE LAST ACTION https://reviews.llvm.org/D66198/new/ https://reviews.llvm.org/D66198 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mai

r371010 - AMDGPU: Add builtins for is_shared/is_private

2019-09-04 Thread Matt Arsenault via cfe-commits
Author: arsenm Date: Wed Sep 4 20:00:43 2019 New Revision: 371010 URL: http://llvm.org/viewvc/llvm-project?rev=371010&view=rev Log: AMDGPU: Add builtins for is_shared/is_private Added: cfe/trunk/test/SemaOpenCL/builtins-amdgcn-error-flat-address-space.cl Modified: cfe/trunk/include/clang

[PATCH] D67058: [clang][CodeGen] Add alias for cpu_dispatch function with IFunc

2019-09-04 Thread Sr.Zhang via Phabricator via cfe-commits
zsrkmyn added inline comments. Comment at: clang/lib/CodeGen/CodeGenModule.cpp:2957 +if (!AliasFunc) { + auto *IFunc = cast(GetOrCreateLLVMFunction( + AliasName, DeclTy, GD, /*ForVTable=*/false, /*DontDefer=*/true, erichkeane wrote: > erichkeane

[PATCH] D65256: [Sema][ObjC] Mark C union fields that have non-trivial ObjC ownership qualifications as unavailable if the union is declared in a system header

2019-09-04 Thread Akira Hatanaka via Phabricator via cfe-commits
ahatanak added inline comments. Comment at: lib/Sema/SemaDecl.cpp:11176 for (const FieldDecl *FD : RD->fields()) - asDerived().visit(FD->getType(), FD, InNonTrivialUnion); + if (!FD->hasAttr()) +asDerived().visit(FD->getType(), FD, InNonTrivialUnion); -

[PATCH] D65256: [Sema][ObjC] Mark C union fields that have non-trivial ObjC ownership qualifications as unavailable if the union is declared in a system header

2019-09-04 Thread Akira Hatanaka via Phabricator via cfe-commits
ahatanak updated this revision to Diff 218826. ahatanak marked 2 inline comments as done. ahatanak added a comment. Address review comments. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65256/new/ https://reviews.llvm.org/D65256 Files: include/clang/AST/ASTCo

r371004 - [c++20] Fix some ambiguities in our mangling of lambdas with explicit

2019-09-04 Thread Richard Smith via cfe-commits
Author: rsmith Date: Wed Sep 4 18:23:47 2019 New Revision: 371004 URL: http://llvm.org/viewvc/llvm-project?rev=371004&view=rev Log: [c++20] Fix some ambiguities in our mangling of lambdas with explicit template parameters. This finishes the implementation of the proposal described in https://git

[PATCH] D67127: [clang-scan-deps] add skip excluded conditional preprocessor block preprocessing optimization

2019-09-04 Thread Alex Lorenz via Phabricator via cfe-commits
arphaman updated this revision to Diff 218821. arphaman added a comment. restructure the code to perform skipping directly in the preprocessor Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67127/new/ https://reviews.llvm.org/D67127 Files: clang/

[PATCH] D66569: [analyzer] ccc-analyzer: handle --sysroot=/path in addition to --sysroot /path

2019-09-04 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL371002: [analyzer] scan-build: handle --sysroot=/path in addition to --sysroot /path. (authored by dergachev, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed

r371002 - [analyzer] scan-build: handle --sysroot=/path in addition to --sysroot /path.

2019-09-04 Thread Artem Dergachev via cfe-commits
Author: dergachev Date: Wed Sep 4 17:44:56 2019 New Revision: 371002 URL: http://llvm.org/viewvc/llvm-project?rev=371002&view=rev Log: [analyzer] scan-build: handle --sysroot=/path in addition to --sysroot /path. Current code assumes flags in CompilerLinkerOptionMap don't use =, which isn't alwa

[PATCH] D67202: Implement Microsoft-compatible mangling for decomposition declarations.

2019-09-04 Thread Nico Weber via Phabricator via cfe-commits
thakis added a comment. Does this pass `ninja check-clang`? If so, can you add a test for the new mangling, based on the manglings in the godbolt example? See `clang/test/CodeGenCXX/mangle-ms-*` for many examples; a new file mangle-ms-cxx20.cpp might be appropriate here. You can run `bin/llvm-l

[PATCH] D67202: Implement Microsoft-compatible mangling for decomposition declarations.

2019-09-04 Thread Nico Weber via Phabricator via cfe-commits
thakis added a comment. ps: Thanks for the patch, this is great :) pps: Since this uses struct mangling, `llvm-undname` can already demangle these names, right? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67202/new/ https://reviews.llvm.org/D67

Re: r369943 - FileManager: Use llvm::Expected in new getFileRef API

2019-09-04 Thread David Blaikie via cfe-commits
On Mon, Aug 26, 2019 at 11:28 AM Duncan P. N. Exon Smith via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Author: dexonsmith > Date: Mon Aug 26 11:29:51 2019 > New Revision: 369943 > > URL: http://llvm.org/viewvc/llvm-project?rev=369943&view=rev > Log: > FileManager: Use llvm::Expected in ne

[PATCH] D67202: Implement Microsoft-compatible mangling for decomposition declarations.

2019-09-04 Thread Eric Astor via Phabricator via cfe-commits
epastor created this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Match cl.exe's mangling for decomposition declarations. Decomposition declarations are considered to be anonymous structs, and use the same convention as for anonymous struct/union declarations.

[PATCH] D67077: [libclang] Refactored SharedParsedRegionsStorage

2019-09-04 Thread Jan Korous via Phabricator via cfe-commits
jkorous accepted this revision. jkorous added a comment. This revision is now accepted and ready to land. Thanks for polishing this! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67077/new/ https://reviews.llvm.org/D67077 ___

[PATCH] D65917: [clang-tidy] Added check for the Google style guide's category method naming rule.

2019-09-04 Thread David Gatwood via Phabricator via cfe-commits
dgatwood updated this revision to Diff 218812. dgatwood marked 22 inline comments as done. dgatwood added a comment. Incorporated additional feedback. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65917/new/ https://reviews.llvm.org/D65917 Files: clang-tools-extra/clang-tidy/google/C

[PATCH] D67200: Add -static-openmp driver option

2019-09-04 Thread Pirama Arumuga Nainar via Phabricator via cfe-commits
pirama created this revision. pirama added reviewers: Hahnfeld, danalbert, srhines, joerg. Herald added a subscriber: guansong. Herald added a reviewer: jdoerfert. Herald added a project: clang. This option forces linking with the static OpenMP host runtime (similar to -static-libgcc and -static-l

[PATCH] D66397: [Diagnostics] Improve -Wxor-used-as-pow

2019-09-04 Thread Dávid Bolvanský via Phabricator via cfe-commits
xbolva00 marked an inline comment as done. xbolva00 added inline comments. Comment at: lib/Sema/SemaExpr.cpp:11067 + + // Do not diagnose 2 ^ 64, but allow special case (2 ^ 64) - 1. + if (SubLHS && SubRHS && (LeftSideValue != 2 || RightSideValue != 64)) xbolva

[PATCH] D67159: [clang] New __attribute__((__clang_builtin)).

2019-09-04 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman added a comment. I share in the discomfort expressed for this attribute, but I don't have a better solution in mind just yet, so I'm giving some other review feedback in the meantime. Comment at: clang/include/clang/Basic/Attr.td:596 } +def ClangBuiltinOverride

r370991 - For PR43213, track whether template parameters are implicit through

2019-09-04 Thread Richard Smith via cfe-commits
Author: rsmith Date: Wed Sep 4 15:14:50 2019 New Revision: 370991 URL: http://llvm.org/viewvc/llvm-project?rev=370991&view=rev Log: For PR43213, track whether template parameters are implicit through template instantiation so we know whether to mangle them in lambda-expressions. Modified: cf

[PATCH] D66827: Add support for MS qualifiers __ptr32, __ptr64, __sptr, __uptr.

2019-09-04 Thread Amy Huang via Phabricator via cfe-commits
akhuang updated this revision to Diff 218795. akhuang added a comment. - Change existing tests so they still pass - Fix so that it parses all the __ptr32/64 attributes instead of just the last one Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D66827

[PATCH] D66397: [Diagnostics] Improve -Wxor-used-as-pow

2019-09-04 Thread JF Bastien via Phabricator via cfe-commits
jfb added inline comments. Comment at: lib/Sema/SemaExpr.cpp:11067 + + // Do not diagnose 2 ^ 64, but allow special case (2 ^ 64) - 1. + if (SubLHS && SubRHS && (LeftSideValue != 2 || RightSideValue != 64)) xbolva00 wrote: > xbolva00 wrote: > > aaron.ballman wr

[PATCH] D64671: [clang-tidy] New check: misc-init-local-variables

2019-09-04 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman added inline comments. Comment at: clang-tools-extra/clang-tidy/cppcoreguidelines/InitVariablesCheck.cpp:76 + else if (TypePtr->isFloatingType()) { +InitializationString = " = NAN"; +AddMathInclude = true; jpakkane wrote: > aaron.ballman w

[PATCH] D66827: Add support for MS qualifiers __ptr32, __ptr64, __sptr, __uptr.

2019-09-04 Thread David Majnemer via Phabricator via cfe-commits
majnemer added inline comments. Comment at: clang/lib/AST/MicrosoftMangle.cpp:1718 QualType PointeeType) { - if (PointersAre64Bit && - (PointeeType.isNull() || !PointeeType->isFunctionType())) + // Check if this is

[PATCH] D66569: [analyzer] ccc-analyzer: handle --sysroot=/path in addition to --sysroot /path

2019-09-04 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ accepted this revision. NoQ added a comment. This revision is now accepted and ready to land. Thanks!! I wish i had time to fix D55683 so that i could have you write tests for it :) Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.o

[PATCH] D66397: [Diagnostics] Improve -Wxor-used-as-pow

2019-09-04 Thread Dávid Bolvanský via Phabricator via cfe-commits
xbolva00 added a comment. Ok, I will fix it as you suggested. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D66397/new/ https://reviews.llvm.org/D66397 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/ma

[PATCH] D66569: [analyzer] ccc-analyzer: handle --sysroot=/path in addition to --sysroot /path

2019-09-04 Thread Chris Laplante via Phabricator via cfe-commits
chris.laplante added a comment. Correct, no access so please commit for me. This is my first contribution. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D66569/new/ https://reviews.llvm.org/D66569 ___ cfe-commits mail

[PATCH] D66919: Warn about zero-parameter K&R definitions in -Wstrict-prototypes

2019-09-04 Thread Aaron Puchert via Phabricator via cfe-commits
aaronpuchert added a comment. In D66919#1658355 , @aaron.ballman wrote: > We do have numerous warnings that are default errors, you can look for > `DefaultError` in the diagnostic .td files to see the uses. Thanks, I hadn't seen that before. It seems t

[PATCH] D66397: [Diagnostics] Improve -Wxor-used-as-pow

2019-09-04 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman added inline comments. Comment at: lib/Sema/SemaExpr.cpp:11067 + + // Do not diagnose 2 ^ 64, but allow special case (2 ^ 64) - 1. + if (SubLHS && SubRHS && (LeftSideValue != 2 || RightSideValue != 64)) jfb wrote: > xbolva00 wrote: > > xbolva00 wr

[PATCH] D66569: [analyzer] ccc-analyzer: handle --sysroot=/path in addition to --sysroot /path

2019-09-04 Thread Chris Laplante via Phabricator via cfe-commits
chris.laplante added a comment. In D66569#1658413 , @NoQ wrote: > I wish i had time to fix D55683 so that i > could have you write tests for it :) I just subscribed to it, so if/when you do, I'll add some tests :) Rep

[PATCH] D58094: Fix -Wnonportable-include-path suppression for header maps with absolute paths.

2019-09-04 Thread Volodymyr Sapsai via Phabricator via cfe-commits
vsapsai updated this revision to Diff 218794. vsapsai added a comment. Herald added a subscriber: ributzka. - Rebase the patch. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D58094/new/ https://reviews.llvm.org/D58094 Files: clang/include/clang/Lex/DirectoryLookup.h clang/lib/Lex/He

[PATCH] D66397: [Diagnostics] Improve -Wxor-used-as-pow

2019-09-04 Thread Dávid Bolvanský via Phabricator via cfe-commits
xbolva00 marked an inline comment as done. xbolva00 added inline comments. Comment at: lib/Sema/SemaExpr.cpp:11067 + + // Do not diagnose 2 ^ 64, but allow special case (2 ^ 64) - 1. + if (SubLHS && SubRHS && (LeftSideValue != 2 || RightSideValue != 64)) aaron.

[PATCH] D66569: [analyzer] ccc-analyzer: handle --sysroot=/path in addition to --sysroot /path

2019-09-04 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ added a comment. You don't have commit access, right? Do you want me to commit this patch for you? Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D66569/new/ https://reviews.llvm.org/D66569 ___ cfe-commits mailing

[PATCH] D58094: Fix -Wnonportable-include-path suppression for header maps with absolute paths.

2019-09-04 Thread Volodymyr Sapsai via Phabricator via cfe-commits
vsapsai added a comment. Ping. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D58094/new/ https://reviews.llvm.org/D58094 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D66397: [Diagnostics] Improve -Wxor-used-as-pow

2019-09-04 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman added inline comments. Comment at: lib/Sema/SemaExpr.cpp:11067 + + // Do not diagnose 2 ^ 64, but allow special case (2 ^ 64) - 1. + if (SubLHS && SubRHS && (LeftSideValue != 2 || RightSideValue != 64)) xbolva00 wrote: > xbolva00 wrote: > > aaron.

[PATCH] D65917: [clang-tidy] Added check for the Google style guide's category method naming rule.

2019-09-04 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman added inline comments. Comment at: clang-tools-extra/clang-tidy/google/RequireCategoryMethodPrefixesCheck.cpp:16 +namespace { +const char *kCustomCategoryMethodIdentifier = "ThisIsACategoryMethod"; +} // anonymous namespace Eugene.Zelenko wrote: >

[PATCH] D67122: [UBSan][clang][compiler-rt] Applying non-zero offset to nullptr is undefined behaviour

2019-09-04 Thread Roman Lebedev via Phabricator via cfe-commits
lebedev.ri added inline comments. Comment at: clang/lib/CodeGen/CGExprScalar.cpp:4538 + llvm::Value * /*OffsetOverflows*/> +EmitGEPOffsetInBytes(Value *BasePtr, Value *GEPVal, + llvm::LLVMContext &VMContext, CodeGenModule &CGM,

[PATCH] D67122: [UBSan][clang][compiler-rt] Applying non-zero offset to nullptr is undefined behaviour

2019-09-04 Thread Roman Lebedev via Phabricator via cfe-commits
lebedev.ri updated this revision to Diff 218792. lebedev.ri marked 25 inline comments as done. lebedev.ri added a comment. @vsk thank you for taking a look! I believe i have addressed or replied to all the comments. Other than the optimization comment, i currently don't believe that is a sound tr

[PATCH] D66397: [Diagnostics] Improve -Wxor-used-as-pow

2019-09-04 Thread Dávid Bolvanský via Phabricator via cfe-commits
xbolva00 marked an inline comment as done. xbolva00 added inline comments. Comment at: lib/Sema/SemaExpr.cpp:11067 + + // Do not diagnose 2 ^ 64, but allow special case (2 ^ 64) - 1. + if (SubLHS && SubRHS && (LeftSideValue != 2 || RightSideValue != 64)) xbolva

[PATCH] D66397: [Diagnostics] Improve -Wxor-used-as-pow

2019-09-04 Thread Dávid Bolvanský via Phabricator via cfe-commits
xbolva00 marked an inline comment as done. xbolva00 added inline comments. Comment at: lib/Sema/SemaExpr.cpp:11067 + + // Do not diagnose 2 ^ 64, but allow special case (2 ^ 64) - 1. + if (SubLHS && SubRHS && (LeftSideValue != 2 || RightSideValue != 64)) aaron.

r370987 - [www] Fix hyperlink syntax in attribute reference.

2019-09-04 Thread Richard Smith via cfe-commits
Author: rsmith Date: Wed Sep 4 14:31:22 2019 New Revision: 370987 URL: http://llvm.org/viewvc/llvm-project?rev=370987&view=rev Log: [www] Fix hyperlink syntax in attribute reference. Modified: cfe/trunk/include/clang/Basic/AttrDocs.td Modified: cfe/trunk/include/clang/Basic/AttrDocs.td URL:

r370986 - [www] Update attribute reference for 'constinit'.

2019-09-04 Thread Richard Smith via cfe-commits
Author: rsmith Date: Wed Sep 4 14:31:21 2019 New Revision: 370986 URL: http://llvm.org/viewvc/llvm-project?rev=370986&view=rev Log: [www] Update attribute reference for 'constinit'. Modified: cfe/trunk/include/clang/Basic/Attr.td cfe/trunk/include/clang/Basic/AttrDocs.td Modified: cfe/t

[PATCH] D67140: [analyzer][NFC] Fix inconsistent references to checkers as "checks"

2019-09-04 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman added a comment. In D67140#1658353 , @gribozavr wrote: > In D67140#1658315 , @aaron.ballman > wrote: > > > In D67140#1656831 , @NoQ wrote: > > > > > Honestly,

[PATCH] D66919: Warn about zero-parameter K&R definitions in -Wstrict-prototypes

2019-09-04 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman added a comment. In D66919#1655052 , @aaronpuchert wrote: > In D66919#1655048 , @dexonsmith > wrote: > > > I went back to read notes from when we deployed `-Wstrict-prototypes` > > (which we have ha

[PATCH] D66397: [Diagnostics] Improve -Wxor-used-as-pow

2019-09-04 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman added inline comments. Comment at: lib/Sema/SemaExpr.cpp:11067 + + // Do not diagnose 2 ^ 64, but allow special case (2 ^ 64) - 1. + if (SubLHS && SubRHS && (LeftSideValue != 2 || RightSideValue != 64)) aaron.ballman wrote: > This comment explains

[PATCH] D67140: [analyzer][NFC] Fix inconsistent references to checkers as "checks"

2019-09-04 Thread Dmitri Gribenko via Phabricator via cfe-commits
gribozavr added a comment. In D67140#1658315 , @aaron.ballman wrote: > In D67140#1656831 , @NoQ wrote: > > > Honestly, i'm much more worried about message capitalization :) > > > Likewise. I wish the static analyze

[PATCH] D67135: [clang-tidy] performance-inefficient-vector-operation: Support proto repeated field

2019-09-04 Thread Cong Liu via Phabricator via cfe-commits
congliu updated this revision to Diff 218789. congliu added a comment. Uploaded the correct diff... CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67135/new/ https://reviews.llvm.org/D67135 Files: clang-tools-extra/clang-tidy/performance/InefficientVectorOperationCheck.cpp clang-too

[PATCH] D64644: [Sema] Fixes an assertion failure while instantiation a template with an incomplete typo corrected type

2019-09-04 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman added inline comments. Comment at: clang/lib/Sema/SemaTemplate.cpp:727 return DependentScopeDeclRefExpr::Create( - Context, SS.getWithLocInContext(Context), TemplateKWLoc, NameInfo, - TemplateArgs); + Context, std::move(QualifierLoc), TemplateKWLoc

r370982 - Diagnose _Atomic as a C11 extension.

2019-09-04 Thread Aaron Ballman via cfe-commits
Author: aaronballman Date: Wed Sep 4 14:01:57 2019 New Revision: 370982 URL: http://llvm.org/viewvc/llvm-project?rev=370982&view=rev Log: Diagnose _Atomic as a C11 extension. Modified: cfe/trunk/lib/Parse/ParseDecl.cpp cfe/trunk/test/Parser/atomic.c cfe/trunk/test/SemaCXX/atomic-type

[PATCH] D67141: [DebugInfo] Emit DW_TAG_enumeration_type for referenced global enumerator.

2019-09-04 Thread Yuanfang Chen via Phabricator via cfe-commits
ychen added a comment. In D67141#1658238 , @rnk wrote: > lgtm, thanks! Thank you! Repository: rL LLVM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67141/new/ https://reviews.llvm.org/D67141 __

[PATCH] D67141: [DebugInfo] Emit DW_TAG_enumeration_type for referenced global enumerator.

2019-09-04 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL370981: [DebugInfo] Emit DW_TAG_enumeration_type for referenced global enumerator. (authored by yuanfang, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed pri

[PATCH] D67127: [clang-scan-deps] add skip excluded conditional preprocessor block preprocessing optimization

2019-09-04 Thread Michael Spencer via Phabricator via cfe-commits
Bigcheese added a comment. Looking more into this I'm not sure you need the PPCallbacks. The Preprocessor should just own the PreprocessorSkippedMappings data structure and the dependency scanner can update it live. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://re

[PATCH] D65744: [PR42707][OpenCL] Fix addr space deduction for auto

2019-09-04 Thread John McCall via Phabricator via cfe-commits
rjmccall added inline comments. Comment at: clang/lib/Sema/SemaType.cpp:7460 + // the initializing expression type during the type deduction. + (T->isUndeducedAutoType() && IsPointee) || // OpenCL spec v2.0 s6.9.b: Okay, I understand why you're d

[PATCH] D67023: Diagnose use of ATOMIC_VAR_INIT

2019-09-04 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman added a comment. In D67023#1656453 , @jfb wrote: > In D67023#1654704 , @aaron.ballman > wrote: > > > In D67023#1654070 , @jfb wrote: > > > > > I refer you to h

r370981 - [DebugInfo] Emit DW_TAG_enumeration_type for referenced global enumerator.

2019-09-04 Thread Yuanfang Chen via cfe-commits
Author: yuanfang Date: Wed Sep 4 13:58:15 2019 New Revision: 370981 URL: http://llvm.org/viewvc/llvm-project?rev=370981&view=rev Log: [DebugInfo] Emit DW_TAG_enumeration_type for referenced global enumerator. This essentially reverts changes from r361400 while keeping behavior for CodeView. Rev

[PATCH] D67135: [clang-tidy] performance-inefficient-vector-operation: Support proto repeated field

2019-09-04 Thread Cong Liu via Phabricator via cfe-commits
congliu added a comment. In D67135#1657034 , @lebedev.ri wrote: > I'm not sure this is the correct approach. > I'd think the check instead should be parametrized, so this patch should > become just extending the config. Could you elaborate how the para

[PATCH] D67140: [analyzer][NFC] Fix inconsistent references to checkers as "checks"

2019-09-04 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman added a comment. In D67140#1656831 , @NoQ wrote: > Honestly, i'm much more worried about message capitalization :) Likewise. I wish the static analyzer would follow the usual conventions followed by clang and clang-tidy. ;-) In D67140#165

[PATCH] D67135: [clang-tidy] performance-inefficient-vector-operation: Support proto repeated field

2019-09-04 Thread Cong Liu via Phabricator via cfe-commits
congliu updated this revision to Diff 218782. congliu added a comment. Updated option descriptions. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67135/new/ https://reviews.llvm.org/D67135 Files: clang-tools-extra/docs/clang-tidy/checks/performance-inefficient-vector-operation.rst

[PATCH] D66988: [NewPM][Sancov] Make Sancov a Module Pass instead of 2 Passes

2019-09-04 Thread Leonard Chan via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL370971: [NewPM][Sancov] Make Sancov a Module Pass instead of 2 Passes (authored by leonardchan, committed by ). Changed prior to commit: https://reviews.llvm.org/D66988?vs=218008&id=218781#toc Reposito

r370972 - [c++20] P1143R2: Add support for the C++20 'constinit' keyword.

2019-09-04 Thread Richard Smith via cfe-commits
Author: rsmith Date: Wed Sep 4 13:30:37 2019 New Revision: 370972 URL: http://llvm.org/viewvc/llvm-project?rev=370972&view=rev Log: [c++20] P1143R2: Add support for the C++20 'constinit' keyword. This is mostly the same as the [[clang::require_constant_initialization]] attribute, but has a coupl

r370970 - Generate parent context id from Decl* instead of DeclContext*.

2019-09-04 Thread Aaron Ballman via cfe-commits
Author: aaronballman Date: Wed Sep 4 13:30:00 2019 New Revision: 370970 URL: http://llvm.org/viewvc/llvm-project?rev=370970&view=rev Log: Generate parent context id from Decl* instead of DeclContext*. Because of multiple inheritance, a DeclContext pointer does not produce the same pointer repres

[PATCH] D67057: [AST][JSON] Generate parent context id from Decl* instead of DeclContext*

2019-09-04 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman closed this revision. aaron.ballman added a comment. In D67057#1658217 , @piscisaureus wrote: > Fixed nit, this is ready to land IMO. Thanks! I've commit on your behalf in r370970. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67

r370971 - [NewPM][Sancov] Make Sancov a Module Pass instead of 2 Passes

2019-09-04 Thread Leonard Chan via cfe-commits
Author: leonardchan Date: Wed Sep 4 13:30:29 2019 New Revision: 370971 URL: http://llvm.org/viewvc/llvm-project?rev=370971&view=rev Log: [NewPM][Sancov] Make Sancov a Module Pass instead of 2 Passes This patch merges the sancov module and funciton passes into one module pass. The reason for thi

[PATCH] D66328: [DebugInfo] Add debug location to stubs generated by CGDeclCXX and mark them as artificial

2019-09-04 Thread Reid Kleckner via Phabricator via cfe-commits
rnk accepted this revision. rnk added a comment. This revision is now accepted and ready to land. lgtm with the GlobalArrayDestructor name. Comment at: include/clang/AST/GlobalDecl.h:34 AtExit, + GlobalDestructor }; Maybe we should rename this GlobalArrayD

[PATCH] D67031: [Clang][Bundler] Error reporting improvements

2019-09-04 Thread Sergey Dmitriev via Phabricator via cfe-commits
sdmitriev marked an inline comment as done. sdmitriev added inline comments. Comment at: clang/tools/clang-offload-bundler/ClangOffloadBundler.cpp:41-44 -#include -#include -#include -#include Hahnfeld wrote: > sdmitriev wrote: > > Hahnfeld wrote: > > > sdmit

[PATCH] D67127: [clang-scan-deps] add skip excluded conditional preprocessor block preprocessing optimization

2019-09-04 Thread Alex Lorenz via Phabricator via cfe-commits
arphaman added inline comments. Comment at: clang/include/clang/Frontend/CompilerInstance.h:256 + /// preprocessor. + void setAdditionalPPCallbacks(std::unique_ptr PPC); + Bigcheese wrote: > This kinda sounds like it can be called multiple times. Is there any

[PATCH] D67031: [Clang][Bundler] Error reporting improvements

2019-09-04 Thread Sergey Dmitriev via Phabricator via cfe-commits
sdmitriev updated this revision to Diff 218776. sdmitriev edited the summary of this revision. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67031/new/ https://reviews.llvm.org/D67031 Files: clang/test/Driver/clang-offload-bundler.c clang/tools/clang-offload-bundler/ClangOffloadBundl

[PATCH] D67127: [clang-scan-deps] add skip excluded conditional preprocessor block preprocessing optimization

2019-09-04 Thread Alex Lorenz via Phabricator via cfe-commits
arphaman updated this revision to Diff 218778. arphaman marked 2 inline comments as done. arphaman added a comment. Add support for chained PP callbacks in setAdditionalPPCallbacks. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67127/new/ https://r

[PATCH] D67141: [DebugInfo] Emit DW_TAG_enumeration_type for referenced global enumerator.

2019-09-04 Thread Reid Kleckner via Phabricator via cfe-commits
rnk accepted this revision. rnk added a comment. This revision is now accepted and ready to land. lgtm, thanks! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67141/new/ https://reviews.llvm.org/D67141

[PATCH] D65256: [Sema][ObjC] Mark C union fields that have non-trivial ObjC ownership qualifications as unavailable if the union is declared in a system header

2019-09-04 Thread John McCall via Phabricator via cfe-commits
rjmccall added inline comments. Comment at: include/clang/AST/ASTContext.h:2060 + /// attr::ObjCOwnership implies an ownership qualifier was explicitly + /// specified rather than being added implicitly by the compiler. + bool isObjCOwnershipAttributedType(QualType Ty) const;

[PATCH] D67057: [AST][JSON] Generate parent context id from Decl* instead of DeclContext*

2019-09-04 Thread Bert Belder via Phabricator via cfe-commits
piscisaureus marked an inline comment as done. piscisaureus added a comment. Fixed nit, this is ready to land IMO. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67057/new/ https://reviews.llvm.org/D67057 ___ cfe-commits mailing list cfe-com

[PATCH] D67057: [AST][JSON] Generate parent context id from Decl* instead of DeclContext*

2019-09-04 Thread Bert Belder via Phabricator via cfe-commits
piscisaureus updated this revision to Diff 218770. piscisaureus added a comment. fix nit CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67057/new/ https://reviews.llvm.org/D67057 Files: clang/lib/AST/JSONNodeDumper.cpp clang/test/AST/ast-dump-decl-context-json.cpp clang/test/AST/a

[PATCH] D67141: [DebugInfo] Emit DW_TAG_enumeration_type for referenced global enumerator.

2019-09-04 Thread Yuanfang Chen via Phabricator via cfe-commits
ychen updated this revision to Diff 218769. ychen added a comment. - address comment Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67141/new/ https://reviews.llvm.org/D67141 Files: clang/lib/CodeGen/CGDebugInfo.cpp clang/test/CodeGen/enum2.c

[PATCH] D67031: [Clang][Bundler] Error reporting improvements

2019-09-04 Thread Jonas Hahnfeld via Phabricator via cfe-commits
Hahnfeld added inline comments. Comment at: clang/tools/clang-offload-bundler/ClangOffloadBundler.cpp:41-44 -#include -#include -#include -#include sdmitriev wrote: > Hahnfeld wrote: > > sdmitriev wrote: > > > Hahnfeld wrote: > > > > sdmitriev wrote: > > > >

[PATCH] D67031: [Clang][Bundler] Error reporting improvements

2019-09-04 Thread Sergey Dmitriev via Phabricator via cfe-commits
sdmitriev marked an inline comment as done. sdmitriev added inline comments. Comment at: clang/tools/clang-offload-bundler/ClangOffloadBundler.cpp:41-44 -#include -#include -#include -#include Hahnfeld wrote: > sdmitriev wrote: > > Hahnfeld wrote: > > > sdmit

[PATCH] D67185: [RISCV] Add support for -ffixed-xX flags

2019-09-04 Thread James Clarke via Phabricator via cfe-commits
jrtc27 added inline comments. Comment at: llvm/test/CodeGen/RISCV/reserved-regs.ll:71 + +; X1-NOT: lw ra, +; X1-NOT: ld ra, lenary wrote: > These tests aren't going to test what you think they are, or at least aren't > going to fail when you hope they are, as th

[PATCH] D65256: [Sema][ObjC] Mark C union fields that have non-trivial ObjC ownership qualifications as unavailable if the union is declared in a system header

2019-09-04 Thread Akira Hatanaka via Phabricator via cfe-commits
ahatanak added inline comments. Comment at: include/clang/AST/ASTContext.h:2060 + /// attr::ObjCOwnership implies an ownership qualifier was explicitly + /// specified rather than being added implicitly by the compiler. + bool isObjCOwnershipAttributedType(QualType Ty) const;

[PATCH] D65256: [Sema][ObjC] Mark C union fields that have non-trivial ObjC ownership qualifications as unavailable if the union is declared in a system header

2019-09-04 Thread Akira Hatanaka via Phabricator via cfe-commits
ahatanak updated this revision to Diff 218767. ahatanak marked 3 inline comments as done. ahatanak added a comment. Address review comments. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65256/new/ https://reviews.llvm.org/D65256 Files: include/clang/AST/ASTCo

[PATCH] D66328: [DebugInfo] Add debug location to stubs generated by CGDeclCXX and mark them as artificial

2019-09-04 Thread Reid Kleckner via Phabricator via cfe-commits
rnk added a comment. I'm trying to figure out what exactly the new GlobalDestructor thing is. :) It's not clear to me why we ever emit `__cxx_global_array_dtor` in the MS ABI. We always call it directly from the `?__F` atexit helper stub instead of registering it, so we really don't need it at

[PATCH] D67122: [UBSan][clang][compiler-rt] Applying non-zero offset to nullptr is undefined behaviour

2019-09-04 Thread Vedant Kumar via Phabricator via cfe-commits
vsk added a subscriber: dtzWill. vsk added a comment. Thanks, this is looking pretty good. Comment at: clang/lib/CodeGen/CGExprScalar.cpp:4538 + llvm::Value * /*OffsetOverflows*/> +EmitGEPOffsetInBytes(Value *BasePtr, Value *GEPVal, + llvm::L

[PATCH] D67031: [Clang][Bundler] Error reporting improvements

2019-09-04 Thread Jonas Hahnfeld via Phabricator via cfe-commits
Hahnfeld added a comment. Also, there should be a summary of the changes in here. Comment at: clang/tools/clang-offload-bundler/ClangOffloadBundler.cpp:41-44 -#include -#include -#include -#include sdmitriev wrote: > Hahnfeld wrote: > > sdmitriev wrote: > >

[PATCH] D67141: [DebugInfo] Emit DW_TAG_enumeration_type for referenced global enumerator.

2019-09-04 Thread Reid Kleckner via Phabricator via cfe-commits
rnk added inline comments. Comment at: clang/lib/CodeGen/CGDebugInfo.cpp:4441-4462 + // Use global variables for enums in CodeView, use DW_TAG_enumeration_type for + // enums for non-CodeView. if (const auto *ECD = dyn_cast(VD)) { const auto *ED = cast(ECD->getDeclCont

[PATCH] D67031: [Clang][Bundler] Error reporting improvements

2019-09-04 Thread Sergey Dmitriev via Phabricator via cfe-commits
sdmitriev updated this revision to Diff 218755. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67031/new/ https://reviews.llvm.org/D67031 Files: clang/test/Driver/clang-offload-bundler.c clang/tools/clang-offload-bundler/ClangOffloadBundler.cpp Index: clang/tools/clang-offload-bundle

[PATCH] D64146: [Clang Interpreter] Initial patch for the constexpr interpreter

2019-09-04 Thread Nandor Licker via Phabricator via cfe-commits
nand added a reviewer: rnk. nand added a subscriber: rnk. nand added a comment. @jfb suggested I add @rnk: I was wondering if you have any suggestions on how to fix the msvc warnings/errors? I'd be grateful if I had some feedback on what features of C++ I should avoid using and what I should rep

[PATCH] D67141: [DebugInfo] Emit DW_TAG_enumeration_type for referenced global enumerator.

2019-09-04 Thread Amy Huang via Phabricator via cfe-commits
akhuang added a comment. What is the reason for this change? The test case and comment changes don't seem related to the actual code change, which would affect `DIGlobalVariable`s. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67141/new/ https://r

[PATCH] D67031: [Clang][Bundler] Error reporting improvements

2019-09-04 Thread Sergey Dmitriev via Phabricator via cfe-commits
sdmitriev marked an inline comment as done. sdmitriev added inline comments. Comment at: clang/tools/clang-offload-bundler/ClangOffloadBundler.cpp:41-44 -#include -#include -#include -#include Hahnfeld wrote: > sdmitriev wrote: > > Hahnfeld wrote: > > > The c

[PATCH] D67031: [Clang][Bundler] Error reporting improvements

2019-09-04 Thread Jonas Hahnfeld via Phabricator via cfe-commits
Hahnfeld added inline comments. Comment at: clang/tools/clang-offload-bundler/ClangOffloadBundler.cpp:41-44 -#include -#include -#include -#include sdmitriev wrote: > Hahnfeld wrote: > > The code still uses (in the order of marked includes) > > * `std::uniqu

[PATCH] D67031: [Clang][Bundler] Error reporting improvements

2019-09-04 Thread Sergey Dmitriev via Phabricator via cfe-commits
sdmitriev marked an inline comment as done. sdmitriev added a comment. Any comments? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67031/new/ https://reviews.llvm.org/D67031 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://li

[PATCH] D67056: Add a bugprone-argument-comment option: IgnoreSingleArgument.

2019-09-04 Thread Yubo Xie via Phabricator via cfe-commits
xyb marked 2 inline comments as done. xyb added inline comments. Comment at: clang-tools-extra/clang-tidy/bugprone/ArgumentCommentCheck.cpp:28 + IgnoreSingleArgument( + Options.getLocalOrGlobal("IgnoreSingleArgument", 0) != 0), CommentBoolLiterals(Options.get

[PATCH] D67174: Rename of constants in ASTImporterVisibilityTest. NFC.

2019-09-04 Thread Shafik Yaghmour via Phabricator via cfe-commits
shafik accepted this revision. shafik added a comment. This revision is now accepted and ready to land. Thank you! LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67174/new/ https://reviews.llvm.org/D67174

[PATCH] D67056: Add a bugprone-argument-comment option: IgnoreSingleArgument.

2019-09-04 Thread Yubo Xie via Phabricator via cfe-commits
xyb updated this revision to Diff 218741. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67056/new/ https://reviews.llvm.org/D67056 Files: clang-tools-extra/clang-tidy/bugprone/ArgumentCommentCheck.cpp clang-tools-extra/clang-tidy/bugprone/ArgumentCommentCheck.h clang-tools-extra/do

[PATCH] D64375: [OpenMP][Docs] Provide implementation status details

2019-09-04 Thread Johannes Doerfert via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. jdoerfert marked an inline comment as done. Closed by commit rL370930: [OpenMP][Docs] Provide implementation status details (authored by jdoerfert, committed by ). Herald added a project: LLVM. Herald added a subscriber: llv

r370930 - [OpenMP][Docs] Provide implementation status details

2019-09-04 Thread Johannes Doerfert via cfe-commits
Author: jdoerfert Date: Wed Sep 4 10:15:37 2019 New Revision: 370930 URL: http://llvm.org/viewvc/llvm-project?rev=370930&view=rev Log: [OpenMP][Docs] Provide implementation status details This adds a more fine-grained list of OpenMP features with their implementation status and associated review

[PATCH] D65527: Avoid assemble step in verbose-output-quoting.c

2019-09-04 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL370928: Avoid assemble step in verbose-output-quoting.c (authored by yuanfang, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit: https://rev

r370928 - Avoid assemble step in verbose-output-quoting.c

2019-09-04 Thread Yuanfang Chen via cfe-commits
Author: yuanfang Date: Wed Sep 4 10:10:18 2019 New Revision: 370928 URL: http://llvm.org/viewvc/llvm-project?rev=370928&view=rev Log: Avoid assemble step in verbose-output-quoting.c Reviewers: hans Subscribers: cfe-commits Tags: #clang Differential Revision: https://reviews.llvm.org/D65527 M

[PATCH] D67185: [RISCV] Add support for -ffixed-xX flags

2019-09-04 Thread Sam Elliott via Phabricator via cfe-commits
lenary added a comment. I don't quite understand all the details of this patch. I understand reserving registers that the compiler would otherwise be using as general-purpose registers. But what do we do about using registers within the calling convention when someone says they should be reser

[PATCH] D67084: [clang-tidy] Fix bugprone-argument-comment bug: negative literal number is not checked.

2019-09-04 Thread Alexander Kornienko via Phabricator via cfe-commits
alexfh added a comment. In D67084#1657534 , @xyb wrote: > Thanks. BTW, I can't commit the patch by myself. The patch doesn't apply cleanly. Please rebase it against current HEAD. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67084/new/ https:/

[PATCH] D67056: Add a bugprone-argument-comment option: IgnoreSingleArgument.

2019-09-04 Thread Alexander Kornienko via Phabricator via cfe-commits
alexfh added inline comments. Comment at: clang-tools-extra/clang-tidy/bugprone/ArgumentCommentCheck.cpp:28 + IgnoreSingleArgument( + Options.getLocalOrGlobal("IgnoreSingleArgument", 0) != 0), CommentBoolLiterals(Options.getLocalOrGlobal("CommentBoolLiterals"

  1   2   >