[PATCH] D147745: Make 'static assertion failed' diagnostics point to the static assertion expression

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG66202d83b5d4: Make static assertion failed diagnostics point to the static assertion… (authored by sousajo, committed by aaron.ballman). Herald

[clang] 66202d8 - Make 'static assertion failed' diagnostics point to the static assertion expression

Author: Jorge Pinto Sousa Date: 2023-04-13T08:15:13-04:00 New Revision: 66202d83b5d47479ae0f8117aebb523ba7eff82d URL: https://github.com/llvm/llvm-project/commit/66202d83b5d47479ae0f8117aebb523ba7eff82d DIFF:

[clang] dfafb7f - [clang][NFC] More range for loops in TextDiagnostic.cpp

Author: Timm Bäder Date: 2023-04-13T13:43:57+02:00 New Revision: dfafb7fe5af3e55ff35fb5dbc685a4337af5b29c URL: https://github.com/llvm/llvm-project/commit/dfafb7fe5af3e55ff35fb5dbc685a4337af5b29c DIFF: https://github.com/llvm/llvm-project/commit/dfafb7fe5af3e55ff35fb5dbc685a4337af5b29c.diff

Re: [clang] c1f7636 - [C++20] [Modules] Continue parsing after we found reserved module names

On Thu, Apr 13, 2023 at 3:14 AM Chuanqi Xu via cfe-commits wrote: > > > Author: Chuanqi Xu > Date: 2023-04-13T15:14:34+08:00 > New Revision: c1f76363e0db41ab6eb9ebedd687ee098491e9b7 > > URL: > https://github.com/llvm/llvm-project/commit/c1f76363e0db41ab6eb9ebedd687ee098491e9b7 > DIFF: >

[clang] 650d69f - [clang][NFC] Fix comment typo

Author: Timm Bäder Date: 2023-04-13T13:42:58+02:00 New Revision: 650d69ffe4d0ff2d2c84b8bf19555a8a9b10aae8 URL: https://github.com/llvm/llvm-project/commit/650d69ffe4d0ff2d2c84b8bf19555a8a9b10aae8 DIFF: https://github.com/llvm/llvm-project/commit/650d69ffe4d0ff2d2c84b8bf19555a8a9b10aae8.diff

[PATCH] D148158: [include-cleaner] Handle incomplete template specializations

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG3d6d2ae6f490: [include-cleaner] Handle incomplete template specializations (authored by kadircet). Repository: rG LLVM Github Monorepo CHANGES

[clang-tools-extra] 3d6d2ae - [include-cleaner] Handle incomplete template specializations

Author: Kadir Cetinkaya Date: 2023-04-13T13:34:12+02:00 New Revision: 3d6d2ae6f490f0698e3d4727ae44db34b30ea33e URL: https://github.com/llvm/llvm-project/commit/3d6d2ae6f490f0698e3d4727ae44db34b30ea33e DIFF:

[PATCH] D148158: [include-cleaner] Handle incomplete template specializations

kadircet added inline comments. Comment at: clang-tools-extra/include-cleaner/unittests/WalkASTTest.cpp:163 ElementsAre(Decl::ClassTemplatePartialSpecialization)); + // Incomplete templates don't have a specific specialization associated. +

[PATCH] D148158: [include-cleaner] Handle incomplete template specializations

kadircet updated this revision to Diff 513173. kadircet marked 2 inline comments as done. kadircet added a comment. - Update comments & test Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D148158/new/ https://reviews.llvm.org/D148158 Files:

[PATCH] D148213: [clangd] Use FileEntryRef for canonicalizing filepaths.

usaxena95 updated this revision to Diff 513168. usaxena95 added a comment. Rebase Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D148213/new/ https://reviews.llvm.org/D148213 Files: clang-tools-extra/clangd/Diagnostics.cpp

[PATCH] D148213: [clangd] Use FileEntryRef for canonicalizing filepaths.

usaxena95 updated this revision to Diff 513165. usaxena95 added a comment. More refactorings. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D148213/new/ https://reviews.llvm.org/D148213 Files: clang-tools-extra/clangd/Diagnostics.cpp

[PATCH] D148213: [clangd] Use FileEntryRef for canonicalizing filepaths.

usaxena95 created this revision. Herald added subscribers: kadircet, arphaman. Herald added a project: All. usaxena95 requested review of this revision. Herald added subscribers: cfe-commits, MaskRay, ilya-biryukov. Herald added a project: clang-tools-extra. Repository: rG LLVM Github Monorepo

