[clang] 13a784f - [clang][AArch64][SVE] Change SVE_VECTOR_OPERATORS macro for VLA vectors

Author: David Truby Date: 2022-08-11T13:23:52Z New Revision: 13a784f368ef062a7bd652829bcf8bbdd94dc659 URL: https://github.com/llvm/llvm-project/commit/13a784f368ef062a7bd652829bcf8bbdd94dc659 DIFF: https://github.com/llvm/llvm-project/commit/13a784f368ef062a7bd652829bcf8bbdd94dc659.diff LOG:

[PATCH] D131573: [clang][AArch64][SVE] Change SVE_VECTOR_OPERATORS macro for VLA vectors

This revision was automatically updated to reflect the committed changes. DavidTruby marked an inline comment as done. Closed by commit rG13a784f368ef: [clang][AArch64][SVE] Change SVE_VECTOR_OPERATORS macro for VLA vectors (authored by DavidTruby). Repository: rG LLVM Github Monorepo CHANGES

[PATCH] D131528: [Clang] Restrict non fixed enum to a value outside the range of the enumeration values warning to context requiring a constant expression

erichkeane added a comment. In D131528#3715841 , @thakis wrote: > We're also still seeing the diag fire after this: > https://ci.chromium.org/p/chromium/builders/ci/ToTLinux > > (And we cleaned up our codebase back when it was still an error.) > > Our

[PATCH] D126907: Deferred Concept Instantiation Implementation Take 2

erichkeane added a comment. In D126907#3715436 , @ChuanqiXu wrote: > In D126907#3712363 , @erichkeane > wrote: > >> In D126907#3711956 , @ChuanqiXu >> wrote: >> >>> In

[PATCH] D131657: Remove redundant condition check, NFC

erichkeane accepted this revision. erichkeane added a comment. LGTM! Let us know if you need this committed for you. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D131657/new/ https://reviews.llvm.org/D131657

[PATCH] D131651: [AST] [Modules] Introduce Decl::getNonTransparentDeclContext to handle exported friends

erichkeane accepted this revision. erichkeane added a comment. This revision is now accepted and ready to land. LGTM, thanks! Comment at: clang/lib/AST/DeclBase.cpp:1037 + assert(getDeclContext()); + return getDeclContext()->getNonTransparentContext(); +}

[PATCH] D131625: [HLSL] Entry functions require param annotation

