[PATCH] D128352: [clang][dataflow] Use diagnosis API in optional checker

samestep updated this revision to Diff 440702. samestep added a comment. Herald added a project: clang. - Merge branch 'diagnose-api' into optional-check-diagnose - Use the updated API Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128352/new/ https

[PATCH] D128745: [Sema] fix trailing parameter pack handling for function template partial ordering

ychen updated this revision to Diff 440703. ychen added a comment. fix typo Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128745/new/ https://reviews.llvm.org/D128745 Files: clang/lib/Sema/SemaTemplateDeduction.cpp clang/test/CXX/drs/dr6xx.cpp

[PATCH] D128747: ISSUE - incorrect -Winfinite-recursion warning on potentially-unevaluated operand #21668

appmonster007 created this revision. appmonster007 added a reviewer: aaron.ballman. appmonster007 added a project: clang. Herald added a project: All. appmonster007 requested review of this revision. Herald added a subscriber: cfe-commits. Fixing issue "incorrect -Winfinite-recursion warning on po

[PATCH] D128352: [clang][dataflow] Use diagnosis API in optional checker

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

[PATCH] D123952: [FPEnv] Allow CompoundStmt to keep FP options

rjmccall added a comment. Looks good, thanks! Approved with the rename as discussed. Comment at: clang/include/clang/Basic/LangOptions.h:747 + /// Return difference with the given option set. + FPOptionsOverride diffWith(const FPOptions &Base); + sepavloff w

[PATCH] D128288: [PowerPC] Fix signatures for vec_replace_unaligned builtin

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

[PATCH] D128288: [PowerPC] Fix signatures for vec_replace_unaligned builtin

amyk accepted this revision. amyk added a comment. LGTM too. Thank you Lei. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128288/new/ https://reviews.llvm.org/D128288 ___ cfe-commits mailing list cfe-com

[PATCH] D127898: [clang][dataflow] Add API to separate analysis from diagnosis

xazax.hun accepted this revision. xazax.hun added a comment. Thanks, LGTM! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D127898/new/ https://reviews.llvm.org/D127898 ___ cfe-commits mailing list cfe-comm

[PATCH] D126907: Deferred Concept Instantiation Implementation Take 2

erichkeane updated this revision to Diff 440712. erichkeane added a comment. Rebased! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D126907/new/ https://reviews.llvm.org/D126907 Files: clang/docs/ReleaseNotes.rst clang/include/clang/AST/Decl.h clang/include/clang/AST/DeclBase.h

[PATCH] D128604: [RISCV] Support Zbpbo extension

sunshaoce updated this revision to Diff 440714. sunshaoce marked an inline comment as done. sunshaoce added a comment. Address @craig.topper's comment. Thanks! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128604/new/ https://reviews.llvm.org/D1286

[PATCH] D128750: [c++20] Implement P2113R0: Changes to the Partial Ordering of Constrained Functions

ychen created this revision. ychen added reviewers: hubert.reinterpretcast, erichkeane, rsmith, aaron.ballman, royjacobson. Herald added a project: All. ychen requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Most of the wordings are impleme

[PATCH] D128750: [c++20] Implement P2113R0: Changes to the Partial Ordering of Constrained Functions

erichkeane added a comment. Is there any chance you can validate this against https://reviews.llvm.org/D126907 as well? We touch similar code, and I'm intending to get that committed in the near future. Otherwise, after a quick look, I don't see anything particualrly bad, other than a lack of

[clang-tools-extra] a0f4c10 - [pseudo] Add error-recovery framework & brace-based recovery

Author: Sam McCall Date: 2022-06-28T21:08:43+02:00 New Revision: a0f4c10ae227a62c2a63611e64eba83f0ff0f577 URL: https://github.com/llvm/llvm-project/commit/a0f4c10ae227a62c2a63611e64eba83f0ff0f577 DIFF: https://github.com/llvm/llvm-project/commit/a0f4c10ae227a62c2a63611e64eba83f0ff0f577.diff LO

[clang-tools-extra] d25361c - [pseudo] Move ellipsis into initializer-list-item. NFC