[PATCH] D148158: [include-cleaner] Handle incomplete template specializations

sammccall accepted this revision. sammccall added inline comments. This revision is now accepted and ready to land. Comment at: clang-tools-extra/include-cleaner/lib/WalkAST.cpp:85 // This is the underlying decl used by TemplateSpecializationType, can be // null when

[PATCH] D148207: [mlir][Tensor] Drop SplitPaddingPatterns.

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG2031d7d66dc8: [mlir][Tensor] Drop SplitPaddingPatterns. (authored by nicolasvasilache). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST

[clang] 2031d7d - [mlir][Tensor] Drop SplitPaddingPatterns.

Author: Nicolas Vasilache Date: 2023-04-13T03:38:29-07:00 New Revision: 2031d7d66dc8bf7ce2168edf6eef6ba568c16d4f URL: https://github.com/llvm/llvm-project/commit/2031d7d66dc8bf7ce2168edf6eef6ba568c16d4f DIFF:

[PATCH] D146987: [Assignment Tracking] Enable by default

Orlando added a comment. In D146987#4264491 , @jmorse wrote: > ... > H, that feels like a legitimate IR input where the configuration of > debug-info / intrinsics is just unexpected -- we probably shouldn't assert in > that situation but gracefully

[PATCH] D143974: [clangd] Inactive regions support via dedicated protocol

hokein accepted this revision. hokein added a comment. This revision is now accepted and ready to land. Thanks, the implementation looks good. Comment at: clang-tools-extra/clangd/ClangdServer.cpp:977 + return CB(InpAST.takeError()); +// Include inactive regions in

[PATCH] D148211: [clang][tests] Fix Flang driver tests for Windows

kaadam created this revision. kaadam added reviewers: kiranchandramohan, bryanpkc, mstorsjo. Herald added a reviewer: sscalpone. Herald added a subscriber: sunshaoce. Herald added a project: All. kaadam requested review of this revision. Herald added a project: clang. Herald added a subscriber:

[PATCH] D147686: [clangd] Fix a nullptr-dereference crash in computeIncludeCleanerFindings.

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG2524000187fc: [clangd] Fix a nullptr-dereference crash in computeIncludeCleanerFindings. (authored by hokein). Repository: rG LLVM Github

[clang-tools-extra] 2524000 - [clangd] Fix a nullptr-dereference crash in computeIncludeCleanerFindings.

Author: Haojian Wu Date: 2023-04-13T12:17:39+02:00 New Revision: 2524000187fc56a3f818a6562199037a90108eda URL: https://github.com/llvm/llvm-project/commit/2524000187fc56a3f818a6562199037a90108eda DIFF: https://github.com/llvm/llvm-project/commit/2524000187fc56a3f818a6562199037a90108eda.diff

[PATCH] D147875: [clang][Diagnostics] WIP: Show line numbers when printing code snippets

tbaeder updated this revision to Diff 513144. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D147875/new/ https://reviews.llvm.org/D147875 Files: clang/include/clang/Basic/DiagnosticOptions.def clang/include/clang/Basic/DiagnosticOptions.h clang/include/clang/Driver/Options.td

[clang-tools-extra] 40276f7 - [include-fixer] Add the missing variant header to the STL header list.

Author: Haojian Wu Date: 2023-04-13T12:09:36+02:00 New Revision: 40276f78fd4dabc4368724c161ca918d6145ef75 URL: https://github.com/llvm/llvm-project/commit/40276f78fd4dabc4368724c161ca918d6145ef75 DIFF: https://github.com/llvm/llvm-project/commit/40276f78fd4dabc4368724c161ca918d6145ef75.diff

[PATCH] D148207: [mlir][Tensor] Drop SplitPaddingPatterns.

nicolasvasilache created this revision. Herald added subscribers: bviyer, hanchung, Moerafaat, bzcheeseman, sdasgup3, wenzhicui, wrengr, cota, teijeong, rdzhabarov, tatianashp, msifontes, jurahul, Kayjukh, grosul1, Joonsoo, liufengdb, aartbik, mgester, arpith-jacob, antiagainst, shauheen,

[clang] 1689a5d - [clang] fix an unused variable warning after 9d0b55f0e4ca55d04ee8abfdf021913ea3c30082