aaron.ballman added inline comments. Comment at: clang/include/clang/AST/Attr.h:193 +class HLSLAnnotationAttr : public InheritableAttr { +protected: Is this intended to be used only for parameters (that's how I read the summary for the patch)? If so, why is

[clang] bbb30bd - [clang][AArch64][SVE] Clarify documentation for sizeof operator on SVE

Author: David Truby Date: 2022-08-11T13:22:23Z New Revision: bbb30bd54a6447702f9f59a2ae4c478eb7133ae0 URL: https://github.com/llvm/llvm-project/commit/bbb30bd54a6447702f9f59a2ae4c478eb7133ae0 DIFF: https://github.com/llvm/llvm-project/commit/bbb30bd54a6447702f9f59a2ae4c478eb7133ae0.diff LOG:

[PATCH] D131651: [AST] [Modules] Introduce Decl::getNonTransparentDeclContext to handle exported friends

erichkeane added a comment. I think this is fine. I vaguely remember doing something like this in the past, but I can't seem to figure out what it was... Comment at: clang/lib/AST/DeclBase.cpp:1037 + assert(getDeclContext()); + return

[PATCH] D119051: Extend the C++03 definition of POD to include defaulted functions

aaron.ballman added a comment. In D119051#3714645 , @dblaikie wrote: > Realized maybe we don't need a separate driver flag for this at all, and rely > only on the abi-compat flag? That seems to be how (at least some) other ABI > compat changes have

[PATCH] D130058: [Clang] Diagnose ill-formed constant expression when setting a non fixed enum to a value outside the range of the enumeration values

erichkeane added a comment. > In D130058#3714882 , @glandium > wrote: > >> Also not caught: a cast of 0 when 0 is not a valid value in the enum. > > I don't think that situation will ever be UB. When the underlying type of the > enum is not fixed, the

[PATCH] D131469: [Clang] change default storing path of `-ftime-trace`

jamieschmeiser added a comment. You should not have debugging information in code that is up for review. If this is debugging information that you plan to leave in for future purposes (which I doubt is the case here), you need to protect it so that it isn't active unless some option is set.

[clang] d09d4bd - [clang][dataflow] Don't crash when caller args are missing storage locations

Author: Sam Estep Date: 2022-08-11T13:00:42Z New Revision: d09d4bd66c864d58b29d74918a4a164f3ad905de URL: https://github.com/llvm/llvm-project/commit/d09d4bd66c864d58b29d74918a4a164f3ad905de DIFF: https://github.com/llvm/llvm-project/commit/d09d4bd66c864d58b29d74918a4a164f3ad905de.diff LOG:

[PATCH] D131675: [clang] SIGSEGV fix at clang::ASTContext::getRawCommentForDeclNoCacheImpl

ivanmurashko created this revision. ivanmurashko added reviewers: sammccall, aaron.ballman. Herald added subscribers: usaxena95, kadircet. Herald added a project: All. ivanmurashko requested review of this revision. Herald added subscribers: cfe-commits, ilya-biryukov. Herald added a project:

[clang] eb91fd5 - [clang][dataflow] Analyze constructor bodies

Author: Sam Estep Date: 2022-08-11T12:46:20Z New Revision: eb91fd5cbc6995099bd38c04c839686cbdd18b92 URL: https://github.com/llvm/llvm-project/commit/eb91fd5cbc6995099bd38c04c839686cbdd18b92 DIFF: https://github.com/llvm/llvm-project/commit/eb91fd5cbc6995099bd38c04c839686cbdd18b92.diff LOG:

[PATCH] D130363: [clang] Give priority to Class context while parsing declarations

furkanusta marked an inline comment as not done. furkanusta added a comment. That would be great, thanks Comment at: clang/test/CodeCompletion/overrides.cpp:41 +// Runs completion at empty line on line 13. +// RUN: %clang_cc1 -fsyntax-only -code-completion-patterns

[PATCH] D130894: [clang] Print more information about failed static assertions

aaron.ballman added a comment. In D130894#3715884 , @xbolva00 wrote: > Btw, > > https://www.open-std.org/jtc1/sc22/wg21/docs/papers/2019/p1697r0.html Yup, that paper is currently closed in the WG21 paper tracker FWIW. Repository: rG LLVM Github

[PATCH] D131580: [clang][SVE] Undefine preprocessor macro defined in

mgabka marked an inline comment as done. mgabka added a comment. It is a fix for https://github.com/llvm/llvm-project/issues/57083 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D131580/new/ https://reviews.llvm.org/D131580

[PATCH] D130894: [clang] Print more information about failed static assertions

xbolva00 added a comment. Btw, https://www.open-std.org/jtc1/sc22/wg21/docs/papers/2019/p1697r0.html Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D130894/new/ https://reviews.llvm.org/D130894 ___

[PATCH] D131645: [clang][dataflow] Allow user-provided lattices to provide a widen operator

ymandel accepted this revision. ymandel added inline comments. This revision is now accepted and ready to land. Comment at: clang/unittests/Analysis/FlowSensitive/TypeErasedDataflowAnalysisTest.cpp:119 + std::unique_ptr AST = + tooling::buildASTFromCodeWithArgs("int x =

[PATCH] D131351: [C] Default implicit function pointer conversions diagnostic to be an error

aaron.ballman added a comment. In D131351#3714968 , @brooksmoses wrote: > For the record, so far we've seen this showing up in the following: > > - A case in a notoriously warning-heavy third-party library where we'd > backported a file from a newer

[PATCH] D131646: [clang][dataflow] Restructure loops to call widen on back edges

ymandel accepted this revision. ymandel added a comment. This revision is now accepted and ready to land. Nice! Comment at: clang/lib/Analysis/FlowSensitive/TypeErasedDataflowAnalysis.cpp:180 + assert(Block->pred_size() == 2); + BackEdge = Pred; +}

[PATCH] D131528: [Clang] Restrict non fixed enum to a value outside the range of the enumeration values warning to context requiring a constant expression

thakis added a comment. We're also still seeing the diag fire after this: https://ci.chromium.org/p/chromium/builders/ci/ToTLinux (And we cleaned up our codebase back when it was still an error.) Our bots have been red since the change to turn this into a warning landed. Repository: rG

[PATCH] D131091: [clang][index] Index unresolved member expression as reference

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

[PATCH] D131396: [pseudo] Use C++17 variant to simplify the DirectiveTree::Chunk class, NFC.

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. hokein marked an inline comment as done. Closed by commit rGbf0e219d0481: [pseudo] Use C++17 variant to simplify the DirectiveTree::Chunk class, NFC. (authored by

[clang-tools-extra] bf0e219 - [pseudo] Use C++17 variant to simplify the DirectiveTree::Chunk class, NFC.

Author: Haojian Wu Date: 2022-08-11T14:27:38+02:00 New Revision: bf0e219d0481212ad12667262ed0b27e5e69f5f2 URL: https://github.com/llvm/llvm-project/commit/bf0e219d0481212ad12667262ed0b27e5e69f5f2 DIFF: https://github.com/llvm/llvm-project/commit/bf0e219d0481212ad12667262ed0b27e5e69f5f2.diff

[PATCH] D131569: [clangd] Allow updates to be canceled after compile flags retrieval

adamcz added inline comments. Comment at: clang-tools-extra/clangd/TUScheduler.cpp:898 +if (isCancelled()) { + log("ASTWorker skipping update {0} for file {1}", Inputs.Version, + FileName); I'd add something like "due to cancellation". I know

[PATCH] D131396: [pseudo] Use C++17 variant to simplify the DirectiveTree::Chunk class, NFC.

hokein updated this revision to Diff 451811. hokein marked 2 inline comments as done. hokein added a comment. address comments. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D131396/new/ https://reviews.llvm.org/D131396 Files:

[PATCH] D131479: Handle explicitly defaulted consteval special members.

aaron.ballman added inline comments. Comment at: clang/test/SemaCXX/cxx2a-consteval.cpp:805 +void func() { + default_ctor fail0; // expected-error-re {{call to consteval function '{{.*::default_ctor<.*::foo>}}::default_ctor' is not a constant expression}} \ +

[PATCH] D130058: [Clang] Diagnose ill-formed constant expression when setting a non fixed enum to a value outside the range of the enumeration values

aaron.ballman added a comment. In D130058#3714880 , @glandium wrote: > In D130058#3714672 , @glandium > wrote: > >> This catches >>

[PATCH] D130894: [clang] Print more information about failed static assertions

aaron.ballman added a comment. In D130894#3715725 , @mstorsjo wrote: > In D130894#3715709 , @aaron.ballman > wrote: > >> In D130894#3715143 , @xbolva00 >> wrote: >> >>>

[PATCH] D130894: [clang] Print more information about failed static assertions

mstorsjo added a comment. In D130894#3715709 , @aaron.ballman wrote: > In D130894#3715143 , @xbolva00 > wrote: > >> In D130894#3715124 , @mstorsjo >> wrote: >> >>>

[PATCH] D131194: [C++20] Fix crash-on-valid with consteval temporary construction through list initialization

aaron.ballman marked 2 inline comments as done. aaron.ballman added a comment. Ping CHANGES SINCE LAST ACTION https://reviews.llvm.org/D131194/new/ https://reviews.llvm.org/D131194 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D131469: [Clang] change default storing path of `-ftime-trace`

dongjunduo updated this revision to Diff 451804. dongjunduo added a comment. Add more debug info Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D131469/new/ https://reviews.llvm.org/D131469 Files: clang/lib/Driver/Driver.cpp

[PATCH] D130894: [clang] Print more information about failed static assertions

aaron.ballman added a comment. In D130894#3715143 , @xbolva00 wrote: > In D130894#3715124 , @mstorsjo > wrote: > >> This broke building with GCC (also noted by buildbot mails): >> >>

[clang] 7309e8c - Do not use the same identifier for a data member as a type; NFC

Author: Aaron Ballman Date: 2022-08-11T07:32:32-04:00 New Revision: 7309e8cfbe59844b4e0128c3f48eb1a68d8bad68 URL: https://github.com/llvm/llvm-project/commit/7309e8cfbe59844b4e0128c3f48eb1a68d8bad68 DIFF: https://github.com/llvm/llvm-project/commit/7309e8cfbe59844b4e0128c3f48eb1a68d8bad68.diff

[PATCH] D129231: [Builtins] Do not claim all libfuncs are readnone with trapping math.

fhahn added inline comments. Comment at: clang/include/clang/Basic/Builtins.def:1409 -LIBBUILTIN(round, "dd", "fnc", "math.h", ALL_LANGUAGES) -LIBBUILTIN(roundf, "ff", "fnc", "math.h", ALL_LANGUAGES) -LIBBUILTIN(roundl, "LdLd", "fnc", "math.h", ALL_LANGUAGES)

[PATCH] D131657: Remove redundant condition check, NFC

aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. LTGM! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D131657/new/ https://reviews.llvm.org/D131657

[PATCH] D129231: [Builtins] Do not claim all libfuncs are readnone with trapping math.

This revision was automatically updated to reflect the committed changes. Closed by commit rGef110a491f70: [Builtins] Do not claim most libfuncs are readnone with trapping math. (authored by fhahn). Changed prior to commit: https://reviews.llvm.org/D129231?vs=448628=451800#toc Repository:

[clang] ef110a4 - [Builtins] Do not claim most libfuncs are readnone with trapping math.

Author: Florian Hahn Date: 2022-08-11T12:29:01+01:00 New Revision: ef110a491f702e84d3f97e7cde6dacea99fd238d URL: https://github.com/llvm/llvm-project/commit/ef110a491f702e84d3f97e7cde6dacea99fd238d DIFF: https://github.com/llvm/llvm-project/commit/ef110a491f702e84d3f97e7cde6dacea99fd238d.diff

[PATCH] D130363: [clang] Give priority to Class context while parsing declarations

kadircet accepted this revision. kadircet added a comment. This revision is now accepted and ready to land. thanks, lgtm! let me know if i should land this for you. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D130363/new/

[clang-tools-extra] e935f7f - [pseudo] Fix a bug in checking the duplicated grammar rules.

Author: Haojian Wu Date: 2022-08-11T13:16:01+02:00 New Revision: e935f7fd0cbc698438bdabc3b836ab25baa9a174 URL: https://github.com/llvm/llvm-project/commit/e935f7fd0cbc698438bdabc3b836ab25baa9a174 DIFF: https://github.com/llvm/llvm-project/commit/e935f7fd0cbc698438bdabc3b836ab25baa9a174.diff

[PATCH] D111283: [clang] template / auto deduction deduces common sugar

mizvekov added a comment. In D111283#3712348 , @erichkeane wrote: > This is a sizable patch, and it isn't really clear from context what I'm > looking at. Can you please improve the description/commit message to better > clarify what you are trying

[PATCH] D111283: [clang] template / auto deduction deduces common sugar

mizvekov edited the summary of this revision. mizvekov updated this revision to Diff 451799. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111283/new/ https://reviews.llvm.org/D111283 Files: clang-tools-extra/clangd/unittests/ASTTests.cpp

[PATCH] D131614: [clang][dataflow] Extend transfer functions for other `CFGElement`s

gribozavr2 added inline comments. Comment at: clang/include/clang/Analysis/FlowSensitive/DataflowAnalysis.h:103 + virtual void transferCFGElement(const CFGElement *Element, Lattice , + Environment ) {} + Instead of adding

[PATCH] D130689: [LLVM] Update C++ standard to 17

thieta added inline comments. Comment at: llvm/docs/ReleaseNotes.rst:57 + +* GCC >= 7.1 +* Clang >= 5.0 xbolva00 wrote: > Do we have bots which uses last supported compiler versions? GCC 7.1, Clang > 5.0 and MSVC 16.7. > > It is bad to promise something and

[PATCH] D131665: [CMake] Support passing arguments to build tool (bootstrap).

CarlosAlbertoEnciso created this revision. CarlosAlbertoEnciso added reviewers: andrewng, phosek, beanz, russell.gallop. CarlosAlbertoEnciso added a project: LLVM. Herald added a subscriber: mgorny. Herald added a project: All. CarlosAlbertoEnciso requested review of this revision. Herald added a

[PATCH] D131469: [Clang] change default storing path of `-ftime-trace`

dongjunduo updated this revision to Diff 451796. dongjunduo added a comment. Add more debug info Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D131469/new/ https://reviews.llvm.org/D131469 Files: clang/lib/Driver/Driver.cpp

[PATCH] D130689: [LLVM] Update C++ standard to 17

xbolva00 added inline comments. Comment at: llvm/docs/ReleaseNotes.rst:57 + +* GCC >= 7.1 +* Clang >= 5.0 Do we have bots which uses last supported compiler versions? GCC 7.1, Clang 5.0 and MSVC 16.7. It is bad to promise something and then breakages here and

[PATCH] D131555: [Clang] Propagate const context info when emitting compound literal

stuij marked 2 inline comments as done. stuij added inline comments. Comment at: clang/lib/CodeGen/CGExprConstant.cpp:2217 assert(E->isFileScope() && "not a file-scope compound literal expr"); - return tryEmitGlobalCompoundLiteral(*this, nullptr, E); + ConstantEmitter

[PATCH] D131555: [Clang] Propagate const context info when emitting compound literal

stuij updated this revision to Diff 451794. stuij added a comment. addressed review comment Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D131555/new/ https://reviews.llvm.org/D131555 Files: clang/lib/CodeGen/CGExprConstant.cpp

[PATCH] D90275: [clang][IR] Add support for leaf attribute

xbolva00 added a comment. Was there any follow up? It seems that this hint (attribute) is currently not used by LLVM passes. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D90275/new/ https://reviews.llvm.org/D90275

[PATCH] D131580: [clang][SVE] Undefine preprocessor macro defined in

mgabka marked an inline comment as done. mgabka added inline comments. Comment at: clang/utils/TableGen/SveEmitter.cpp:1285 OS << "#endif\n\n"; + OS << "#undef __ai\n\n"; OS << "#endif /*__ARM_FEATURE_SVE */\n\n"; paulwalker-arm wrote: > Can you also do

[PATCH] D131580: [clang][SVE] Undefine preprocessor macro defined in

mgabka updated this revision to Diff 451791. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D131580/new/ https://reviews.llvm.org/D131580 Files: clang/utils/TableGen/SveEmitter.cpp Index: clang/utils/TableGen/SveEmitter.cpp

[PATCH] D131469: [Clang] change default storing path of `-ftime-trace`

dongjunduo updated this revision to Diff 451785. dongjunduo added a comment. [Clang] add -### for debug Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D131469/new/ https://reviews.llvm.org/D131469 Files: clang/lib/Driver/Driver.cpp

[PATCH] D130701: [clang-tidy] Rename a local cmake variables to match the new tool name. NFC.

mstorsjo added a comment. Ping @sammccall (although I guess I could just go ahead and commit this as it is quite trivial, but as I sent it out for review in the first place...) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D130701/new/

[PATCH] D131555: [Clang] Propagate const context info when emitting compound literal

vhscampos added inline comments. Comment at: clang/lib/CodeGen/CGExprConstant.cpp:2217 assert(E->isFileScope() && "not a file-scope compound literal expr"); - return tryEmitGlobalCompoundLiteral(*this, nullptr, E); + ConstantEmitter emitter(*this, nullptr); + return

[PATCH] D131555: [Clang] Propagate const context info when emitting compound literal

vhscampos added inline comments. Comment at: clang/lib/CodeGen/CGExprConstant.cpp:2217 assert(E->isFileScope() && "not a file-scope compound literal expr"); - return tryEmitGlobalCompoundLiteral(*this, nullptr, E); + ConstantEmitter emitter(*this, nullptr); + return

[PATCH] D131662: [clang] Try to improve diagnostics about uninitialized constexpr variables

tbaeder created this revision. tbaeder added a reviewer: aaron.ballman. Herald added a project: All. tbaeder requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Consider: constexpr int a; the error message for this is currently: error:

[PATCH] D131555: [Clang] Propagate const context info when emitting compound literal

stuij updated this revision to Diff 451783. stuij marked an inline comment as done. stuij added a comment. also added a comment above the runline. Initially thought it wouldn't be necessary as we're now doing a check, but I do agree that it's more clear to be specific about the cmdline arg.

[PATCH] D131555: [Clang] Propagate const context info when emitting compound literal

stuij updated this revision to Diff 451781. stuij added a comment. addressed review comments Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D131555/new/ https://reviews.llvm.org/D131555 Files: clang/lib/CodeGen/CGExprConstant.cpp

[PATCH] D126907: Deferred Concept Instantiation Implementation Take 2

ChuanqiXu added a comment. In D126907#3712363 , @erichkeane wrote: > In D126907#3711956 , @ChuanqiXu > wrote: > >> In D126907#3710656 , @erichkeane >> wrote: >> >>>

[PATCH] D126880: [clang-tidy] Add cppcoreguidelines-avoid-const-or-ref-data-members check

This revision was not accepted when it landed; it landed in state "Needs Review". This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG9ae5896d9673: [clang-tidy] Add

[clang-tools-extra] 9ae5896 - [clang-tidy] Add cppcoreguidelines-avoid-const-or-ref-data-members check

Author: Carlos Galvez Date: 2022-08-11T07:46:04Z New Revision: 9ae5896d9673a54f4a6cf656624891bafc192857 URL: https://github.com/llvm/llvm-project/commit/9ae5896d9673a54f4a6cf656624891bafc192857 DIFF: https://github.com/llvm/llvm-project/commit/9ae5896d9673a54f4a6cf656624891bafc192857.diff

[PATCH] D126880: [clang-tidy] Add cppcoreguidelines-avoid-const-or-ref-data-members check

bigdavedev accepted this revision. bigdavedev added a comment. LGTM. All comments have been addressed. Any improvements to the diagnostics can be made down the line depending on how users feel about them. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D126880: [clang-tidy] Add cppcoreguidelines-avoid-const-or-ref-data-members check

carlosgalvezp updated this revision to Diff 451760. carlosgalvezp added a comment. Rebase Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D126880/new/ https://reviews.llvm.org/D126880 Files:

[PATCH] D131563: [WIP] [clang] Fix clang multiarch isssue with musl

listout marked an inline comment as done. listout added a comment. @barannikov88 made some changes. What do you think? Comment at: clang/lib/Driver/ToolChains/Linux.cpp:49 + std::string EnvName = TargetTriple.isMusl() ? "musl" : "gnu"; + barannikov88 wrote:

[PATCH] D131563: [WIP] [clang] Fix clang multiarch isssue with musl

listout updated this revision to Diff 451757. listout added a comment. Using `getEnvironmentTypeName()` instead of `isMusl()` CHANGES SINCE LAST ACTION https://reviews.llvm.org/D131563/new/ https://reviews.llvm.org/D131563 Files: clang/lib/Driver/ToolChains/Linux.cpp Index:

[PATCH] D131657: Remove redundant condition check, NFC

junaire created this revision. Herald added a project: All. junaire requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Signed-off-by: Jun Zhang Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D131657 Files:

[PATCH] D131448: Introduce iterator sentinel to make graph traversal implementation more efficient and cleaner

rusyaev-roman added a comment. First of all, thank you for your feedback! I've tried to address all your comments. Comment at: llvm/include/llvm/ADT/BreadthFirstIterator.h:128-131 + bool operator==(iterator_sentinel) const { return VisitQueue.empty(); } + + bool

[PATCH] D131065: [clang][dataflow] Store DeclContext of block being analysed in Environment if available.

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG2cb51449f0d9: [clang][dataflow] Store DeclContext of block being analysed in Environment if… (authored by wyt). Repository: rG LLVM Github

[PATCH] D131134: [X86] Report error if the amx enabled on the non-64-bits target

pengfei added a comment. In D131134#3715079 , @craig.topper wrote: > In D131134#3715062 , @pengfei wrote: > >> In D131134#3715024 , @pengfei >> wrote: >> >>> In

[clang] 2cb5144 - [clang][dataflow] Store DeclContext of block being analysed in Environment if available.

Author: Wei Yi Tee Date: 2022-08-11T07:36:57Z New Revision: 2cb51449f0d9ed06de87b4a47b5074eb6eec2e23 URL: https://github.com/llvm/llvm-project/commit/2cb51449f0d9ed06de87b4a47b5074eb6eec2e23 DIFF: https://github.com/llvm/llvm-project/commit/2cb51449f0d9ed06de87b4a47b5074eb6eec2e23.diff LOG:

[PATCH] D123967: Disable update_cc_test_checks.py tests in stand-alone builds

h-vetinari added a comment. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D123967/new/ https://reviews.llvm.org/D123967 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D131528: [Clang] Restrict non fixed enum to a value outside the range of the enumeration values warning to context requiring a constant expression

mstorsjo added a comment. There are still some cases that were broken by D131307 , that aren't fixed by this patch. Building https://martin.st/temp/qt-enum.cpp with `clang -target i686-w64-mingw32 -c -std=c++17 qt-enum.cpp -Wno-ignored-attributes

[PATCH] D131065: [clang][dataflow] Store DeclContext of block being analysed in Environment if available.

wyt updated this revision to Diff 451754. wyt added a comment. Initialise DeclCtx field with nullptr. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D131065/new/ https://reviews.llvm.org/D131065 Files:

[PATCH] D131655: [analyzer] Nullability: Enable analysis of non-inlined nullable objc properties.

NoQ created this revision. NoQ added reviewers: t-rasmud, usama54321, ziqingluo-90, malavikasamak, xazax.hun. Herald added subscribers: steakhal, manas, ASDenysPetrov, martong, dkrupp, donat.nagy, Szelethus, mikhail.ramalho, a.sidorin, rnkovacs, szepet, baloghadamsoftware. Herald added a

[PATCH] D130058: [Clang] Diagnose ill-formed constant expression when setting a non fixed enum to a value outside the range of the enumeration values

glandium added a comment. Having looked around in the Firefox codebase for cases that now fail to build because of this, and looking at the clarification in DR2338, I wonder if enums in `extern "C"` sections should be treated as if they had an explicit type of `int` as if it were e.g. `enum

[PATCH] D131388: [docs] Add "C++20 Modules"

ChuanqiXu added inline comments. Comment at: clang/docs/CPlusPlus20Modules.rst:665 + # This is not allowed! + $ clang++ iostream.pcm -c -o iostream.o + dblaikie wrote: > could this use a strikethrough, perhaps? (not sure if you can strikethrough > inside a

[PATCH] D131388: [docs] Add "C++20 Modules"

ChuanqiXu updated this revision to Diff 451750. ChuanqiXu marked 2 inline comments as done. ChuanqiXu added a comment. Address comments: - Change `module file` to `BMI`. - Replace `rm -f` with `rm`. - Add a conclusion paragraph to the `How module speed up compilation` section. CHANGES SINCE

[clang] 5d89f90 - [clang] [HLSL] Fix GCC warnings about virtual methods that are hidden

Author: Martin Storsjö Date: 2022-08-11T10:15:13+03:00 New Revision: 5d89f9044392db3838348a3f44e2723e3436dd28 URL: https://github.com/llvm/llvm-project/commit/5d89f9044392db3838348a3f44e2723e3436dd28 DIFF:

[PATCH] D130969: [clang] [HLSL] Fix GCC warnings about virtual methods that are hidden

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG5d89f9044392: [clang] [HLSL] Fix GCC warnings about virtual methods that are hidden (authored by mstorsjo). Repository: rG LLVM Github Monorepo

[PATCH] D130894: [clang] Print more information about failed static assertions

xbolva00 added a comment. In D130894#3715124 , @mstorsjo wrote: > This broke building with GCC (also noted by buildbot mails): > > ../tools/clang/lib/Sema/SemaDeclCXX.cpp: In member function ‘void > clang::Sema::DiagnoseStaticAssertDetails(const

[PATCH] D130894: [clang] Print more information about failed static assertions

mstorsjo added a comment. This broke building with GCC (also noted by buildbot mails): ../tools/clang/lib/Sema/SemaDeclCXX.cpp: In member function ‘void clang::Sema::DiagnoseStaticAssertDetails(const clang::Expr*)’: ../tools/clang/lib/Sema/SemaDeclCXX.cpp:1:19: error: declaration of

[clang] ebe0674 - [clang] Try to fix builders

Author: Timm Bäder Date: 2022-08-11T09:07:44+02:00 New Revision: ebe0674acb8bb3404d0e2a6b689d5e3cd02bb0b6 URL: https://github.com/llvm/llvm-project/commit/ebe0674acb8bb3404d0e2a6b689d5e3cd02bb0b6 DIFF: https://github.com/llvm/llvm-project/commit/ebe0674acb8bb3404d0e2a6b689d5e3cd02bb0b6.diff

[clang-tools-extra] 15bf2aa - [clangd][test] Fix error message in SerializationTest.BinaryConversions

Author: Tom Praschan Date: 2022-08-11T08:50:23+02:00 New Revision: 15bf2aa44aa1ffa049782316273f84c2db682e50 URL: https://github.com/llvm/llvm-project/commit/15bf2aa44aa1ffa049782316273f84c2db682e50 DIFF: https://github.com/llvm/llvm-project/commit/15bf2aa44aa1ffa049782316273f84c2db682e50.diff

[PATCH] D130894: [clang] Print more information about failed static assertions

This revision was automatically updated to reflect the committed changes. Closed by commit rG09117b21890c: [clang][sema] Print more information about failed static assertions (authored by tbaeder). Changed prior to commit: https://reviews.llvm.org/D130894?vs=451143=451744#toc Repository: rG

[clang] 09117b2 - [clang][sema] Print more information about failed static assertions

Author: Timm Bäder Date: 2022-08-11T08:44:38+02:00 New Revision: 09117b21890c652994f7ada0229d309b35b44259 URL: https://github.com/llvm/llvm-project/commit/09117b21890c652994f7ada0229d309b35b44259 DIFF: https://github.com/llvm/llvm-project/commit/09117b21890c652994f7ada0229d309b35b44259.diff

[PATCH] D131134: [X86] Report error if the amx enabled on the non-64-bits target

craig.topper added a comment. In D131134#3715062 , @pengfei wrote: > In D131134#3715024 , @pengfei wrote: > >> In D131134#3714860 , @craig.topper >> wrote: >> >>> Yes

[clang] bcc90f6 - [Clang] Update cxx_dr_status [NFC]

Author: Corentin Jabot Date: 2022-08-11T08:24:31+02:00 New Revision: bcc90f6268182a42205bd546be996fac6d05a071 URL: https://github.com/llvm/llvm-project/commit/bcc90f6268182a42205bd546be996fac6d05a071 DIFF:

[PATCH] D131651: [AST] [Modules] Introduce Decl::getNonTransparentDeclContext to handle exported friends

ChuanqiXu added inline comments. Comment at: clang/include/clang/AST/DeclBase.h:2020 /// Examples of transparent contexts include: enumerations (except for - /// C++0x scoped enums), and C++ linkage specifications. + /// C++0x scoped enums), C++ linkage specifications and

[PATCH] D131651: [AST] [Modules] Introduce Decl::getNonTransparentDeclContext to handle exported friends

ChuanqiXu added inline comments. Comment at: clang/lib/Sema/SemaTemplateInstantiateDecl.cpp:6152 +// of `getDeclContext()` once we find the invalid test case. if (FD->getFriendObjectKind() && FD->getDeclContext()->isFileContext()){ DC =

[PATCH] D131134: [X86] Report error if the amx enabled on the non-64-bits target

pengfei added a comment. In D131134#3715024 , @pengfei wrote: > In D131134#3714860 , @craig.topper > wrote: > >> Yes AMX is only supported in 64-bit mode but I doubt CPUID checks which mode >> the program is

[PATCH] D131651: [AST] [Modules] Introduce Decl::getNonTransparentDeclContext to handle exported friends

ChuanqiXu created this revision. ChuanqiXu added reviewers: aaron.ballman, erichkeane, ilya-biryukov, iains. ChuanqiXu added a project: clang. Herald added a project: All. ChuanqiXu requested review of this revision. Herald added a subscriber: cfe-commits. Closing

[PATCH] D128807: [clang][transformer] Finish plumbing `Note` all the way to the output.

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG156c0754bc2e: [clang][transformer] Finish plumbing `Note` all the way to the output. (authored by courbet). Repository: rG LLVM Github Monorepo

[clang] 156c075 - [clang][transformer] Finish plumbing `Note` all the way to the output.

Author: Clement Courbet Date: 2022-08-11T07:54:44+02:00 New Revision: 156c0754bc2ee0b7d198cfeff230374a1961ac04 URL: https://github.com/llvm/llvm-project/commit/156c0754bc2ee0b7d198cfeff230374a1961ac04 DIFF:

[PATCH] D131479: Handle explicitly defaulted consteval special members.

usaxena95 added inline comments. Comment at: clang/test/SemaCXX/cxx2a-consteval.cpp:805 +void func() { + default_ctor fail0; // expected-error-re {{call to consteval function '{{.*::default_ctor<.*::foo>}}::default_ctor' is not a constant expression}} \ +

[PATCH] D131134: [X86] Report error if the amx enabled on the non-64-bits target

pengfei added a comment. In D131134#3714860 , @craig.topper wrote: > Yes AMX is only supported in 64-bit mode but I doubt CPUID checks which mode > the program is running in. I guess `HasAMXSave` is `false` on 32-bit mode

<    1   2   3