[PATCH] D63640: [clang] Improve Serialization/Imporing of APValues

Tyker updated this revision to Diff 207977. Tyker marked 2 inline comments as done. Tyker set the repository for this revision to rC Clang. Tyker added a comment. fixed comment typo. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63640/new/ https://reviews.llvm.or

[PATCH] D63640: [clang] Improve Serialization/Imporing of APValues

Tyker added inline comments. Comment at: clang/lib/AST/ASTImporter.cpp:8503 +llvm::Expected ASTImporter::Import(const APValue &FromValue) { + APValue Result; martong wrote: > Looks okay, but could we have unit tests for this in ASTImporterTest.cpp? I tested im

[PATCH] D63919: [clangd] Emit publishSemanticHighlighting in LSP if enabled

hokein accepted this revision. hokein added a comment. This revision is now accepted and ready to land. thanks, looks good. Comment at: clang-tools-extra/clangd/ClangdLSPServer.h:59 + void onHighlightingsReady(PathRef File, + std::vector Highlightings)

[clang-tools-extra] r365111 - [clangd] Emit publishSemanticHighlighting in LSP if enabled

Author: jvikstrom Date: Thu Jul 4 00:53:12 2019 New Revision: 365111 URL: http://llvm.org/viewvc/llvm-project?rev=365111&view=rev Log: [clangd] Emit publishSemanticHighlighting in LSP if enabled Summary: Emit publishSemanticHighlighting in LSP if enabled Reviewers: hokein, kadircet Subscribers

[PATCH] D63919: [clangd] Emit publishSemanticHighlighting in LSP if enabled

This revision was automatically updated to reflect the committed changes. jvikstrom marked 2 inline comments as done. Closed by commit rL365111: [clangd] Emit publishSemanticHighlighting in LSP if enabled (authored by jvikstrom, committed by ). Herald added a project: LLVM. Herald added a subscrib

[PATCH] D64137: [clangd] Add a hidden tweak to annotate all highlighting tokens of the file.

sammccall added inline comments. Comment at: clang-tools-extra/clangd/refactor/tweaks/AnnotateHighlightings.cpp:15 + +llvm::StringRef highlightingKindToString(HighlightingKind Kind) { + switch (Kind) { it would be nice to use the textmate class name rather than

[clang-tools-extra] r365112 - [clangd] Add missing changes for 365111

Author: jvikstrom Date: Thu Jul 4 01:06:48 2019 New Revision: 365112 URL: http://llvm.org/viewvc/llvm-project?rev=365112&view=rev Log: [clangd] Add missing changes for 365111 Modified: clang-tools-extra/trunk/clangd/ClangdLSPServer.h clang-tools-extra/trunk/clangd/test/semantic-highlight

[PATCH] D64147: [clangd] Make HadErrors part of background index's internal state

sammccall accepted this revision. sammccall added a comment. This revision is now accepted and ready to land. Logic looks great, just naming/comment nits Comment at: clang-tools-extra/clangd/index/Background.cpp:276 +const auto DigestIt = IndexedFilesSnapshot.find(AbsPath);

[PATCH] D64196: [clangd] Query driver reads stderr and passes driver as first argument

kadircet created this revision. kadircet added a reviewer: sammccall. Herald added subscribers: cfe-commits, arphaman, jkorous, MaskRay, ilya-biryukov. Herald added a project: clang. gcc invokes cc1 through a path deduced from argv[0] therefore it must be correctly set. Also it prints the search

[PATCH] D64147: [clangd] Make HadErrors part of background index's internal state

kadircet updated this revision to Diff 207986. kadircet marked 6 inline comments as done. kadircet added a comment. - Address comments Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64147/new/ https://reviews.llvm.org/D64147 Files: clang-tools-ex

[PATCH] D63936: [clang][Driver][ARM] Favor -mfpu over default CPU features

ostannard added inline comments. Comment at: llvm/lib/Support/ARMTargetParser.cpp:412 - if (Extensions & AEK_CRC) -Features.push_back("+crc"); - else -Features.push_back("-crc"); - - if (Extensions & AEK_DSP) -Features.push_back("+dsp"); - else -Features.pus

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

rsandifo-arm updated this revision to Diff 207991. rsandifo-arm added a comment. Changes since last version: - Define the SVE types only for AArch64 targets - Use the vendor extension 'u' mangling, as per the spec - Report an error for @encode on the types - Fixed the licence text on the new file

[PATCH] D64019: [clangd] Turn background-index on by default

kadircet marked an inline comment as done. kadircet added inline comments. Comment at: clang-tools-extra/clangd/tool/ClangdMain.cpp:191 "Index project code in the background and persist index on disk. " "Experimental"), +llvm::cl::init(true)); --

[PATCH] D62960: Add SVE opaque built-in types

rsandifo-arm added a comment. In D62960#1566265 , @rovka wrote: > This looks much better, thanks! Shouldn't there be more tests, e.g. for > mangling and maybe the ASTImporter? Thanks. I've added tests for ASTImporter and mangling in the latest version,

[PATCH] D64074: [OpenCL][Sema] Fix builtin rewriting

mantognini updated this revision to Diff 207994. mantognini added a comment. Addressed missing colon in FIXME comment. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64074/new/ https://reviews.llvm.org/D64074 Files: clang/include/clang/Basic/Buil

[PATCH] D64074: [OpenCL][Sema] Fix builtin rewriting

mantognini added a comment. I'll commit this along with D64083 in a few days in case John has any feedback. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64074/new/ https://reviews.llvm.org/D64074 _

[PATCH] D63640: [clang] Improve Serialization/Imporing of APValues

martong added inline comments. Comment at: clang/include/clang/AST/ASTImporter.h:331 +/// +/// \return the equivalent APValue in the "from" Constext or the import +/// error. martong wrote: > typo: `Constext` -> `Context` > Also, we return with the AP

[PATCH] D64137: [clangd] Add a hidden tweak to annotate all highlighting tokens of the file.

hokein updated this revision to Diff 207996. hokein marked 3 inline comments as done. hokein added a comment. address comments: - emit textmate scope names - rescope to the selected node rather than TU decl. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm

[PATCH] D62960: Add SVE opaque built-in types

martong added a comment. The ASTImporter and the test for it looks good to me, thanks! Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62960/new/ https://reviews.llvm.org/D62960 ___ cfe-commits mailing list cfe-commits

[clang-tools-extra] r365120 - [clangd] Also cache failures while indexing

Author: kadircet Date: Thu Jul 4 02:51:43 2019 New Revision: 365120 URL: http://llvm.org/viewvc/llvm-project?rev=365120&view=rev Log: [clangd] Also cache failures while indexing Summary: Clangd currently doesn't cache any indexing failures, which results in retrying those failed files even if th

[clang-tools-extra] r365123 - [clangd] Make HadErrors part of background index's internal state

Author: kadircet Date: Thu Jul 4 02:52:12 2019 New Revision: 365123 URL: http://llvm.org/viewvc/llvm-project?rev=365123&view=rev Log: [clangd] Make HadErrors part of background index's internal state Reviewers: sammccall Subscribers: ilya-biryukov, MaskRay, jkorous, arphaman, cfe-commits Tags:

[clang-tools-extra] r365121 - [clangd] Store hash of command line in index shards.

Author: kadircet Date: Thu Jul 4 02:51:53 2019 New Revision: 365121 URL: http://llvm.org/viewvc/llvm-project?rev=365121&view=rev Log: [clangd] Store hash of command line in index shards. Summary: This is to enable cache invalidation when command line flags changes. Reviewers: sammccall Subscri

[clang-tools-extra] r365122 - [clangd] Add HadErrors field into shards

Author: kadircet Date: Thu Jul 4 02:52:04 2019 New Revision: 365122 URL: http://llvm.org/viewvc/llvm-project?rev=365122&view=rev Log: [clangd] Add HadErrors field into shards Reviewers: sammccall Subscribers: ilya-biryukov, MaskRay, jkorous, arphaman, cfe-commits Tags: #clang Differential Rev

[PATCH] D63986: [clangd] Also cache failures while indexing

This revision was automatically updated to reflect the committed changes. Closed by commit rL365120: [clangd] Also cache failures while indexing (authored by kadircet, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews

[PATCH] D63920: [CTU] Add support for virtual functions

martong updated this revision to Diff 208002. martong added a comment. - Remove ipa mode from the test Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63920/new/ https://reviews.llvm.org/D63920 Files: clang/lib/StaticAnalyzer/Core/CallEvent.cpp

[PATCH] D64133: [clangd] Add HadErrors field into shards

This revision was automatically updated to reflect the committed changes. Closed by commit rL365122: [clangd] Add HadErrors field into shards (authored by kadircet, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.ll

[PATCH] D64018: [clangd] Store hash of command line in index shards.

This revision was automatically updated to reflect the committed changes. Closed by commit rL365121: [clangd] Store hash of command line in index shards. (authored by kadircet, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit: https:

[PATCH] D64147: [clangd] Make HadErrors part of background index's internal state

This revision was automatically updated to reflect the committed changes. Closed by commit rL365123: [clangd] Make HadErrors part of background index's internal state (authored by kadircet, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to com

[PATCH] D64199: [clangd] Added highlighting for variable references (declrefs)

jvikstrom created this revision. jvikstrom added reviewers: hokein, sammccall. Herald added subscribers: cfe-commits, kadircet, arphaman, jkorous, MaskRay, ilya-biryukov. Herald added a project: clang. Added highlighting for variable references using VisitDeclRefExpr. Repository: rG LLVM Gith

[clang-tools-extra] r365124 - [clangd] Turn background-index on by default

Author: kadircet Date: Thu Jul 4 02:56:24 2019 New Revision: 365124 URL: http://llvm.org/viewvc/llvm-project?rev=365124&view=rev Log: [clangd] Turn background-index on by default Reviewers: sammccall Subscribers: ilya-biryukov, MaskRay, jkorous, arphaman, cfe-commits Tags: #clang Differential

[PATCH] D63920: [CTU] Add support for virtual functions

Szelethus accepted this revision. Szelethus 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/D63920/new/ https://reviews.llvm.org/D63920 _

[PATCH] D63920: [CTU] Add support for virtual functions

martong added a comment. Thanks for the review! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63920/new/ https://reviews.llvm.org/D63920 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://

[PATCH] D64137: [clangd] Add a hidden tweak to annotate all highlighting tokens of the file.

sammccall accepted this revision. sammccall added inline comments. This revision is now accepted and ready to land. Comment at: clang-tools-extra/clangd/refactor/tweaks/AnnotateHighlightings.cpp:16 + +llvm::StringRef toTextMateScope(HighlightingKind Kind) { + static const auto&

[PATCH] D64019: [clangd] Turn background-index on by default

This revision was automatically updated to reflect the committed changes. Closed by commit rL365124: [clangd] Turn background-index on by default (authored by kadircet, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit: https://review

[PATCH] D63936: [clang][Driver][ARM] Favor -mfpu over default CPU features

labrinea added inline comments. Comment at: llvm/lib/Support/ARMTargetParser.cpp:412 - if (Extensions & AEK_CRC) -Features.push_back("+crc"); - else -Features.push_back("-crc"); - - if (Extensions & AEK_DSP) -Features.push_back("+dsp"); - else -Features.push

r365126 - Fix -Wdocumentation warning.

Author: rksimon Date: Thu Jul 4 03:28:31 2019 New Revision: 365126 URL: http://llvm.org/viewvc/llvm-project?rev=365126&view=rev Log: Fix -Wdocumentation warning. Modified: cfe/trunk/include/clang/AST/ASTImporter.h Modified: cfe/trunk/include/clang/AST/ASTImporter.h URL: http://llvm.org/vie

[PATCH] D64048: [TargetParser][ARM] Account dependencies when processing target features

labrinea updated this revision to Diff 208010. labrinea added a comment. Added the dependency of mve on dsp and some missing tests to cover those cases. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64048/new/ https://reviews.llvm.org/D64048 Files

[PATCH] D64137: [clangd] Add a hidden tweak to annotate all highlighting tokens of the file.

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

[clang-tools-extra] r365130 - [clangd] Add a hidden tweak to annotate all highlighting tokens of the file.

Author: hokein Date: Thu Jul 4 03:49:32 2019 New Revision: 365130 URL: http://llvm.org/viewvc/llvm-project?rev=365130&view=rev Log: [clangd] Add a hidden tweak to annotate all highlighting tokens of the file. Reviewers: sammccall, jvikstrom Subscribers: mgorny, ilya-biryukov, MaskRay, jkorous,

[PATCH] D64137: [clangd] Add a hidden tweak to annotate all highlighting tokens of the file.

This revision was automatically updated to reflect the committed changes. Closed by commit rL365130: [clangd] Add a hidden tweak to annotate all highlighting tokens of the file. (authored by hokein, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed pri

[PATCH] D63640: [clang] Improve Serialization/Imporing of APValues

Tyker updated this revision to Diff 208015. Tyker added a comment. fixed the comment for good, sorry for that. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63640/new/ https://reviews.llvm.org/D63640 Files: clang/include/clang/AST/APValue.h clang/include/clang/AST/ASTContext.h cl

[clang-tools-extra] r365132 - [clangd] Query driver reads stderr and passes driver as first argument

Author: kadircet Date: Thu Jul 4 04:34:23 2019 New Revision: 365132 URL: http://llvm.org/viewvc/llvm-project?rev=365132&view=rev Log: [clangd] Query driver reads stderr and passes driver as first argument Summary: gcc invokes cc1 through a path deduced from argv[0] therefore it must be correctly

[PATCH] D64202: [clangd] Some tweaks on semantic highlighting lookuptable.

hokein created this revision. hokein added reviewers: sammccall, jvikstrom. Herald added subscribers: kadircet, arphaman, jkorous, MaskRay, ilya-biryukov. Herald added a project: clang. - move toTextMateScope to SemanticHighlighting.h; - move the buildLookupTable to LSP layer (as LSP requires such

[PATCH] D63640: [clang] Improve Serialization/Imporing of APValues

Tyker marked 2 inline comments as done. Tyker added inline comments. Comment at: clang/lib/AST/ASTImporter.cpp:8503 +llvm::Expected ASTImporter::Import(const APValue &FromValue) { + APValue Result; martong wrote: > Tyker wrote: > > martong wrote: > > > Looks o

[PATCH] D64196: [clangd] Query driver reads stderr and passes driver as first argument

This revision was automatically updated to reflect the committed changes. Closed by commit rL365132: [clangd] Query driver reads stderr and passes driver as first argument (authored by kadircet, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior t

r365133 - [CTU] Add support for virtual functions

Author: martong Date: Thu Jul 4 04:39:00 2019 New Revision: 365133 URL: http://llvm.org/viewvc/llvm-project?rev=365133&view=rev Log: [CTU] Add support for virtual functions Reviewers: Szelethus, xazax.hun Subscribers: rnkovacs, dkrupp, gamesh411, cfe-commits Tags: #clang Differential Revision

[PATCH] D63920: [CTU] Add support for virtual functions

This revision was automatically updated to reflect the committed changes. Closed by commit rL365133: [CTU] Add support for virtual functions (authored by martong, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.llvm

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

sammccall added inline comments. Comment at: clang-tools-extra/clangd/refactor/tweaks/ExtractVariable.cpp:42 + // Generate Replacement for replacing selected expression with given VarName + tooling::Replacement replaceWithVar(std::string VarName) const; + // Generate Replaceme

[PATCH] D64199: [clangd] Added highlighting for variable references (declrefs)

hokein added inline comments. Comment at: clang-tools-extra/clangd/SemanticHighlighting.cpp:46 + + bool VisitDeclRefExpr(DeclRefExpr *Ref) { +if (Ref->getNameInfo().getName().getNameKind() == The `DeclRefExpr` is a very general expression, which can referenc

[PATCH] D64202: [clangd] Some tweaks on semantic highlighting lookuptable.

sammccall accepted this revision. sammccall added inline comments. This revision is now accepted and ready to land. Comment at: clang-tools-extra/clangd/ClangdLSPServer.cpp:87 +// to the LSP client. +std::vector> buildTextMateScopeLookupTable() { + std::vector> LookupTable;

[PATCH] D49074: [Analyzer] [WIP] Basic support for multiplication and division in the constraint manager

baloghadamsoftware added a comment. I tested this on several open-source projects. Lots of false-positives disappeared. There are also some new findings, mainly new false-positives but not because of this feature but of "loops executed 0 times". CHANGES SINCE LAST ACTION https://reviews.llvm

[PATCH] D64199: [clangd] Added highlighting for variable references (declrefs)

sammccall added inline comments. Comment at: clang-tools-extra/clangd/SemanticHighlighting.cpp:46 + + bool VisitDeclRefExpr(DeclRefExpr *Ref) { +if (Ref->getNameInfo().getName().getNameKind() == hokein wrote: > The `DeclRefExpr` is a very general expression,

[clang-tools-extra] r365134 - [clangd] Fix a lifetime bug in QueryDriver

Author: kadircet Date: Thu Jul 4 05:24:17 2019 New Revision: 365134 URL: http://llvm.org/viewvc/llvm-project?rev=365134&view=rev Log: [clangd] Fix a lifetime bug in QueryDriver Modified: clang-tools-extra/trunk/clangd/QueryDriverDatabase.cpp Modified: clang-tools-extra/trunk/clangd/QueryDri

[PATCH] D64202: [clangd] Some tweaks on semantic highlighting lookuptable.

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

[clang-tools-extra] r365135 - [clangd] Some tweaks on semantic highlighting lookuptable.

Author: hokein Date: Thu Jul 4 05:27:21 2019 New Revision: 365135 URL: http://llvm.org/viewvc/llvm-project?rev=365135&view=rev Log: [clangd] Some tweaks on semantic highlighting lookuptable. Summary: - move toTextMateScope to SemanticHighlighting.h; - move the buildLookupTable to LSP layer (as L

[PATCH] D64202: [clangd] Some tweaks on semantic highlighting lookuptable.

This revision was automatically updated to reflect the committed changes. Closed by commit rL365135: [clangd] Some tweaks on semantic highlighting lookuptable. (authored by hokein, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit: ht

[PATCH] D64199: [clangd] Added highlighting for variable references (declrefs)

jvikstrom updated this revision to Diff 208028. jvikstrom added a comment. Added overload for addToken and added more code to the test cases. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64199/new/ https://reviews.llvm.org/D64199 Files: clang-t

[PATCH] D64199: [clangd] Added highlighting for variable references (declrefs)

jvikstrom updated this revision to Diff 208029. jvikstrom added a comment. Removed VisitVarDecl and VisitFuncDecl in favor of VisitNamedDecl. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64199/new/ https://reviews.llvm.org/D64199 Files: clang-t

RE: [clang-tools-extra] r365123 - [clangd] Make HadErrors part of background index's internal state

Hi Kadir, Your change is causing a build failure on our internal linux build bot running gcc 5.4: FAILED: CCACHE_CPP2=yes CCACHE_HASHDIR=yes /usr/bin/ccache /usr/lib/ccache/g++ -DGTEST_HAS_RTTI=0 -D_DEBUG -D_GNU_SOURCE -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS -It

[clang-tools-extra] r365140 - [clangd] Fix breakage on gcc 5.4

Author: kadircet Date: Thu Jul 4 06:47:51 2019 New Revision: 365140 URL: http://llvm.org/viewvc/llvm-project?rev=365140&view=rev Log: [clangd] Fix breakage on gcc 5.4 Modified: clang-tools-extra/trunk/clangd/index/Background.h Modified: clang-tools-extra/trunk/clangd/index/Background.h URL:

Re: [clang-tools-extra] r365123 - [clangd] Make HadErrors part of background index's internal state

https://reviews.llvm.org/rL365140 should fix it, please let me know if it doesnt. On Thu, Jul 4, 2019 at 3:45 PM wrote: > Hi Kadir, > > Your change is causing a build failure on our internal linux build bot > running gcc 5.4: > > FAILED: CCACHE_CPP2=yes CCACHE_HASHDIR=yes /usr/bin/ccache > /usr/

[PATCH] D64199: [clangd] Added highlighting for variable references (declrefs)

jvikstrom updated this revision to Diff 208030. jvikstrom added a comment. Removed debug prints from test. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64199/new/ https://reviews.llvm.org/D64199 Files: clang-tools-extra/clangd/SemanticHighlight

[PATCH] D64199: [clangd] Added highlighting for variable references (declrefs)

sammccall added inline comments. Comment at: clang-tools-extra/clangd/SemanticHighlighting.cpp:38 + bool VisitNamedDecl(NamedDecl *ND) { +if (ND->getDeclName().isEmpty()) + // Don't add symbols that don't have any length. I think you might want to bail

[PATCH] D64128: [CodeGen] Generate llvm.ptrmask instead of inttoptr(and(ptrtoint, C)) if possible.

fhahn added a comment. Thanks for the quick responses and the helpful comments. Thank you very much Hal, for summarizing the argument from previous discussions. My initial understanding indeed was that by generating ptrmask directly for C/C++ expressions, we can circumvent the issues that come

[PATCH] D64128: [CodeGen] Generate llvm.ptrmask instead of inttoptr(and(ptrtoint, C)) if possible.

fhahn added a comment. In D64128#1569836 , @hfinkel wrote: > In D64128#1569817 , @rjmccall wrote: > > > The pointer/integer conversion is "implementation-defined", but it's not > > totally unconstrained. C notes t

[PATCH] D64199: [clangd] Added highlighting for variable references (declrefs)

jvikstrom updated this revision to Diff 208041. jvikstrom marked 2 inline comments as done. jvikstrom added a comment. Added testcae. Added another bailout from VisitNamedDecl. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64199/new/ https://review

[PATCH] D64199: [clangd] Added highlighting for variable references (declrefs)

jvikstrom added inline comments. Comment at: clang-tools-extra/clangd/SemanticHighlighting.cpp:38 + bool VisitNamedDecl(NamedDecl *ND) { +if (ND->getDeclName().isEmpty()) + // Don't add symbols that don't have any length. sammccall wrote: > I think you

[PATCH] D63767: [NFC] Make some ObjectFormatType switches covering

sfertile accepted this revision. sfertile added a comment. This revision is now accepted and ready to land. Herald added a subscriber: wuzish. LGTM. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63767/new/ https://reviews.llvm.org/D63767 ___

[PATCH] D64199: [clangd] Added highlighting for variable references (declrefs)

sammccall accepted this revision. sammccall added a comment. This revision is now accepted and ready to land. (LG from my side) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64199/new/ https://reviews.llvm.org/D64199

[PATCH] D64199: [clangd] Added highlighting for variable references (declrefs)

sammccall added inline comments. Comment at: clang-tools-extra/clangd/SemanticHighlighting.cpp:38 + bool VisitNamedDecl(NamedDecl *ND) { +if (ND->getDeclName().isEmpty()) + // Don't add symbols that don't have any length. jvikstrom wrote: > sammccall wr

[PATCH] D64210: [ARM] Fix vector vsqadd intrinsics operands

dnsampaio created this revision. dnsampaio added a reviewer: LukeCheeseman. Herald added subscribers: cfe-commits, kristof.beyls, javed.absar. Herald added a project: clang. Change the vsqadd vector instrinsics to have the second argument as signed values, not unsigned, accordingly to https://de

[PATCH] D64211: [ARM] Fix vector vuqadd intrinsics operands

dnsampaio created this revision. dnsampaio added a reviewer: LukeCheeseman. Herald added subscribers: cfe-commits, kristof.beyls, javed.absar. Herald added a project: clang. Change the vuqadd vector instrinsics to have the second argument as unsigned values, not signed, accordingly to https://de

[PATCH] D64210: [ARM] Fix vector vsqadd intrinsics operands

LukeCheeseman added a comment. Are there some changes/addition to tests attached to this? Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64210/new/ https://reviews.llvm.org/D64210 ___ cfe-commits mailing list cfe-comm

[PATCH] D64211: [ARM] Fix vector vuqadd intrinsics operands

LukeCheeseman added a comment. Are there some changes/addition to tests attached to this? Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64211/new/ https://reviews.llvm.org/D64211 ___ cfe-commits mailing list cfe-comm

[PATCH] D63325: [Support][Time profiler] Make FE codegen blocks to be inside frontend blocks

anton-afanasyev added a comment. F9459646: spirit-c5ceba.json F9459650: Снимок экрана от 2019-07-04 18-54-58.png I've upload json-file and printscreen of its visualization (used https://speedscope.app). This is an example o

[PATCH] D43159: Modernize: Use nullptr more.

ldionne added inline comments. Comment at: include/__threading_support:323 bool __libcpp_thread_isnull(const __libcpp_thread_t *__t) { - return *__t == 0; + return *__t == nullptr; } mclow.lists wrote: > mclow.lists wrote: > > This one is wrong. > `__libcpp_t

[clang-tools-extra] r365151 - [clangd] Mark a couple of unimportant flags as hidden. NFC

Author: sammccall Date: Thu Jul 4 11:49:08 2019 New Revision: 365151 URL: http://llvm.org/viewvc/llvm-project?rev=365151&view=rev Log: [clangd] Mark a couple of unimportant flags as hidden. NFC Modified: clang-tools-extra/trunk/clangd/tool/ClangdMain.cpp Modified: clang-tools-extra/trunk/cl

[PATCH] D64128: [CodeGen] Generate llvm.ptrmask instead of inttoptr(and(ptrtoint, C)) if possible.

rjmccall added a comment. In D64128#1569836 , @hfinkel wrote: > In D64128#1569817 , @rjmccall wrote: > > > The pointer/integer conversion is "implementation-defined", but it's not > > totally unconstrained. C note

r365153 - [modules] Add PP callbacks for entering and leaving a submodule.

Author: vvassilev Date: Thu Jul 4 12:06:52 2019 New Revision: 365153 URL: http://llvm.org/viewvc/llvm-project?rev=365153&view=rev Log: [modules] Add PP callbacks for entering and leaving a submodule. Modified: cfe/trunk/include/clang/Lex/PPCallbacks.h cfe/trunk/lib/Lex/PPLexerChange.cpp

[PATCH] D53295: Mark store and load of block invoke function as invariant.group

yaxunl added a comment. In D53295#1561890 , @rjmccall wrote: > Great, thank you. Yaxun, are you planning to pick this back up? I know it's > been a long time. Sorry I caught up with some other work. Currently there has been another change about block

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

v.g.vassilev closed this revision. v.g.vassilev added a comment. Herald added subscribers: wuzish, MaskRay. r365153. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62293/new/ https://reviews.llvm.org/D62293 ___ cfe-co

[PATCH] D62960: Add SVE opaque built-in types

rjmccall added inline comments. Comment at: lib/AST/ItaniumMangle.cpp:2680 +break; +#include "clang/Basic/AArch64SVEACLETypes.def" } rsandifo-arm wrote: > rovka wrote: > > erik.pilkington wrote: > > > rsandifo-arm wrote: > > > > erik.pilkington wrote: > >

[PATCH] D53295: Mark store and load of block invoke function as invariant.group

rjmccall added a reviewer: ahatanak. rjmccall added a comment. Herald added a subscriber: dexonsmith. Okay. Akira, do you have any interest in looking into this as a general block optimization? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D53295/new/ https://reviews.llvm.org/D53295

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

aaron.ballman added a comment. In D63845#1566987 , @jdoerfert wrote: > In D63845#1561983 , @lebedev.ri > wrote: > > > In D63845#1561793 , @jdoerfert > > wrote: > > > > > In

[PATCH] D64156: Make joined instances of JoinedOrSeparate flags point to the unaliased args, like all other arg types do

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

[PATCH] D63161: Devirtualize destructor of final class.

xbolva00 added a comment. https://reviews.llvm.org/rL364359 Why this was reverted ? Repository: rL LLVM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63161/new/ https://reviews.llvm.org/D63161 ___ cfe-commits mailing list cfe-commits@li

[PATCH] D63161: Devirtualize destructor of final class.

xbolva00 added a subscriber: rdhindsa. xbolva00 added a comment. You, @rdhindsa, should report here why you reverted this commit. It is really weird to revert random commits without more information. Repository: rL LLVM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63161/new/ https:

r365160 - [NFC] Make some ObjectFormatType switches covering

Author: hubert.reinterpretcast Date: Thu Jul 4 14:40:28 2019 New Revision: 365160 URL: http://llvm.org/viewvc/llvm-project?rev=365160&view=rev Log: [NFC] Make some ObjectFormatType switches covering Summary: This patch removes the `default` case from some switches on `llvm::Triple::ObjectFormatT

[PATCH] D63767: [NFC] Make some ObjectFormatType switches covering

This revision was automatically updated to reflect the committed changes. Closed by commit rL365160: [NFC] Make some ObjectFormatType switches covering (authored by hubert.reinterpretcast, committed by ). Herald added a subscriber: kristina. Changed prior to commit: https://reviews.llvm.org/D63

[PATCH] D62883: [analyzer] Track terminator conditions on which a tracked expressions depends

Szelethus updated this revision to Diff 208084. Szelethus marked 6 inline comments as done. Szelethus added a comment. - Add two more test cases when a "Returning value" note is meaningful, and one where it's not - Fix inlines! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62883/new/ h

[PATCH] D64222: [sanitizers] Use covering ObjectFormatType switches

hubert.reinterpretcast created this revision. hubert.reinterpretcast added reviewers: sfertile, jasonliu, daltenty. Herald added subscribers: sunfish, aheejin, hiraditya. Herald added projects: clang, LLVM. This patch removes the `default` case from some switches on `llvm::Triple::ObjectFormatTyp

[PATCH] D63602: [Sanitizers] Don't use clang_rt.sancov_{begin, end} on Solaris

ro added a comment. Ping^2. This one, like its companion https://reviews.llvm.org/D63601 has remained unreviewed for two weeks. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63602/new/ https://reviews.llvm.org/D63602 _

[PATCH] D64232: [analyzer] Prune calls to functions with linear CFGs that return a non-zero constrained value

Szelethus created this revision. Szelethus added reviewers: NoQ, xazax.hun, rnkovacs, baloghadamsoftware, dcoughlin, Charusso. Szelethus added a project: clang. Herald added subscribers: cfe-commits, gamesh411, dkrupp, donat.nagy, mikhail.ramalho, a.sidorin, szepet, whisperity. During the evalua

[PATCH] D63538: [CFG] Add a new function to get the proper condition of a CFGBlock

NoQ accepted this revision. NoQ added a comment. This revision is now accepted and ready to land. Looks great, thanks! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63538/new/ https://reviews.llvm.org/D63538 ___ cfe-commits mailing list cfe

[PATCH] D62883: [analyzer] Track terminator conditions on which a tracked expressions depends

NoQ accepted this revision. NoQ added a comment. Accept². Comment at: clang/lib/StaticAnalyzer/Core/BugReporterVisitors.cpp:1824-1825 + // because of this, let's explain why we believe control reached this point. + // TODO: Shouldn't we track control dependencies of every bug

[PATCH] D64232: [analyzer] Prune calls to functions with linear CFGs that return a non-zero constrained value

NoQ added a comment. I guess this makes sense. The results look good. I'm slightly worried that we're fighting the symptoms rather than the root cause here: why were these values tracked that far in the first place when we already have no interest in tracking them at the end of the function? I.

[PATCH] D64123: Add clang-llvm-rename tool.

ruiu updated this revision to Diff 208116. ruiu added a comment. - updated a few special mappings Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64123/new/ https://reviews.llvm.org/D64123 Files: clang-tools-extra/CMakeLists.txt clang-tools-extr

r365174 - Silence gcc warning "control reaches end of non-void function" [NFCI]

Author: uabelho Date: Thu Jul 4 23:12:24 2019 New Revision: 365174 URL: http://llvm.org/viewvc/llvm-project?rev=365174&view=rev Log: Silence gcc warning "control reaches end of non-void function" [NFCI] Without this fix gcc (7.4) complains with /data/repo/master/clang/lib/CodeGen/CGObjCMac.cpp