Author: Sam McCall Date: 2022-06-28T21:08:43+02:00 New Revision: d25361c3afa6c3774c4f814a1142ceee9085db9e URL: https://github.com/llvm/llvm-project/commit/d25361c3afa6c3774c4f814a1142ceee9085db9e DIFF: https://github.com/llvm/llvm-project/commit/d25361c3afa6c3774c4f814a1142ceee9085db9e.diff LO

[PATCH] D128486: [pseudo] Add error-recovery framework & brace-based recovery

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 rGa0f4c10ae227: [pseudo] Add error-recovery framework & brace-based

[PATCH] D128486: [pseudo] Add error-recovery framework & brace-based recovery

sammccall reopened this revision. sammccall added a comment. Sorry, I committed this by mistake when working on another change. Reverted and this is ready for review. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128486/new/ https://reviews.llvm.or

[clang-tools-extra] 743971f - Revert "[pseudo] Add error-recovery framework & brace-based recovery"

Author: Sam McCall Date: 2022-06-28T21:11:09+02:00 New Revision: 743971faaf84eaa0e6310bdc22c68f34c20330f1 URL: https://github.com/llvm/llvm-project/commit/743971faaf84eaa0e6310bdc22c68f34c20330f1 DIFF: https://github.com/llvm/llvm-project/commit/743971faaf84eaa0e6310bdc22c68f34c20330f1.diff LO

[PATCH] D128750: [c++20] Implement P2113R0: Changes to the Partial Ordering of Constrained Functions

ychen added a comment. In D128750#3616449 , @erichkeane wrote: > Is there any chance you can validate this against > https://reviews.llvm.org/D126907 as well? We touch similar code, and I'm > intending to get that committed in the near future. Otherwi

[PATCH] D128752: [CUDA] Stop adding CUDA features twice

jhuber6 created this revision. jhuber6 added reviewers: jdoerfert, tra, yaxunl. Herald added subscribers: mattd, carlosgalvezp. Herald added a project: All. jhuber6 requested review of this revision. Herald added subscribers: cfe-commits, MaskRay. Herald added a project: clang. We currently call t

[PATCH] D125624: [gold] Remove an external dependency to GNU binutils' header file

tstellar added a comment. In D125624#3614298 , @ruiu wrote: > In D125624#3552770 , @tstellar > wrote: > >> In D125624#3552463 , @ruiu wrote: >> OK, as I mentioned. I

[PATCH] D126864: [clang] Introduce -fstrict-flex-arrays= for stricter handling of flexible arrays