Author: Krasimir Georgiev Date: 2023-04-13T10:03:37Z New Revision: 1689a5d756f81d615bcfca8de2631b0ccc8a3917 URL: https://github.com/llvm/llvm-project/commit/1689a5d756f81d615bcfca8de2631b0ccc8a3917 DIFF: https://github.com/llvm/llvm-project/commit/1689a5d756f81d615bcfca8de2631b0ccc8a3917.diff

[PATCH] D146987: [Assignment Tracking] Enable by default

jmorse added a comment. Many thanks for all the feedback, In D146987#4263139 , @aeubanks wrote: > I'm seeing > > llc: ../../llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp:6214: void > llvm::SelectionDAGBuilder::visitIntrinsicCall(const

[PATCH] D148206: [clang] Do not crash after suggesting typo correction to constexpr if condition

Fznamznon created this revision. Herald added a project: All. Fznamznon requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. In some cases non-null non-constant yet valid expression may reach point where `ConditionResult` is created. For

[PATCH] D147686: [clangd] Fix a nullptr-dereference crash in computeIncludeCleanerFindings.

hokein added inline comments. Comment at: clang-tools-extra/clangd/unittests/IncludeCleanerTests.cpp:450 +void test() { + 1s; +} kadircet wrote: > hokein wrote: > > here is the `UserDefinedLiteral` AST node: > > > > ``` > > `-UserDefinedLiteral

[PATCH] D147686: [clangd] Fix a nullptr-dereference crash in computeIncludeCleanerFindings.

hokein updated this revision to Diff 513134. hokein marked an inline comment as not done. hokein added a comment. use syntax::spelledTokensTouching. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D147686/new/ https://reviews.llvm.org/D147686 Files:

[PATCH] D146987: [Assignment Tracking] Enable by default

Orlando added a comment. In D146987#4263263 , @rupprecht wrote: > In D146987#4263048 , @MaskRay wrote: > >> The latest reland 3820e9a2b29a2e268319ed6635da0d59e18d736d >>

[PATCH] D148148: [clang] Bump AS_GNU to 1

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG053bdb77b0ce: [clang] Bump AS_GNU to 1 (authored by rsandifo-arm). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D148105: [clang] Fix FIXME in isAlignasAttribute()

This revision was automatically updated to reflect the committed changes. Closed by commit rGbd41371be02f: [clang] Fix FIXME in isAlignasAttribute() (authored by rsandifo-arm). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D148105/new/

[PATCH] D148104: [clang] Type safety tweak for AttributeCommonInfo::Form

This revision was automatically updated to reflect the committed changes. Closed by commit rGaec3f951bf36: [clang] Type safety tweak for AttributeCommonInfo::Form (authored by rsandifo-arm). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D148104/new/

[PATCH] D148103: [clang] Allow attributes to be constructed from keyword tokens

This revision was automatically updated to reflect the committed changes. Closed by commit rG265d87e46535: [clang] Allow attributes to be constructed from keyword tokens (authored by rsandifo-arm). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D148102: [clang] Specify attribute syntax & spelling with a single argument

This revision was automatically updated to reflect the committed changes. Closed by commit rGb6d4d51f8f5a: [clang] Specify attribute syntax spelling with a single argument (authored by rsandifo-arm). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D148101: [clang] Ensure that Attr::Create(Implicit) chooses a valid syntax

This revision was automatically updated to reflect the committed changes. Closed by commit rGe841d5092634: [clang] Ensure that Attr::Create(Implicit) chooses a valid syntax (authored by rsandifo-arm). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[clang] 053bdb7 - [clang] Bump AS_GNU to 1

Author: Richard Sandiford Date: 2023-04-13T10:14:50+01:00 New Revision: 053bdb77b0ce4506d15ed381b6db0dddafe52c3e URL: https://github.com/llvm/llvm-project/commit/053bdb77b0ce4506d15ed381b6db0dddafe52c3e DIFF:

[clang] bd41371 - [clang] Fix FIXME in isAlignasAttribute()

Author: Richard Sandiford Date: 2023-04-13T10:14:50+01:00 New Revision: bd41371be02f6f5713459a2f6fe109cd3c01b4a4 URL: https://github.com/llvm/llvm-project/commit/bd41371be02f6f5713459a2f6fe109cd3c01b4a4 DIFF:

[clang] aec3f95 - [clang] Type safety tweak for AttributeCommonInfo::Form

Author: Richard Sandiford Date: 2023-04-13T10:14:49+01:00 New Revision: aec3f951bf368d45b441554dac6e027678f9f3ee URL: https://github.com/llvm/llvm-project/commit/aec3f951bf368d45b441554dac6e027678f9f3ee DIFF:

