[PATCH] D63518: BitStream reader: propagate errors

abrachet added inline comments. Comment at: llvm/trunk/lib/Bitcode/Reader/BitcodeReader.cpp:205 + return MaybeBitCode.takeError(); +switch (unsigned BitCode = MaybeBitCode.get()) { default: // Default behavior: reject This and an identical switch on

[PATCH] D63048: Update __VERSION__ to remove the hardcoded 4.2.1 version

sylvestre.ledru added a comment. @rnk how do you feel about removing both? I can take care of that if you want Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63048/new/ https://reviews.llvm.org/D63048 ___ cfe-commits

[PATCH] D63821: [clangd] Added C++ API code for semantic highlighting

jvikstrom updated this revision to Diff 206792. jvikstrom added a comment. Herald added a subscriber: jfb. Moved semantic highlighting to be processed in onMainAST Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63821/new/ https://reviews.llvm.org/D6

[PATCH] D63821: [clangd] Added C++ API code for semantic highlighting

jvikstrom marked 2 inline comments as done. jvikstrom added inline comments. Comment at: clang-tools-extra/clangd/ClangdServer.h:60 + // Called by ClangdServer when some \p Highlightings for \p File are ready. + virtual void onHighlightingsReady(PathRef File, +

[PATCH] D63821: [clangd] Added C++ API code for semantic highlighting

nik added inline comments. Comment at: clang-tools-extra/clangd/ClangdServer.h:60 + // Called by ClangdServer when some \p Highlightings for \p File are ready. + virtual void onHighlightingsReady(PathRef File, + std::vector Highlightings) = 0; -

[PATCH] D63821: [clangd] Added C++ API code for semantic highlighting

hokein added a comment. thanks, it is much clearer now. Could you please make the commit message be more specific what this patch does? C++ APIs is too generous, (we already have C++ APIs and data structures for semantic highlightings which are in `SemanticHighlighting.h`).

[PATCH] D60499: [ASTImporter] Various source location and range import fixes.

balazske added a comment. Ping Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60499/new/ https://reviews.llvm.org/D60499 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/l

[clang-tools-extra] r364519 - [clangd] Address limitations in SelectionTree:

Author: sammccall Date: Thu Jun 27 04:17:13 2019 New Revision: 364519 URL: http://llvm.org/viewvc/llvm-project?rev=364519&view=rev Log: [clangd] Address limitations in SelectionTree: Summary: - nodes can have special-cased hit ranges including "holes" (FunctionTypeLoc in void foo()) - token co

[PATCH] D63760: [clangd] Address limitations in SelectionTree:

This revision was automatically updated to reflect the committed changes. sammccall marked 4 inline comments as done. Closed by commit rL364519: [clangd] Address limitations in SelectionTree: (authored by sammccall, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-comm

[PATCH] D63845: [WIP] Create a clang attribute that lets users specify LLVM attributes

aaron.ballman added a reviewer: aaron.ballman. aaron.ballman added a comment. In D63845#1559995 , @lebedev.ri wrote: > What's the target use-case here? What can't be solved with normal attributes? > I wonder if this should go to cfe+llvm -dev lists first,

[PATCH] D63821: [clangd] Added C++ API code for semantic highlighting

jvikstrom updated this revision to Diff 206832. jvikstrom marked 9 inline comments as done. jvikstrom added a comment. Separated test and gave consumer an empty definition. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63821/new/ https://reviews.ll

[PATCH] D63082: [Diagnostics] Added support for -Wint-in-bool-context

aaron.ballman added a comment. In terms of the code, I think this is ready to go. However, I'm still not happy with `'*' in boolean context` as a diagnostic. Perhaps appending something about the resulting value always being true|false would help most of these diagnostics be more obvious. `blah

[PATCH] D63872: [clangd] Fix a case where we fail to detect a header-declared symbol in rename.

hokein created this revision. hokein added a reviewer: sammccall. Herald added subscribers: kadircet, arphaman, jkorous, MaskRay, ilya-biryukov. Herald added a project: clang. Failing case: #include "foo.h" void fo^o() {} getRenameDecl() returns the decl of the symbol under the cursor (which

[PATCH] D62960: SVE opaque type for C intrinsics demo

rovka added a comment. Just a few nits/suggestions. Comment at: include/clang/Basic/AArch64SVEACLETypes.def:10 +// +// This file defines the database about various builtin singleton types. +// You can be more specific :) Comment at: include/

[PATCH] D63276: [AST] Add FunctionDecl::getParametersSourceRange()

aaron.ballman added inline comments. Comment at: clang/include/clang/AST/Decl.h:2331 + /// Attempt to compute an informative source range covering the + /// function parameters. This omits the ellipsis of a variadic function. + SourceRange getParametersSourceRange() const; ---

[PATCH] D63874: [clangd] No need to setTraversalScope in SemanticHighlighting.

hokein created this revision. hokein added a reviewer: jvikstrom. Herald added subscribers: kadircet, arphaman, jkorous, MaskRay, ilya-biryukov. Herald added a project: clang. We have already set it when the AST is being built, and setting TraversalScope is not free (it will clear the cache, which

[PATCH] D63082: [Diagnostics] Added support for -Wint-in-bool-context

xbolva00 added a comment. I wanted to follow GCC’s behaviour -Wall but then dicussion moved to “tautological compare group” - I was fine with that too.. I can again revert it to -Wall... If this should be off by default even on -Wall, then all work was useless.. CHANGES SINCE LAST ACTION http

[PATCH] D63872: [clangd] Fix a case where we fail to detect a header-declared symbol in rename.

sammccall added inline comments. Comment at: clang-tools-extra/clangd/refactor/Rename.cpp:157 assert(RenameDecl && "symbol must be found at this point"); + RenameDecl = cast(RenameDecl->getCanonicalDecl()); if (auto Reject = renamableWithinFile(*RenameDecl, File, Index)) {

[PATCH] D63874: [clangd] No need to setTraversalScope in SemanticHighlighting.

jvikstrom accepted this revision. jvikstrom added a comment. This revision is now accepted and ready to land. lgtm Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63874/new/ https://reviews.llvm.org/D63874 _

[clang-tools-extra] r364528 - [clangd] No need to setTraversalScope in SemanticHighlighting.

Author: hokein Date: Thu Jun 27 05:22:18 2019 New Revision: 364528 URL: http://llvm.org/viewvc/llvm-project?rev=364528&view=rev Log: [clangd] No need to setTraversalScope in SemanticHighlighting. Summary: We have already set it when the AST is being built, and setting TraversalScope is not free (

[PATCH] D63874: [clangd] No need to setTraversalScope in SemanticHighlighting.

This revision was automatically updated to reflect the committed changes. Closed by commit rL364528: [clangd] No need to setTraversalScope in SemanticHighlighting. (authored by hokein, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit:

[PATCH] D63082: [Diagnostics] Added support for -Wint-in-bool-context

xbolva00 added a comment. “Perhaps appending something about the resulting value always being true|false would help most of these diagnostics be more obvious” I dont know, this could only diagnose constant multiplications (maybe this is already handled by some “always true” check). Anyway, I ha

[PATCH] D63821: [clangd] Added C++ API code for semantic highlighting

hokein added inline comments. Comment at: clang-tools-extra/clangd/ClangdLSPServer.h:60 + void + onHighlightingsReady(PathRef File, + std::vector Highlightings) override; nit: you can remove this override, since we have provided an empty d

[PATCH] D63559: [clangd] Added functionality for getting semantic highlights for variable and function declarations

hokein closed this revision. hokein added a comment. the patch was landed in rL364421 , not sure why it was not associated with this review. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63559/new/ https://review

[PATCH] D63082: [Diagnostics] Added support for -Wint-in-bool-context

xbolva00 added a comment. Alternative “* in boolean context; did you mean to compare it?” CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63082/new/ https://reviews.llvm.org/D63082 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https

[clang-tools-extra] r364535 - [clang-tidy] Fix NDEBUG build [NFC]

Author: uabelho Date: Thu Jun 27 05:47:57 2019 New Revision: 364535 URL: http://llvm.org/viewvc/llvm-project?rev=364535&view=rev Log: [clang-tidy] Fix NDEBUG build [NFC] Modified: clang-tools-extra/trunk/clang-tidy/utils/TransformerClangTidyCheck.cpp Modified: clang-tools-extra/trunk/clang-t

[PATCH] D63821: [clangd] Added C++ API code for semantic highlighting

jvikstrom updated this revision to Diff 206841. jvikstrom marked an inline comment as done. jvikstrom added a comment. Simplified test. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63821/new/ https://reviews.llvm.org/D63821 Files: clang-tools-e

[PATCH] D63821: [clangd] Added C++ API code for semantic highlighting

jvikstrom marked an inline comment as done. jvikstrom added inline comments. Comment at: clang-tools-extra/clangd/ClangdLSPServer.h:60 + void + onHighlightingsReady(PathRef File, + std::vector Highlightings) override; hokein wrote: > nit:

[PATCH] D63082: [Diagnostics] Added support for -Wint-in-bool-context

xbolva00 updated this revision to Diff 206842. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63082/new/ https://reviews.llvm.org/D63082 Files: include/clang/Basic/DiagnosticGroups.td include/clang/Basic/DiagnosticSemaKinds.td lib/Sema/SemaChecking.cpp test/Analysis/misc-ps.c te

[PATCH] D63872: [clangd] Fix a case where we fail to detect a header-declared symbol in rename.

hokein updated this revision to Diff 206844. hokein marked 2 inline comments as done. hokein added a comment. Address comment. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63872/new/ https://reviews.llvm.org/D63872 Files: clang-tools-extra/clan

[PATCH] D63872: [clangd] Fix a case where we fail to detect a header-declared symbol in rename.

hokein added inline comments. Comment at: clang-tools-extra/clangd/refactor/Rename.cpp:157 assert(RenameDecl && "symbol must be found at this point"); + RenameDecl = cast(RenameDecl->getCanonicalDecl()); if (auto Reject = renamableWithinFile(*RenameDecl, File, Index)) { --

[PATCH] D63872: [clangd] Fix a case where we fail to detect a header-declared symbol in rename.

sammccall accepted this revision. sammccall added inline comments. This revision is now accepted and ready to land. Comment at: clang-tools-extra/clangd/refactor/Rename.cpp:157 assert(RenameDecl && "symbol must be found at this point"); + RenameDecl = cast(RenameDecl->getCano

[PATCH] D63821: [clangd] Added C++ API code for semantic highlighting

hokein added inline comments. Comment at: clang-tools-extra/clangd/ClangdLSPServer.h:60 + void + onHighlightingsReady(PathRef File, + std::vector Highlightings) override; jvikstrom wrote: > hokein wrote: > > nit: you can remove this overri

[clang-tools-extra] r364537 - [clangd] Fix a case where we fail to detect a header-declared symbol in rename.

Author: hokein Date: Thu Jun 27 06:24:10 2019 New Revision: 364537 URL: http://llvm.org/viewvc/llvm-project?rev=364537&view=rev Log: [clangd] Fix a case where we fail to detect a header-declared symbol in rename. Summary: Failing case: ``` #include "foo.h" void fo^o() {} ``` getRenameDecl() r

[PATCH] D63872: [clangd] Fix a case where we fail to detect a header-declared symbol in rename.

This revision was automatically updated to reflect the committed changes. Closed by commit rL364537: [clangd] Fix a case where we fail to detect a header-declared symbol in rename. (authored by hokein, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed

[PATCH] D63876: [OpenCL] Define CLK_NULL_EVENT without cast

svenvh created this revision. svenvh added reviewers: Anastasia, yaxunl. Herald added a project: clang. Herald added a subscriber: cfe-commits. Defining CLK_NULL_EVENT with a `(void*)` cast has the (unintended?) side-effect that the address space will be fixed (as generic in OpenCL 2.0 mode). The

[PATCH] D63821: [clangd] Added C++ API code for semantic highlighting

jvikstrom updated this revision to Diff 206854. jvikstrom marked 2 inline comments as done. jvikstrom added a comment. Made test safe again Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63821/new/ https://reviews.llvm.org/D63821 Files: clang-too

[PATCH] D63878: [CTU] Add missing statistics

martong created this revision. martong added a reviewer: xazax.hun. Herald added subscribers: cfe-commits, gamesh411, Szelethus, dkrupp, rnkovacs. Herald added a project: clang. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D63878 Files: clang/lib/CrossTU/CrossTranslationUnit.

[PATCH] D62977: [clang-tidy]: Google: new check 'google-upgrade-googletest-case'

astrelni updated this revision to Diff 206855. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62977/new/ https://reviews.llvm.org/D62977 Files: clang-tools-extra/clang-tidy/google/CMakeLists.txt clang-tools-extra/clang-tidy/google/GoogleTidyModule.cpp clang-tools-extra/clang-tidy/go

[PATCH] D62977: [clang-tidy]: Google: new check 'google-upgrade-googletest-case'

astrelni added a comment. In D62977#1559649 , @lebedev.ri wrote: > In D62977#1559637 , @astrelni wrote: > > > In D62977#1559628 , @lebedev.ri > > wrote: > > > > > It //sound

[PATCH] D63773: [clangd] dummy variable extraction on a function scope

sammccall added a comment. Please forgive the mix of high-level and low-level comments here. Happy to discuss further of course! Comment at: clang-tools-extra/clangd/refactor/tweaks/ExtractVariable.cpp:55 +// checks whether any variable in a given expr is declared in the DeclSt

[PATCH] D63638: [clang][NewPM] Add new pass manager RUN lines to avx512f-builtins.c

spatel added a comment. I skimmed D63174 but haven't applied either of these patches to test locally, so I may not have the full picture. IMO, we do not want clang regression tests running -instcombine/-instsimplify. That can cause clang tests to break when an

[PATCH] D60455: [SYCL] Implement SYCL device code outlining

Fznamznon updated this revision to Diff 206861. Fznamznon added a comment. Added warning diagnostic for `sycl_kernel` attribute. Now if the `sycl_kernel` attribute applied to a function which doesn't meet requirements for OpenCL kernel generation, attribute will be ignored and diagnostic will b

[PATCH] D62977: [clang-tidy]: Google: new check 'google-upgrade-googletest-case'

lebedev.ri resigned from this revision. lebedev.ri added a comment. Looks reasonable. I did not review the check itself though. Are `test/clang-tidy/google-upgrade-googletest-case-nosuite.cpp` and `test/clang-tidy/google-upgrade-googletest-case.cpp ` identical other than the included header and

[PATCH] D63821: [clangd] Added C++ API code for semantic highlighting

hokein accepted this revision. hokein added a comment. This revision is now accepted and ready to land. Let's rephrase the commit message, I think this patch is just to "emit the semantic highlighting tokens when the main AST is built". Comment at: clang-tools-extra/clangd/uni

[PATCH] D63638: [clang][NewPM] Add new pass manager RUN lines to avx512f-builtins.c

lebedev.ri added a comment. In D63638#1560846 , @spatel wrote: > I skimmed D63174 but haven't applied either > of these patches to test locally, so I may not have the full picture. > > IMO, we do not want clang regression

[clang-tools-extra] r364551 - [clangd] Emit semantic highlighting tokens when the main AST is built.

Author: jvikstrom Date: Thu Jun 27 08:13:03 2019 New Revision: 364551 URL: http://llvm.org/viewvc/llvm-project?rev=364551&view=rev Log: [clangd] Emit semantic highlighting tokens when the main AST is built. Differential Revision: https://reviews.llvm.org/D63821 Modified: clang-tools-extra/tr

[PATCH] D63821: [clangd] Added C++ API code for semantic highlighting

This revision was automatically updated to reflect the committed changes. Closed by commit rL364551: [clangd] Emit semantic highlighting tokens when the main AST is built. (authored by jvikstrom, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior

[PATCH] D62648: [Sema][Typo] Fix assertion failure for expressions with multiple typos

dgoldman updated this revision to Diff 206870. dgoldman marked 9 inline comments as done. dgoldman added a comment. - Minor fixes Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62648/new/ https://reviews.llvm.org/D62648 Files: lib/Sema/SemaExprCXX.cpp test/Se

[PATCH] D62648: [Sema][Typo] Fix assertion failure for expressions with multiple typos

dgoldman added inline comments. Comment at: lib/Sema/SemaExprCXX.cpp:7762-7764 + llvm::SmallDenseMap NewTransformCache; + auto SavedTransformCache = TransformCache; + TransformCache = NewTransformCache; Should I do the same `std::move` and `clear`

[PATCH] D60455: [SYCL] Implement SYCL device code outlining

Fznamznon updated this revision to Diff 206873. Fznamznon added a comment. Minor fix Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60455/new/ https://reviews.llvm.org/D60455 Files: clang/include/clang/Basic/Attr.td clang/include/clang/Basic/At

[PATCH] D63638: [clang][NewPM] Add new pass manager RUN lines to avx512f-builtins.c

craig.topper added a comment. In D63638#1560846 , @spatel wrote: > I skimmed D63174 but haven't applied either > of these patches to test locally, so I may not have the full picture. > > IMO, we do not want clang regressi

[PATCH] D63878: [CTU] Add missing statistics

xazax.hun accepted this revision. xazax.hun added a comment. This revision is now accepted and ready to land. LGTM! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63878/new/ https://reviews.llvm.org/D63878

[PATCH] D63518: BitStream reader: propagate errors

jfb marked 2 inline comments as done. jfb added a subscriber: hans. jfb added inline comments. Comment at: llvm/trunk/lib/Bitcode/Reader/BitcodeReader.cpp:205 + return MaybeBitCode.takeError(); +switch (unsigned BitCode = MaybeBitCode.get()) { default: // Default be

[PATCH] D59474: [OpenMP 5.0] Codegen support for user-defined mappers

lildmh marked an inline comment as done. lildmh added inline comments. Comment at: lib/CodeGen/CGOpenMPRuntime.cpp:8739 +/// // For each component specified by this mapper: +/// if (currentComponent.hasMapper()) +/// (*currentComponent.Mapper())(rt_mapper_handle, ar

[PATCH] D63756: [AMDGPU] Increased the number of implicit argument bytes for both OpenCL and HIP (CLANG).

cdevadas added a comment. I have created the review for adding the metadata in the backend. https://reviews.llvm.org/D63886 Marking the current review depended on D63886 . Repository: rC Clang CHANGES SINCE LAST ACTION http

[PATCH] D59474: [OpenMP 5.0] Codegen support for user-defined mappers

ABataev added inline comments. Comment at: lib/CodeGen/CGOpenMPRuntime.cpp:8739 +/// // For each component specified by this mapper: +/// if (currentComponent.hasMapper()) +/// (*currentComponent.Mapper())(rt_mapper_handle, arg_base, arg_begin, lild

[PATCH] D59474: [OpenMP 5.0] Codegen support for user-defined mappers

lildmh marked an inline comment as done. lildmh added inline comments. Comment at: lib/CodeGen/CGOpenMPRuntime.cpp:8739 +/// // For each component specified by this mapper: +/// if (currentComponent.hasMapper()) +/// (*currentComponent.Mapper())(rt_mapper_handle, ar

[PATCH] D59474: [OpenMP 5.0] Codegen support for user-defined mappers

ABataev added inline comments. Comment at: lib/CodeGen/CGOpenMPRuntime.cpp:8739 +/// // For each component specified by this mapper: +/// if (currentComponent.hasMapper()) +/// (*currentComponent.Mapper())(rt_mapper_handle, arg_base, arg_begin, lild

[PATCH] D61809: [BPF] Preserve debuginfo array/union/struct type/access index

yonghong-song added a comment. @eli.friedman Hi, Just ping. I have removed getParents() for ASTContext and added description of the new intrinsic in language doc. Could you take a look? Thanks! Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61809/new/ https://re

[PATCH] D59474: [OpenMP 5.0] Codegen support for user-defined mappers

lildmh marked an inline comment as done. lildmh added inline comments. Comment at: lib/CodeGen/CGOpenMPRuntime.cpp:8739 +/// // For each component specified by this mapper: +/// if (currentComponent.hasMapper()) +/// (*currentComponent.Mapper())(rt_mapper_handle, ar

[PATCH] D59474: [OpenMP 5.0] Codegen support for user-defined mappers

ABataev added inline comments. Comment at: lib/CodeGen/CGOpenMPRuntime.cpp:8739 +/// // For each component specified by this mapper: +/// if (currentComponent.hasMapper()) +/// (*currentComponent.Mapper())(rt_mapper_handle, arg_base, arg_begin, lild

[PATCH] D59474: [OpenMP 5.0] Codegen support for user-defined mappers

lildmh marked an inline comment as done. lildmh added inline comments. Comment at: lib/CodeGen/CGOpenMPRuntime.cpp:8739 +/// // For each component specified by this mapper: +/// if (currentComponent.hasMapper()) +/// (*currentComponent.Mapper())(rt_mapper_handle, ar

[PATCH] D63773: [clangd] dummy variable extraction on a function scope

SureYeaah marked 14 inline comments as done. SureYeaah added inline comments. Comment at: clang-tools-extra/clangd/refactor/tweaks/ExtractVariable.cpp:173 + // give up if extraction will take a variable out of scope + if (!extractionAllowed(ParStmt, N, M)) +brea

[PATCH] D59474: [OpenMP 5.0] Codegen support for user-defined mappers

ABataev added inline comments. Comment at: lib/CodeGen/CGOpenMPRuntime.cpp:8739 +/// // For each component specified by this mapper: +/// if (currentComponent.hasMapper()) +/// (*currentComponent.Mapper())(rt_mapper_handle, arg_base, arg_begin, lild

[PATCH] D63856: [ObjC] Add a -Wtautological-compare warning for BOOL

rjmccall added a comment. In D63856#1560213 , @erik.pilkington wrote: > In D63856#1560180 , @rjmccall wrote: > > > This only applies to relational operators, right? I'm a little > > uncomfortable with calling thi

[PATCH] D59474: [OpenMP 5.0] Codegen support for user-defined mappers

lildmh marked an inline comment as done. lildmh added inline comments. Comment at: lib/CodeGen/CGOpenMPRuntime.cpp:8739 +/// // For each component specified by this mapper: +/// if (currentComponent.hasMapper()) +/// (*currentComponent.Mapper())(rt_mapper_handle, ar

[PATCH] D62977: [clang-tidy]: Google: new check 'google-upgrade-googletest-case'

astrelni updated this revision to Diff 206889. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62977/new/ https://reviews.llvm.org/D62977 Files: clang-tools-extra/clang-tidy/google/CMakeLists.txt clang-tools-extra/clang-tidy/google/GoogleTidyModule.cpp clang-tools-extra/clang-tidy/go

[PATCH] D62960: SVE opaque type for C intrinsics demo

rsandifo-arm updated this revision to Diff 206891. rsandifo-arm added a comment. - Fix comments in AArch64SVEACLETypes.def - Rename BUILTIN_TYPE to SVE_TYPE and use it where possible - Report errors for TODOs instead of using llvm_unreachable - Add a test for the errors - Formatting fixes Reposi

[PATCH] D62977: [clang-tidy]: Google: new check 'google-upgrade-googletest-case'

astrelni marked 4 inline comments as done. astrelni added a comment. In D62977#1560879 , @lebedev.ri wrote: > Looks reasonable. I did not review the check itself though. > Are `test/clang-tidy/google-upgrade-googletest-case-nosuite.cpp` and > `test/clang

[PATCH] D63856: [ObjC] Add a -Wtautological-compare warning for BOOL

erik.pilkington added a comment. In D63856#1561112 , @rjmccall wrote: > In D63856#1560213 , @erik.pilkington > wrote: > > > In D63856#1560180 , @rjmccall > > wrote: > > > >

[PATCH] D62960: SVE opaque type for C intrinsics demo

rsandifo-arm marked 7 inline comments as done. rsandifo-arm added a comment. Thanks for the reviews! Comment at: include/clang/Basic/AArch64SVEACLETypes.def:10 +// +// This file defines the database about various builtin singleton types. +// rovka wrote: > You

[PATCH] D62961: [AST] Add new Type queries for sizeless types

rsandifo-arm updated this revision to Diff 206894. rsandifo-arm added a comment. - Update for new version of D62960 Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62961/new/ https://reviews.llvm.org/D62961 Files: include/clang/

[PATCH] D63856: [ObjC] Add a -Wtautological-compare warning for BOOL

rjmccall added a comment. In D63856#1561132 , @erik.pilkington wrote: > In D63856#1561112 , @rjmccall wrote: > > > In D63856#1560213 , > > @erik.pilkington wrote: > > > > >

[PATCH] D59474: [OpenMP 5.0] Codegen support for user-defined mappers

lildmh updated this revision to Diff 206895. lildmh added a comment. Change the type of size from `size_t` to `int64_t`, and rebase CHANGES SINCE LAST ACTION https://reviews.llvm.org/D59474/new/ https://reviews.llvm.org/D59474 Files: include/clang/AST/GlobalDecl.h lib/AST/ASTContext.cpp

[PATCH] D62829: [clang-tidy] Check for dynamically initialized statics in headers.

czhang added a comment. ping CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62829/new/ https://reviews.llvm.org/D62829 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r364575 - [OPENMP]Generate correctly implicit flags for mapped data.

Author: abataev Date: Thu Jun 27 11:53:07 2019 New Revision: 364575 URL: http://llvm.org/viewvc/llvm-project?rev=364575&view=rev Log: [OPENMP]Generate correctly implicit flags for mapped data. Implicit flag must not be emitted for explicitly specified firstprivate variables, but for implicitly ca

[PATCH] D63538: [analyzer][CFG] Return the correct terminator condition

xazax.hun added a comment. This change will be really useful for the lifetime analysis as well! Thanks! I have one concern though. The analyzer is using linerarized (or threaded) CFGs with every subexpression being a separate entry in the basic blocks. Will your change give sensible answers for

[PATCH] D63889: Check possible warnings on global initializers for reachability

Nathan-Huckleberry created this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Create CFG for initializers and perform analysis based warnings on global variables Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D63889 Files: clang/include/cla

[PATCH] D62293: [modules] Add PP callbacks for entering and leaving a submodule.

v.g.vassilev updated this revision to Diff 206887. v.g.vassilev marked an inline comment as done. v.g.vassilev added a comment. Add comments, add missing callbacks. Thanks Richard! Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62293/new/ https://reviews.llvm.org/

[PATCH] D63892: [LibTooling] Extend `RewriteRule` with support for adding includes.

ymandel created this revision. ymandel added reviewers: ilya-biryukov, gribozavr. Herald added a project: clang. This revision allows users to specify the insertion of an included directive (at the top of the file being rewritten) as part of a rewrite rule. These directives are bundled with `Rewr

[PATCH] D63893: [clang-tidy] Extend TransformerClangTidyCheck to support adding includes.

ymandel created this revision. ymandel added reviewers: ilya-biryukov, gribozavr. Herald added a subscriber: xazax.hun. Herald added a project: clang. ymandel added a parent revision: D63892: [LibTooling] Extend `RewriteRule` with support for adding includes.. This revision implements support for

[PATCH] D63533: [analyzer] Fix clang-tidy crash on GCCAsmStmt

Nathan-Huckleberry updated this revision to Diff 206919. Nathan-Huckleberry added a comment. - Add assertion message and simplify test case Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63533/new/ https://reviews.llvm.org/D63533 Files: clang/lib

[PATCH] D63857: [clang-doc] Structured HTML generator

DiegoAstiazaran updated this revision to Diff 206920. DiegoAstiazaran marked 7 inline comments as done. DiegoAstiazaran added a comment. Fix TagType enum name and members. Add anonymous namespace. Separate the implementation from the definition for some functions. Use emplace_back instead of push_

[PATCH] D63894: [CXX] Exercise all paths through these tests

probinson created this revision. probinson added a reviewer: rsmith. Herald added a project: clang. Herald added a subscriber: cfe-commits. These 3 tests have dialect-based conditionals, but weren't running Clang with enough different dialects to actually enable those conditional sections. Repo

[PATCH] D63533: [analyzer] Fix clang-tidy crash on GCCAsmStmt

NoQ accepted this revision. NoQ added a comment. This revision is now accepted and ready to land. Looks great now, thanks! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63533/new/ https://reviews.llvm.org/D63533 _

r364595 - Pattern match struct types in test case.

Author: ahatanak Date: Thu Jun 27 14:16:19 2019 New Revision: 364595 URL: http://llvm.org/viewvc/llvm-project?rev=364595&view=rev Log: Pattern match struct types in test case. This simplifies the test cases in a patch I'm planning to send later. Modified: cfe/trunk/test/CodeGenObjC/strong-in

[PATCH] D63533: [analyzer] Fix clang-tidy crash on GCCAsmStmt

nickdesaulniers added inline comments. Comment at: clang/lib/StaticAnalyzer/Core/CoreEngine.cpp:401 + case Stmt::GCCAsmStmtClass: +assert(cast(Term)->isAsmGoto() == true && "Encountered GCCAsmStmt without labels"); +// TODO: Handle jumping to labels

[PATCH] D63857: [clang-doc] Structured HTML generator

jakehehrlich added a comment. Overall looks better. One of my comments causes a sweeping change to occur so I'll respond back after thats done. Comment at: clang-tools-extra/clang-doc/HTMLGenerator.cpp:91 +struct HTMLFile { + llvm::SmallString<16> DoctypeDecl{""}; + std::vec

[PATCH] D63623: [clang-tidy] new check: bugprone-posix-return

jcai19 updated this revision to Diff 206946. jcai19 added a comment. Update the check to catch redundant code and update the warning message to be more speicific. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63623/new/ https://reviews.llvm.org/D6

[PATCH] D63623: [clang-tidy] new check: bugprone-posix-return

jcai19 marked 2 inline comments as done. jcai19 added inline comments. Comment at: clang-tools-extra/test/clang-tidy/bugprone-posix-return.cpp:96 + if (posix_fadvise(0, 0, 0, 0) < 0) {} + if (posix_fadvise(0, 0, 0, 0) >= 0) {} else {} + if (posix_fadvise(0, 0, 0, 0) == -1) {}

[PATCH] D63623: [clang-tidy] new check: bugprone-posix-return

jcai19 updated this revision to Diff 206947. jcai19 added a comment. Update test names. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63623/new/ https://reviews.llvm.org/D63623 Files: clang-tools-extra/clang-tidy/bugprone/BugproneTidyModule.cpp

[PATCH] D62441: [analyzer] NFC: Introduce a convenient CallDescriptionMap class.

NoQ updated this revision to Diff 206951. NoQ added a comment. Indeed. Subtle! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62441/new/ https://reviews.llvm.org/D62441 Files: clang/include/clang/StaticAnalyzer/Core/PathSensitive/CallEvent.h clang/unittests/StaticAnalyzer/CMakeLists

[PATCH] D62648: [Sema][Typo] Fix assertion failure for expressions with multiple typos

rsmith accepted this revision. rsmith added inline comments. This revision is now accepted and ready to land. Comment at: lib/Sema/SemaExprCXX.cpp:7762-7764 + llvm::SmallDenseMap NewTransformCache; + auto SavedTransformCache = TransformCache; + TransformCache = New

[libclc] r364604 - Creating release candidate rc3 from release_801 branch

Author: tstellar Date: Thu Jun 27 15:33:04 2019 New Revision: 364604 URL: http://llvm.org/viewvc/llvm-project?rev=364604&view=rev Log: Creating release candidate rc3 from release_801 branch Added: libclc/tags/RELEASE_801/rc3/ - copied from r364603, libclc/branches/release_80/ _

[libunwind] r364604 - Creating release candidate rc3 from release_801 branch

Author: tstellar Date: Thu Jun 27 15:33:04 2019 New Revision: 364604 URL: http://llvm.org/viewvc/llvm-project?rev=364604&view=rev Log: Creating release candidate rc3 from release_801 branch Added: libunwind/tags/RELEASE_801/rc3/ - copied from r364603, libunwind/branches/release_80/ ___

[PATCH] D63533: [analyzer] Fix clang-tidy crash on GCCAsmStmt

Nathan-Huckleberry updated this revision to Diff 206954. Nathan-Huckleberry added a comment. - Minor style change on assert Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63533/new/ https://reviews.llvm.org/D63533 Files: clang/lib/StaticAnalyzer/

r364605 - [analyzer] Fix clang-tidy crash on GCCAsmStmt

Author: nathan-huckleberry Date: Thu Jun 27 15:46:40 2019 New Revision: 364605 URL: http://llvm.org/viewvc/llvm-project?rev=364605&view=rev Log: [analyzer] Fix clang-tidy crash on GCCAsmStmt Summary: Added entry in switch statement to recognize GCCAsmStmt as a possible block terminator. Handling

[PATCH] D63533: [analyzer] Fix clang-tidy crash on GCCAsmStmt

This revision was automatically updated to reflect the committed changes. Closed by commit rL364605: [analyzer] Fix clang-tidy crash on GCCAsmStmt (authored by Nathan-Huckleberry, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit: htt

[PATCH] D63048: Update __VERSION__ to remove the hardcoded 4.2.1 version

rnk added a comment. I'd be in favor of removing them. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63048/new/ https://reviews.llvm.org/D63048 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.ll

  1   2   >