serge-sans-paille added a comment. GCC and Clang don't have the same behavior wrt. macros-as-abound and standard-layout-requirement, see https://godbolt.org/z/3vc4TcTYz I'm fine with keeping the CLang behavior, but do we want to keep it only for level=0, and drop it for higher level (this would

[PATCH] D128752: [CUDA] Stop adding CUDA features twice

tra added a comment. > we no longer will have a cached CUDA installation so we will usually create > it twice. Does that result in extra output in case we find an unexpected CUDA version, or when compiler is run with `-v` ? We may want to wrap installation detector creation into some sort of s

[PATCH] D128182: [NFC] Switch FloatModeKind enum class to use bitmask enums

aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. LGTM again, this time for real. Comment at: clang/include/clang/Basic/TargetInfo.h:225-226 unsigned HasAlignMac68kSupport : 1; - unsigned RealTypeUsesObjCFP

[PATCH] D128752: [CUDA] Stop adding CUDA features twice

jhuber6 added a comment. In D128752#3616553 , @tra wrote: >> we no longer will have a cached CUDA installation so we will usually create >> it twice. > > Does that result in extra output in case we find an unexpected CUDA version, > or when compiler is

[PATCH] D127812: [AArch64] Function multiversioning support added.

danielkiss added a comment. In D127812#3602688 , @aaron.ballman wrote: > In D127812#3602645 , @erichkeane > wrote: > >> In D127812#3601476 , @danielkiss >> wrote: >> >>

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

tstellar added a comment. ping. 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 https://lists.llvm.org/c

[PATCH] D128754: Refactor LLVM compression namespaces

ckissane created this revision. ckissane added a reviewer: MaskRay. ckissane added a project: LLVM. Herald added subscribers: StephenFan, wenlei, usaxena95, kadircet, arphaman, hiraditya, arichardson, mgorny, emaste. Herald added a reviewer: alexander-shaposhnikov. Herald added a reviewer: rupprec

[PATCH] D128754: Refactor LLVM compression namespaces

ckissane updated this revision to Diff 440736. ckissane added a comment. up release notes CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128754/new/ https://reviews.llvm.org/D128754 Files: clang-tools-extra/clangd/index/Serialization.cpp clang-tools-extra/clangd/unittests/Serializat

[PATCH] D128048: Add a new clang option "-ftime-trace-path"

MaskRay added inline comments. Comment at: clang/include/clang/Driver/Options.td:2819 "'per-pass-run': one report for each pass invocation">; def ftime_trace : Flag<["-"], "ftime-trace">, Group, HelpText<"Turn on time profiler. Generates JSON file based on output

[PATCH] D128750: [c++20] Implement P2113R0: Changes to the Partial Ordering of Constrained Functions

ychen added a comment. In D128750#3616471 , @ychen wrote: > In D128750#3616449 , @erichkeane > wrote: > >> Is there any chance you can validate this against >> https://reviews.llvm.org/D126907 as well? We touch

[PATCH] D128752: [CUDA] Stop adding CUDA features twice

tra added a comment. In D128752#3616579 , @jhuber6 wrote: > In D128752#3616553 , @tra wrote: > >>> we no longer will have a cached CUDA installation so we will usually create >>> it twice. >> >> Does that result

[PATCH] D128752: [CUDA] Stop adding CUDA features twice

jhuber6 added a comment. In D128752#3616675 , @tra wrote: > We already heard complaints that searching for CUDA installation in multiple > places does add a measurable delay when the search hits NFS-mounted > directories. > > Replacing uses of `CudaInst

[PATCH] D128372: [Clang-Tidy] Empty Check

abrahamcd updated this revision to Diff 440742. abrahamcd added a comment. Progress Update Hit a roadblock with determining unused return value, uploading current state for further work. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128372/new/ ht

[PATCH] D128757: [Driver][test] Add -fuse-ld= option tests for NetBSD

fcambus created this revision. fcambus added a reviewer: MaskRay. Herald added a subscriber: StephenFan. Herald added a project: All. fcambus requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo https://rev

[PATCH] D128372: [Clang-Tidy] Empty Check

abrahamcd planned changes to this revision. abrahamcd added a comment. Stuck on determining unused return value, uploaded current state for @denik and @cjdb to take a look. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128372/new/ https://reviews.

[clang] 870b6d2 - Improve handling of static assert messages.

Author: Corentin Jabot Date: 2022-06-28T22:26:00+02:00 New Revision: 870b6d21839707a3e4c40a29b526995f065a220f URL: https://github.com/llvm/llvm-project/commit/870b6d21839707a3e4c40a29b526995f065a220f DIFF: https://github.com/llvm/llvm-project/commit/870b6d21839707a3e4c40a29b526995f065a220f.diff

[PATCH] D108469: Improve handling of static assert messages.

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG870b6d218397: Improve handling of static assert messages. (authored by cor3ntin). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D128465: Zstandard as a second compression method to LLVM

ckissane updated this revision to Diff 440747. ckissane edited the summary of this revision. ckissane added a comment. make it the second stage CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128465/new/ https://reviews.llvm.org/D128465 Files: clang-tools-extra/clangd/CMakeLists.txt

[PATCH] D127898: [clang][dataflow] Add API to separate analysis from diagnosis

gribozavr2 accepted this revision. gribozavr2 added inline comments. Comment at: clang/include/clang/Analysis/FlowSensitive/Models/UncheckedOptionalAccessModel.h:78 +public: + UncheckedOptionalAccessDiagnosis( + ASTContext &AstContext, UncheckedOptionalAccessModelOptions O

[PATCH] D128352: [clang][dataflow] Use diagnosis API in optional checker

gribozavr2 accepted this revision. gribozavr2 added inline comments. Comment at: clang-tools-extra/clang-tidy/bugprone/UncheckedOptionalAccessCheck.cpp:67 +std::move(StmtDiagnostics.begin(), StmtDiagnostics.end(), + std::back_inserter(Diagnostics

[PATCH] D128757: [Driver][test] Add -fuse-ld= option tests for NetBSD

MaskRay added a comment. I am unsure it is useful to add a test for all OSxfuse_ld_value combinations, especially for an ELF OS like NetBSD which shares many characteristics with other ELF OSes. It will be overwhelming. The option may be better added in a netbsd specific file. Repository: rG

[PATCH] D122768: [Clang][C++20] Support capturing structured bindings in lambdas

shafik added a comment. I think I need to do another sweep of this PR but mostly minor comments. You should add tests for structured bindings binding to array and tuple-like types b/c the AST under the `BindingDecl` will be different for each of those cases and it would be worth verifying it wo

[PATCH] D113545: [C++20] [Module] Support reachable definition initially/partially

rsmith accepted this revision. rsmith added a comment. This revision is now accepted and ready to land. A few comments, but they're all minor things or FIXMEs. I'm happy for this to land once they're addressed. Comment at: clang/include/clang/AST/DeclBase.h:229 +/// This d

[PATCH] D128752: [CUDA] Stop adding CUDA features twice

tra accepted this revision. tra added a comment. This revision is now accepted and ready to land. Do we have tests that verify `-target-feature` arguments? It may be worth adding a test case there checking for redundant features. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D128752: [CUDA] Stop adding CUDA features twice

jhuber6 added a comment. In D128752#3616831 , @tra wrote: > Do we have tests that verify `-target-feature` arguments? It may be worth > adding a test case there checking for redundant features. Yeah, we have some existing tests that check for including

[PATCH] D128402: [clang-tidy] Don't treat invalid branches as identical

ishaangandhi added a comment. Does anybody on this thread have land permissions? If not, would anyone know who to tag? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128402/new/ https://reviews.llvm.org/D128402 ___ cfe-commits mailing list cf

[PATCH] D128679: [pseudo] Define a clangPseudoCLI library.

hokein updated this revision to Diff 440765. hokein marked an inline comment as done. hokein added a comment. address comments Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128679/new/ https://reviews.llvm.org/D128679 Files: clang-tools-extra/ps

[PATCH] D128103: Adds AST Matcher for ObjCStringLiteral

t-rasmud updated this revision to Diff 440766. t-rasmud added a comment. Removes manually added documentation and adds it in the header file. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128103/new/ https://reviews.llvm.org/D128103 Files: clang/include/clang/ASTMatchers/ASTMatchers.

[PATCH] D128679: [pseudo] Define a clangPseudoCLI library.

hokein added inline comments. Comment at: clang-tools-extra/pseudo/include/clang-pseudo/GLR.h:115 // Parameters for the GLR parsing. +// FIXME: refine it with the ParseLang struct. struct ParseParams { sammccall wrote: > You're already touching the callsites of

[PATCH] D126857: [HLSL] Add WaveActiveCountBits as Langugage builtin function for HLSL

beanz added inline comments. Comment at: clang/include/clang/Basic/Builtins.def:1697 +// HLSL +LANGBUILTIN(WaveActiveCountBits, "Uib", "nc", HLSL_LANG) + beanz wrote: > python3kgae wrote: > > Anastasia wrote: > > > python3kgae wrote: > > > > Anastasia wrote: > >

[PATCH] D124624: [OpenMP] Add variant extension that applies to declarations

jdoerfert accepted this revision. jdoerfert added a comment. This revision is now accepted and ready to land. LG. one nit. Comment at: clang/include/clang/Basic/AttrDocs.td:4312 declarations with the same name. The template parameters for the base functions -are used to instan

[PATCH] D128333: [clang][flang] Disable defaulting to `-fpie` for LLVM Flang

MaskRay accepted this revision. MaskRay added inline comments. This revision is now accepted and ready to land. Comment at: flang/test/Driver/no-pie.f90:3 + +!- +! RUN COMMANDS awarzynski wrote: > MaskRay wrote: > > The `! RUN COMMANDS` and `EXPECTED

[PATCH] D128747: ISSUE - incorrect -Winfinite-recursion warning on potentially-unevaluated operand #21668

shafik added a comment. Thank for looking at this bug, just a small comment on the test. Comment at: clang/test/SemaCXX/warn-infinite-recursion.cpp:178 +struct Q { + virtual ~Q(){}; +}; Looks like your missing `// expected-warning{{call itself}}` Repository

[PATCH] D128653: [PowerPC] Fix the check for scalar MASS conversion

bmahjour added inline comments. Comment at: clang/test/CodeGen/lower-mass-end-to-end.c:33 +// CHECK-MASS-AFN: __xl_sin +// CHECK-NO-MASS-FAST-NOT: __xl_sin{{_finite}} +// CHECK-NO-MASS-AFN-NOT: __xl_sin{{_finite}} I don't think this works the way you expect it:

[PATCH] D114728: [Coroutine] Remove the prologue data of `-fsanitize=function` for split functions

ychen abandoned this revision. ychen added a comment. Herald added a project: All. D115844 and D116130 supersede this. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D114728/new/

[PATCH] D128752: [CUDA] Stop adding CUDA features twice

tra added a comment. In D128752#3616837 , @jhuber6 wrote: > In D128752#3616831 , @tra wrote: > >> Do we have tests that verify `-target-feature` arguments? It may be worth >> adding a test case there checking for

[PATCH] D108469: Improve handling of static assert messages.

leonardchan added a comment. I think this might be the cause of the many libc++ static_assert failures we're seeing on our builders: https://luci-milo.appspot.com/ui/p/fuchsia/builders/toolchain.ci/clang-linux-x64/b8810079977358640657/overview Repository: rG LLVM Github Monorepo CHANGES SIN

[PATCH] D128569: Start support for HLSL `RWBuffer`

beanz added inline comments. Comment at: clang/include/clang/Basic/Builtins.def:1703 LANGBUILTIN(WaveActiveCountBits, "Uib", "nc", HLSL_LANG) +LANGBUILTIN(__builtin_hlsl_get_resource_pointer, "v*", "i", HLSL_LANG) +LANGBUILTIN(__builtin_hlsl_get_resource_status, "Ui", "i", HLSL_

[PATCH] D128569: Start support for HLSL `RWBuffer`

beanz updated this revision to Diff 440778. beanz added a comment. Updating based on PR feedback and removing the new builtins. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128569/new/ https://reviews.llvm.org/D128569 Files: clang/include/clang

[PATCH] D128762: [Clang] Rename StringLiteral::isAscii() => isOrdinary() [NFC]

cor3ntin created this revision. Herald added a reviewer: aaron.ballman. Herald added a project: All. cor3ntin requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. "Ascii" StringLiteral instances are actually narrow strings that are UTF-8 encoded

[clang] a774ba7 - Revert "Improve handling of static assert messages."

Author: Corentin Jabot Date: 2022-06-29T00:03:23+02:00 New Revision: a774ba7f60d1fef403b5507b1b1a7475d3684d71 URL: https://github.com/llvm/llvm-project/commit/a774ba7f60d1fef403b5507b1b1a7475d3684d71 DIFF: https://github.com/llvm/llvm-project/commit/a774ba7f60d1fef403b5507b1b1a7475d3684d71.diff

[PATCH] D108469: Improve handling of static assert messages.

cor3ntin added a comment. In D108469#3616983 , @leonardchan wrote: > I think this might be the cause of the many libc++ static_assert failures > we're seeing on our builders: > https://luci-milo.appspot.com/ui/p/fuchsia/builders/toolchain.ci/clang-linu

[PATCH] D128766: Update references to Discourse instead of the mailing lists.

tonic created this revision. Herald added a project: All. tonic requested review of this revision. Herald added a reviewer: jdoerfert. Herald added subscribers: cfe-commits, sstefan1. Herald added a project: clang. Update the references to the old Mailman mailing lists to point to Discourse forum

[PATCH] D128750: [c++20] Implement P2113R0: Changes to the Partial Ordering of Constrained Functions

ychen updated this revision to Diff 440789. ychen added a comment. - add release notes - update www-status Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128750/new/ https://reviews.llvm.org/D128750 Files: clang/docs/ReleaseNotes.rst clang/incl

[PATCH] D128685: [OpenMP][NFC] Remove unused check lines in Clang/OpenMP tests

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG178674e23a71: [OpenMP][NFC] Remove unused check lines in Clang/OpenMP tests (authored by jdoerfert). Repository: rG LLVM Github Monorepo CHANGES

[PATCH] D128103: Adds AST Matcher for ObjCStringLiteral

NoQ added a comment. Yes the use case is coming up in D126097 ; it's not used yet because `hasDescendant(StringLiteral)` is used to skip it but this matchert would be a more precise solution (yes there's an actual `StringLiteral` inside `ObjCStringLiteral`).

[PATCH] D128329: [clangd] Also mark output arguments of operator call expressions

nridge accepted this revision. nridge added a comment. This revision is now accepted and ready to land. Thanks! I'm going to take the liberty of approving this, as it seems straightforward and unlikely to be contentious in any way. Please let me know if you need me to commit it. Repository:

[PATCH] D124690: [clangd] add inlay hints for std::forward-ed parameter packs

nridge added inline comments. Comment at: clang-tools-extra/clangd/InlayHints.cpp:483 + !Type.getNonReferenceType().isConstQualified() && + !isExpandedParameterPack(Param); } sammccall wrote: > why is this check needed if we already decline

[clang] 57fa688 - [Driver][ARM][AArch64] Use err_drv_unsupported_option_argument for -march=/-mcpu=/-mtune= diagnostics

Author: Fangrui Song Date: 2022-06-28T16:01:30-07:00 New Revision: 57fa68897bb76e1fe5442784bca4e0629033c384 URL: https://github.com/llvm/llvm-project/commit/57fa68897bb76e1fe5442784bca4e0629033c384 DIFF: https://github.com/llvm/llvm-project/commit/57fa68897bb76e1fe5442784bca4e0629033c384.diff

[PATCH] D108469: Improve handling of static assert messages.

cor3ntin updated this revision to Diff 440804. cor3ntin added a comment. - Undo the change of diagnostic message formatting as they break libc++ tests so we will keep printing instead Changing the format should be done -if at all- in a separate change. Repository: rG LLVM Github Monorepo

[PATCH] D128754: Refactor LLVM compression namespaces

leonardchan added a comment. > Feedback needed on: > this namespace approach will result in tooling compression methods being set > at compile time, however we may want to aim for a runtime configurable > approach in the future, likely a new revision of the compressed data formats > that llvm i

[PATCH] D128402: [clang-tidy] Don't treat invalid branches as identical

njames93 added inline comments. Comment at: clang-tools-extra/test/clang-tidy/checkers/bugprone/branch-clone-unknown-expr.cpp:1 +// RUN: %check_clang_tidy -fix-errors %s bugprone-branch-clone %t + is -fix-errors necessary here, given we aren't verifying any fixi

[PATCH] D128772: [Lex] Make sure to notify `MultipleIncludeOpt` for "read tokens" during fast dependency directive lexing

akyrtzi created this revision. Herald added a project: All. akyrtzi requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Otherwise a header may be erroneously marked as having a header macro guard and won't get re-included. Repository: rG L

[PATCH] D128774: [libTooling] Add a comment about comment parsing to getAssociatedRange.

jacobsa created this revision. jacobsa added a reviewer: klimek. Herald added a project: All. jacobsa requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. It took me multiple hours of debugging plus asking an expert for help to figure out why th

[PATCH] D128645: Update developer policy.

hubert.reinterpretcast added inline comments. Comment at: llvm/docs/DeveloperPolicy.rst:88 +#. Patches should be unified diffs with "infinite context" (i.e. using something + like `git diff -U99 main`). + Using `git diff` like this, there are risks that `m

[PATCH] D128645: Update developer policy.

hubert.reinterpretcast added inline comments. Comment at: llvm/docs/DeveloperPolicy.rst:403-405 +Note, these mailing lists are moderated and it is not unusual for a large +commit to require a moderator to approve the email, so do not be concerned if a +commit does not immediately

[PATCH] D128747: ISSUE - incorrect -Winfinite-recursion warning on potentially-unevaluated operand #21668

appmonster007 updated this revision to Diff 440819. appmonster007 added a comment. fixed warning comments for test cases Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128747/new/ https://reviews.llvm.org/D128747 Files: clang/lib/Analysis/CFG.cpp

[PATCH] D128777: clang/cmake: Drop use of llvm-config for LLVM install discovery

tstellar created this revision. Herald added a subscriber: mgorny. Herald added a project: All. tstellar requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. This has been deprecated for a while, remove it in favor of using cmake's find_package(

[PATCH] D128550: [OpenMP] Change OpenMP code generation for target region entries

jhuber6 updated this revision to Diff 440822. jhuber6 added a comment. Herald added subscribers: mattd, asavonic. Fix tests, lots of lines. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128550/new/ https://reviews.llvm.org/D128550 Files: clang/l

[PATCH] D128487: [ODRHash diagnostics] Move repetetive code at lambda calls into lambdas themselves. NFC.

vsapsai added a comment. Thanks for the review! In D128487#3614251 , @ChuanqiXu wrote: > Is it possible to combine the several `DiagNote` into `DiagError`? So that > the code would be further reduced. I am OK to do this kind of change in other > revisi

[PATCH] D105584: [MLIR][OpenMP] Distribute Construct Operation

abidmalikwaterloo added a comment. // CHECK-LABEL: omp_DistributeOp func.func @omp_DistributeOp(%lb : index, %ub : index, %step : index, %data_var : memref, %chunk_var : i32) -> () { // CHECK: omp.wsloop collapse(2) "omp.DistributeOp" (%lb, %ub, %step) ({ ^bb0(%iv: index):

[PATCH] D128754: Refactor LLVM compression namespaces

ckissane marked an inline comment as done. ckissane added inline comments. Comment at: llvm/unittests/Support/CompressionTest.cpp:68 - 0x414FA339U, - zlib::crc32(StringRef("The quick brown fox jumps over the lazy dog"))); -} leonardchan wrote: > Should

[PATCH] D128754: Refactor LLVM compression namespaces

ckissane marked an inline comment as done. ckissane added inline comments. Comment at: clang-tools-extra/clangd/index/Serialization.cpp:242 +return error( +"Compressed string table, but compression::serialize is unavailable"); leonardchan wrote: > n

[PATCH] D127189: [clang][AIX] Add option to control quadword lock free atomics ABI on AIX

amyk added a comment. This patch makes sense. I had some questions regarding it. Comment at: clang/include/clang/Driver/Options.td:3611 HelpText<"Enable the default Altivec ABI on AIX (AIX only). Uses only volatile vector registers.">; +def maix_quadword_atomics : Flag<["-"

[PATCH] D127189: [clang][AIX] Add option to control quadword lock free atomics ABI on AIX

amyk added inline comments. Comment at: clang/test/Driver/ppc-unsupported.c:12 +// RUN: -c %s 2>&1 | FileCheck %s +// RUN: not %clang -target powerpc64le-unknown-linux -maix64-quadword-atomics \ +// RUN: -c %s 2>&1 | FileCheck %s amyk wrote: > Should we have

[PATCH] D128550: [OpenMP] Change OpenMP code generation for target region entries

jhuber6 updated this revision to Diff 440831. jhuber6 added a comment. Format Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128550/new/ https://reviews.llvm.org/D128550 Files: clang/lib/CodeGen/CGOpenMPRuntime.cpp clang/test/OpenMP/capturing_i

[PATCH] D128747: ISSUE - incorrect -Winfinite-recursion warning on potentially-unevaluated operand #21668

appmonster007 updated this revision to Diff 440835. appmonster007 added a comment. included typeinfo file, in clang/test/SemaCXX directory Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128747/new/ https://reviews.llvm.org/D128747 Files: clang/li

[PATCH] D127189: [clang][AIX] Add option to control quadword lock free atomics ABI on AIX

lkail added inline comments. Comment at: clang/test/Driver/ppc-unsupported.c:12 +// RUN: -c %s 2>&1 | FileCheck %s +// RUN: not %clang -target powerpc64le-unknown-linux -maix64-quadword-atomics \ +// RUN: -c %s 2>&1 | FileCheck %s amyk wrote: > amyk wrote: >

[PATCH] D128747: ISSUE - incorrect -Winfinite-recursion warning on potentially-unevaluated operand #21668

appmonster007 updated this revision to Diff 440845. appmonster007 added a comment. #include "typeinfo" Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128747/new/ https://reviews.llvm.org/D128747 Files: clang/lib/Analysis/CFG.cpp clang/test/Sema

[PATCH] D128747: ISSUE - incorrect -Winfinite-recursion warning on potentially-unevaluated operand #21668

appmonster007 added inline comments. Comment at: clang/test/SemaCXX/warn-infinite-recursion.cpp:3 +#include "typeinfo" + `#include ` resulted in 'typeinfo' file not found with include; use "quotes" instead Repository: rG LLVM Github Monorepo CHANGES SINC

[PATCH] D113545: [C++20] [Module] Support reachable definition initially/partially

ChuanqiXu updated this revision to Diff 440847. ChuanqiXu marked 15 inline comments as done. ChuanqiXu added a comment. Address comments. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D113545/new/ https://reviews.llvm.org/D113545 Files: clang/include/clang/AST/DeclBase.h clang/inclu

[PATCH] D113545: [C++20] [Module] Support reachable definition initially/partially

ChuanqiXu added a comment. In D113545#3616814 , @rsmith wrote: > A few comments, but they're all minor things or FIXMEs. I'm happy for this to > land once they're addressed. Thanks for reviewing! Comment at: clang/lib/Sema/SemaLookup

[PATCH] D124750: [MLIR] Add a utility to sort the operands of commutative ops

srishti-pm updated this revision to Diff 440848. srishti-pm marked 2 inline comments as done. srishti-pm added a comment. Fixed memory leak. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D124750/new/ https://reviews.llvm.org/D124750 Files: mlir/i

[PATCH] D124750: [MLIR] Add a utility to sort the operands of commutative ops

Mogball added inline comments. Comment at: mlir/lib/Transforms/Utils/CommutativityUtils.cpp:178 +using UniquePtrOperandBFS = std::unique_ptr; +using ArrayRefOperandBFS = ArrayRef; + Please remove these. They don't improve readability. Repository: rG LLVM Gith

[clang] a223ba0 - [C++20] [Modules] Implement Reachable initiallly

Author: Chuanqi Xu Date: 2022-06-29T12:32:31+08:00 New Revision: a223ba0a697c1598b434cf2495c9cd9ec5640fc7 URL: https://github.com/llvm/llvm-project/commit/a223ba0a697c1598b434cf2495c9cd9ec5640fc7 DIFF: https://github.com/llvm/llvm-project/commit/a223ba0a697c1598b434cf2495c9cd9ec5640fc7.diff LO

[clang] 7a54140 - Revert "[C++20] [Modules] Implement Reachable initiallly"

Author: Chuanqi Xu Date: 2022-06-29T12:43:26+08:00 New Revision: 7a541406b5a23a811a4f37432292a6de3307b0f1 URL: https://github.com/llvm/llvm-project/commit/7a541406b5a23a811a4f37432292a6de3307b0f1 DIFF: https://github.com/llvm/llvm-project/commit/7a541406b5a23a811a4f37432292a6de3307b0f1.diff LO

[clang] 9c04851 - [C++20] [Module] Support reachable definition initially/partially

Author: Chuanqi Xu Date: 2022-06-29T12:48:48+08:00 New Revision: 9c04851cf5809c80862183481f8ced0b3e9ee301 URL: https://github.com/llvm/llvm-project/commit/9c04851cf5809c80862183481f8ced0b3e9ee301 DIFF: https://github.com/llvm/llvm-project/commit/9c04851cf5809c80862183481f8ced0b3e9ee301.diff LO

[PATCH] D113545: [C++20] [Module] Support reachable definition initially/partially

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG9c04851cf580: [C++20] [Module] Support reachable definition initially/partially (authored by ChuanqiXu). Repository: rG LLVM Github Monorepo CHAN

[PATCH] D128248: [clang] Avoid an assertion in APValue::hasArrayFiller()

tbaeder added a comment. If you apply e.g. @@ -10739,6 +10779,7 @@ bool ArrayExprEvaluator::VisitInitListExpr(const InitListExpr *E, << NumEltsToInit << ".\n"); Result = APValue(APValue::UninitArray(), NumEltsToInit, NumElts); + llvm::errs() << "Result "

[PATCH] D125291: Introduce @llvm.threadlocal.address intrinsic to access TLS variable

ChuanqiXu added a comment. @jyknight ping! Or someone else is willing to review this one? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D125291/new/ https://reviews.llvm.org/D125291 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http

<    1   2   3   >