[clang] 265d87e - [clang] Allow attributes to be constructed from keyword tokens

Author: Richard Sandiford Date: 2023-04-13T10:14:49+01:00 New Revision: 265d87e46535bef2b718759ba39bb9fa30b1ef48 URL: https://github.com/llvm/llvm-project/commit/265d87e46535bef2b718759ba39bb9fa30b1ef48 DIFF:

[clang] b6d4d51 - [clang] Specify attribute syntax & spelling with a single argument

Author: Richard Sandiford Date: 2023-04-13T10:14:49+01:00 New Revision: b6d4d51f8f5aab311df34c753b925760578729bd URL: https://github.com/llvm/llvm-project/commit/b6d4d51f8f5aab311df34c753b925760578729bd DIFF:

[clang] e841d50 - [clang] Ensure that Attr::Create(Implicit) chooses a valid syntax

Author: Richard Sandiford Date: 2023-04-13T10:14:48+01:00 New Revision: e841d50926347c3596009dd4fbc4cbd1a1a2e0b8 URL: https://github.com/llvm/llvm-project/commit/e841d50926347c3596009dd4fbc4cbd1a1a2e0b8 DIFF:

[PATCH] D146987: [Assignment Tracking] Enable by default

Orlando added a comment. In D146987#4263139 , @aeubanks wrote: > I'm seeing > > llc: ../../llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp:6214: void > llvm::SelectionDAGBuilder::visitIntrinsicCall(const CallInst &, unsigned > int): Assertion

[PATCH] D145214: [TSAN] add support for riscv64

alexfanqi added a comment. friendly ping? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D145214/new/ https://reviews.llvm.org/D145214 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[libunwind] ebae562 - [libunwind] [SEH] Initialize _msContext with RtlCaptureContext

Author: Martin Storsjö Date: 2023-04-13T12:03:35+03:00 New Revision: ebae5622d11655feaa74e64151067897153f9c71 URL: https://github.com/llvm/llvm-project/commit/ebae5622d11655feaa74e64151067897153f9c71 DIFF:

[libunwind] 5b9d969 - [libunwind] [SEH] Allow setting/getting the register UNW_X86_64_RIP

Author: Martin Storsjö Date: 2023-04-13T12:03:35+03:00 New Revision: 5b9d969e7c07d720080eac386467c1112c45a76f URL: https://github.com/llvm/llvm-project/commit/5b9d969e7c07d720080eac386467c1112c45a76f DIFF:

[clang] 39938f2 - Fix warn-unsafe-buffer-usage-fixits-pre-increment.cpp for Windows

Author: Hans Wennborg Date: 2023-04-13T10:21:15+02:00 New Revision: 39938f2d096ca9ed03ecc17ea169ed3195682f18 URL: https://github.com/llvm/llvm-project/commit/39938f2d096ca9ed03ecc17ea169ed3195682f18 DIFF: https://github.com/llvm/llvm-project/commit/39938f2d096ca9ed03ecc17ea169ed3195682f18.diff

[PATCH] D146941: [clangd] Use all inputs to SystemIncludeExtractor in cache key

kadircet updated this revision to Diff 513098. kadircet marked an inline comment as done. kadircet added a comment. Fix typo Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D146941/new/ https://reviews.llvm.org/D146941 Files:

[PATCH] D147905: [clangd] Avoid passing -xobjective-c++-header to the system include extractor

kadircet added inline comments. Comment at: clang-tools-extra/clangd/SystemIncludeExtractor.cpp:340 +// is not installed. +if (Lang == "objective-c++-header") { + Lang = "c++-header"; this feels like too much of a layering violation and might

[PATCH] D145868: [clang][ASTImporter] Fix import of typedef with unnamed structures

bjope added inline comments. Comment at: clang/unittests/AST/ASTImporterTest.cpp:8627 + )"; + Decl *ToTU = getToTuDecl("", Lang_CXX11); + Decl *FromTU = getTuDecl(Code, Lang_CXX11); With Werror we get: ```

[PATCH] D146941: [clangd] Use all inputs to SystemIncludeExtractor in cache key

nridge accepted this revision. nridge added a comment. This revision is now accepted and ready to land. Thanks! Do you have any thoughts on merging D147905 first, with a view to backporting D147905 to the 16.x branch?

[PATCH] D146987: [Assignment Tracking] Enable by default

Orlando added a comment. Thank you for the reports and revert, I will dig into it. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D146987/new/ https://reviews.llvm.org/D146987 ___ cfe-commits mailing list

[PATCH] D147520: Fix a time-trace issue of incorrect header hierarchy when a header contains a template function for its last symbol.

MaggieYi added a comment. Gentle ping ... Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D147520/new/ https://reviews.llvm.org/D147520 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D148200: [clang-format] Correctly handle "// clang-format off" above macros

owenpan created this revision. owenpan added reviewers: MyDeveloperDay, HazardyKnusperkeks, rymiel. owenpan added a project: clang-format. Herald added projects: All, clang. Herald added a subscriber: cfe-commits. owenpan requested review of this revision. Fixes

[PATCH] D145868: [clang][ASTImporter] Fix import of typedef with unnamed structures

This revision was automatically updated to reflect the committed changes. Closed by commit rG9d0b55f0e4ca: [clang][ASTImporter] Fix import of typedef with unnamed structures (authored by balazske). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[clang] 9d0b55f - [clang][ASTImporter] Fix import of typedef with unnamed structures

Author: Balázs Kéri Date: 2023-04-13T09:25:12+02:00 New Revision: 9d0b55f0e4ca55d04ee8abfdf021913ea3c30082 URL: https://github.com/llvm/llvm-project/commit/9d0b55f0e4ca55d04ee8abfdf021913ea3c30082 DIFF: https://github.com/llvm/llvm-project/commit/9d0b55f0e4ca55d04ee8abfdf021913ea3c30082.diff

[clang] c1f7636 - [C++20] [Modules] Continue parsing after we found reserved module names

Author: Chuanqi Xu Date: 2023-04-13T15:14:34+08:00 New Revision: c1f76363e0db41ab6eb9ebedd687ee098491e9b7 URL: https://github.com/llvm/llvm-project/commit/c1f76363e0db41ab6eb9ebedd687ee098491e9b7 DIFF: https://github.com/llvm/llvm-project/commit/c1f76363e0db41ab6eb9ebedd687ee098491e9b7.diff

[PATCH] D147307: [clang] Do not require GNUInlineAttr for inline builtins

This revision was automatically updated to reflect the committed changes. Closed by commit rGfd8745c252bc: [clang] Do not require GNUInlineAttr for inline builtins (authored by serge-sans-paille). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[clang] fd8745c - [clang] Do not require GNUInlineAttr for inline builtins

Author: serge-sans-paille Date: 2023-04-13T09:03:58+02:00 New Revision: fd8745c252bcd4bc974ce3ff30c19f1cc5c4e3e0 URL: https://github.com/llvm/llvm-project/commit/fd8745c252bcd4bc974ce3ff30c19f1cc5c4e3e0 DIFF:

[PATCH] D147686: [clangd] Fix a nullptr-dereference crash in computeIncludeCleanerFindings.

kadircet added inline comments. Comment at: clang-tools-extra/clangd/unittests/IncludeCleanerTests.cpp:450 +void test() { + 1s; +} hokein wrote: > here is the `UserDefinedLiteral` AST node: > > ``` > `-UserDefinedLiteral 0x5556682e4500 'int' >

[PATCH] D146941: [clangd] Use all inputs to SystemIncludeExtractor in cache key

kadircet updated this revision to Diff 513070. kadircet marked 2 inline comments as done. kadircet added a comment. - Address comments Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D146941/new/ https://reviews.llvm.org/D146941 Files:

[PATCH] D146941: [clangd] Use all inputs to SystemIncludeExtractor in cache key

kadircet marked 3 inline comments as done. kadircet added inline comments. Comment at: clang-tools-extra/clangd/SystemIncludeExtractor.cpp:80 + std::string Driver; + std::string Directory; + // Whether certain includes should be part of query. nridge wrote: >

[PATCH] D147121: [hwasan] remove requirment for PIE

This revision was automatically updated to reflect the committed changes. Closed by commit rGb356cd70a27d: [hwasan] remove requirment for PIE (authored by Enna1). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D147121/new/

[clang] b356cd7 - [hwasan] remove requirment for PIE

Author: Enna1 Date: 2023-04-13T14:15:32+08:00 New Revision: b356cd70a27dd5be2cbcc72e9f1ac56f1fb788cb URL: https://github.com/llvm/llvm-project/commit/b356cd70a27dd5be2cbcc72e9f1ac56f1fb788cb DIFF: https://github.com/llvm/llvm-project/commit/b356cd70a27dd5be2cbcc72e9f1ac56f1fb788cb.diff LOG:

<    1   2