[clang] [clang] Correct FixIt ranges for unused capture warnings (PR #141148)

2025-05-23 Thread via cfe-commits
@@ -2164,15 +2164,29 @@ ExprResult Sema::BuildLambdaExpr(SourceLocation StartLoc, SourceLocation EndLoc, bool IsLast = (I + 1) == LSI->NumExplicitCaptures; SourceRange FixItRange; if (CaptureRange.isValid()) { +auto GetTrailingEndLocat

[clang] [clang] Correct FixIt ranges for unused capture warnings (PR #141148)

2025-05-23 Thread via cfe-commits
@@ -84,6 +84,21 @@ SourceLocation Sema::getLocForEndOfToken(SourceLocation Loc, unsigned Offset) { return Lexer::getLocForEndOfToken(Loc, Offset, SourceMgr, LangOpts); } +SourceRange Sema::getRangeForNextToken(SourceLocation Loc, + bool

[clang] [clang] Correct FixIt ranges for unused capture warnings (PR #141148)

2025-05-23 Thread via cfe-commits
cor3ntin wrote: This change needs a release note. Please add an entry to `clang/docs/ReleaseNotes.rst` in the section the most adapted to the change, and referencing any Github issue this change fixes. Thanks! https://github.com/llvm/llvm-project/pull/141148 ___

[clang] [OpenCL] No need to check array of struct for kernel arguments (PR #138894)

2025-05-23 Thread via cfe-commits
github-actions[bot] wrote: @jiefwo Congratulations on having your first Pull Request (PR) merged into the LLVM Project! Your changes will be combined with recent changes from other authors, then tested by our [build bots](https://lab.llvm.org/buildbot/). If there is a problem with a build,

[clang] [OpenCL] No need to check array of struct for kernel arguments (PR #138894)

2025-05-23 Thread Sven van Haastregt via cfe-commits
https://github.com/svenvh closed https://github.com/llvm/llvm-project/pull/138894 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] 8f1d1dd - [KeyInstr][Clang] Assignment atom group (#134637)

2025-05-23 Thread via cfe-commits
Author: Orlando Cazalet-Hyams Date: 2025-05-23T09:40:09+01:00 New Revision: 8f1d1dde8a9b4eb317bd515fcb4838f97adbf5f2 URL: https://github.com/llvm/llvm-project/commit/8f1d1dde8a9b4eb317bd515fcb4838f97adbf5f2 DIFF: https://github.com/llvm/llvm-project/commit/8f1d1dde8a9b4eb317bd515fcb4838f97adbf5

[clang] [KeyInstr][Clang] Assignment atom group (PR #134637)

2025-05-23 Thread Orlando Cazalet-Hyams via cfe-commits
https://github.com/OCHyams closed https://github.com/llvm/llvm-project/pull/134637 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] Follow style configuration in clangd when inserting missing includes (PR #140594)

2025-05-23 Thread kadir çetinkaya via cfe-commits
kadircet wrote: just a high level question, I wasn't following the recent developments closely, but we seem to have both an `AngledHeaders` and `QuotedHeaders` option. Why are we only following one here? I guess the signals for angled includes are much stricter (search paths need to be marked

[clang] [clang] Correct FixIt ranges for unused capture warnings (PR #141148)

2025-05-23 Thread via cfe-commits
@@ -84,6 +84,21 @@ SourceLocation Sema::getLocForEndOfToken(SourceLocation Loc, unsigned Offset) { return Lexer::getLocForEndOfToken(Loc, Offset, SourceMgr, LangOpts); } +SourceRange Sema::getRangeForNextToken(SourceLocation Loc, + bool

[clang] [KeyInstr][Clang] Member initalization atom (PR #134640)

2025-05-23 Thread Orlando Cazalet-Hyams via cfe-commits
https://github.com/OCHyams updated https://github.com/llvm/llvm-project/pull/134640 >From 58615d2b93f4644bc7574d86b580d05edf1ca03c Mon Sep 17 00:00:00 2001 From: Orlando Cazalet-Hyams Date: Thu, 3 Apr 2025 17:00:09 +0100 Subject: [PATCH 1/2] [KeyInstr][Clang] Member initalization atom This pat

[clang] [KeyInstr][Clang] Catch variable init atom (PR #134641)

2025-05-23 Thread Orlando Cazalet-Hyams via cfe-commits
https://github.com/OCHyams edited https://github.com/llvm/llvm-project/pull/134641 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [Clang][LoongArch] Support target attribute for function (PR #140700)

2025-05-23 Thread via cfe-commits
https://github.com/Ami-zhang updated https://github.com/llvm/llvm-project/pull/140700 >From 3355685f4debcd901f27efccc346de0f302a796c Mon Sep 17 00:00:00 2001 From: Ami-zhang Date: Wed, 14 May 2025 15:21:54 +0800 Subject: [PATCH 1/2] [Clang][LoongArch] Support target attribute for function This

[clang] [OpenCL] No need to check array of struct for kernel arguments (PR #138894)

2025-05-23 Thread Sven van Haastregt via cfe-commits
svenvh wrote: Merged now, thanks for your contribution! https://github.com/llvm/llvm-project/pull/138894 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [FixIt] Improve Source Ranges and Fix-It Hints for Unused Lambda Captures #106445 (PR #117953)

2025-05-23 Thread via cfe-commits
charan-003 wrote: > @charan-003 Oliver told me you were happy with him taking over in #141148. > Can you confirm? I apologize that we incorrectly identified this bug as a > good first issue when it clearly wasn't. Were you able to find other issues > to work on? @cor3ntin yes, I confirm Olive

[clang] [KeyInstr][Clang] Do stmt atom (PR #134644)

2025-05-23 Thread Orlando Cazalet-Hyams via cfe-commits
https://github.com/OCHyams edited https://github.com/llvm/llvm-project/pull/134644 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [KeyInstr][Clang] Do stmt atom (PR #134644)

2025-05-23 Thread Orlando Cazalet-Hyams via cfe-commits
https://github.com/OCHyams updated https://github.com/llvm/llvm-project/pull/134644 >From 1219e5b01cd9cc38fe85d3377354c6e56e51134a Mon Sep 17 00:00:00 2001 From: Orlando Cazalet-Hyams Date: Thu, 3 Apr 2025 18:53:31 +0100 Subject: [PATCH 1/2] [KeyInstr][Clang] Do stmt atom See test comment for

[clang] 435a542 - [KeyInstr] Fix clang tests

2025-05-23 Thread Orlando Cazalet-Hyams via cfe-commits
Author: Orlando Cazalet-Hyams Date: 2025-05-23T11:19:30+01:00 New Revision: 435a542009d45d85fbd9a2ff0df735572d5cc276 URL: https://github.com/llvm/llvm-project/commit/435a542009d45d85fbd9a2ff0df735572d5cc276 DIFF: https://github.com/llvm/llvm-project/commit/435a542009d45d85fbd9a2ff0df735572d5cc2

[clang] [Modules] Don't fail when an unused textual header is missing. (PR #138227)

2025-05-23 Thread Ilya Biryukov via cfe-commits
ilya-biryukov wrote: > Ok, I'm going to revert the change to help you out. But I'm going to re-land > it in a week or when you are ready, whichever comes first. > > There was no indication there is anything wrong with the change or if the > issue is wide-spread. And if a single company relies o

[clang] [Clang] Fix the access checking for non-aggregates in default arguments (PR #141207)

2025-05-23 Thread via cfe-commits
@@ -282,4 +283,25 @@ static_assert(S().SizeOfT() == sizeof(short *), ""); } // namespace GH68490 +namespace GH83608 { + +class single; + +class check_constructible { + // This makes it a non-aggregate in C++20+. + check_constructible() = default; + + friend class single;

[clang-tools-extra] [clang-tidy] Disable bugprone-multi-level-pointer-conversion in C code (PR #141209)

2025-05-23 Thread Piotr Zegar via cfe-commits
https://github.com/PiotrZSL commented: I'm not sure about this. This check is about IMPLICIT casts, you can always do explicit cast in C (via functional cast or c-style cast) or C++ (reinterpret_cast) to say that mutli-level pointer conversion is expected in that place. If some C project got

[clang] [Modules] Don't fail when an unused textual header is missing. (PR #138227)

2025-05-23 Thread Ilya Biryukov via cfe-commits
ilya-biryukov wrote: Back to the original issue. > I'm willing to help figure out how to achieve the desired result in a > different way. But for that need to know what is the desired result. We would need to example I shared above to keep working. We rely on an optimization that picks a modu

[clang] 319feac - [clang][bytecode] Fix AccessKinds in placement new CheckStore() call (#141123)

2025-05-23 Thread via cfe-commits
Author: Timm Baeder Date: 2025-05-23T13:00:57+02:00 New Revision: 319feac43d8d1362307076ade85dc6d5bc4b4007 URL: https://github.com/llvm/llvm-project/commit/319feac43d8d1362307076ade85dc6d5bc4b4007 DIFF: https://github.com/llvm/llvm-project/commit/319feac43d8d1362307076ade85dc6d5bc4b4007.diff L

[clang] [openmp] [OpenMP 6.0 ]Codegen for Reduction over private variables with reduction clause (PR #134709)

2025-05-23 Thread Alexey Bataev via cfe-commits
@@ -5748,7 +5752,8 @@ void CodeGenFunction::EmitOMPScanDirective(const OMPScanDirective &S) { } CGM.getOpenMPRuntime().emitReduction( *this, ParentDir.getEndLoc(), Privates, LHSs, RHSs, ReductionOps, - {/*WithNowait=*/true, /*SimpleReduction=*/tr

[clang] [Clang][OpenMP][LoopTransformations] Fix incorrect number of generated loops for Tile and Reverse directives (PR #140532)

2025-05-23 Thread Alexey Bataev via cfe-commits
alexey-bataev wrote: > @alexey-bataev After conducting an examination of the directive handling > logic, I can confidently state that the number of generated loops > (`NumGeneratedLoops`) does not affect the semantic checks for the majority of > transformations. This is because values are usua

[clang] [openmp] [OpenMP 6.0 ]Codegen for Reduction over private variables with reduction clause (PR #134709)

2025-05-23 Thread Alexey Bataev via cfe-commits
@@ -3943,7 +3946,8 @@ static void emitScanBasedDirective( PrivScope.Privatize(); CGF.CGM.getOpenMPRuntime().emitReduction( CGF, S.getEndLoc(), Privates, LHSs, RHSs, ReductionOps, - {/*WithNowait=*/true, /*SimpleReduction=*/true, OMPD_unknown}); +

[clang] [Clang] Fix the access checking for non-aggregates in default arguments (PR #141207)

2025-05-23 Thread Younan Zhang via cfe-commits
https://github.com/zyn0217 edited https://github.com/llvm/llvm-project/pull/141207 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [KeyInstr][Clang] Catch variable init atom (PR #134641)

2025-05-23 Thread Orlando Cazalet-Hyams via cfe-commits
https://github.com/OCHyams updated https://github.com/llvm/llvm-project/pull/134641 >From 7c86cc1b0b0bfaba4c304a31b5b0f2a1f391ad63 Mon Sep 17 00:00:00 2001 From: Orlando Cazalet-Hyams Date: Thu, 3 Apr 2025 17:31:32 +0100 Subject: [PATCH 1/2] [KeyInstr][Clang] Catch variable init atom This patc

[clang] [KeyInstr][Clang] Catch variable init atom (PR #134641)

2025-05-23 Thread Orlando Cazalet-Hyams via cfe-commits
https://github.com/OCHyams converted_to_draft https://github.com/llvm/llvm-project/pull/134641 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Fix the access checking for non-aggregates in default arguments (PR #141207)

2025-05-23 Thread Younan Zhang via cfe-commits
@@ -282,4 +283,25 @@ static_assert(S().SizeOfT() == sizeof(short *), ""); } // namespace GH68490 +namespace GH83608 { + +class single; + +class check_constructible { + // This makes it a non-aggregate in C++20+. + check_constructible() = default; zyn0217 w

[clang] [Clang] Fix the access checking for non-aggregates in default arguments (PR #141207)

2025-05-23 Thread Younan Zhang via cfe-commits
zyn0217 wrote: > Does this fixes #62444 ? Yes 😎 https://github.com/llvm/llvm-project/pull/141207 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Fix the access checking for non-aggregates in default arguments (PR #141207)

2025-05-23 Thread Younan Zhang via cfe-commits
https://github.com/zyn0217 updated https://github.com/llvm/llvm-project/pull/141207 >From c63f3f82d6e4c576051532b2272abf4ac0055d7f Mon Sep 17 00:00:00 2001 From: Younan Zhang Date: Fri, 23 May 2025 15:29:49 +0800 Subject: [PATCH 1/2] [Clang] Fix the access checking for non-aggregates in defaul

[clang] [llvm] [clang][RISCV] Emit RISCV function-signature-based CFI label in llvm::Function metadata (PR #111661)

2025-05-23 Thread Ming-Yi Lai via cfe-commits
mylai-mtk wrote: @efriedma-quic Thanks for pointing out the potential type incompatibilities regarding the current mangling scheme. Regarding this issue, I believe there's currently no perfect solution given the complexity of C type compatibility. It looks like even in the AArch64 PtrAuth im

[clang] [clang][bytecode] Fix AccessKinds in placement new CheckStore() call (PR #141123)

2025-05-23 Thread Timm Baeder via cfe-commits
https://github.com/tbaederr closed https://github.com/llvm/llvm-project/pull/141123 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] replace the extension with the right functions (PR #141110)

2025-05-23 Thread Abhina Sree via cfe-commits
https://github.com/abhina-sree approved this pull request. LGTM https://github.com/llvm/llvm-project/pull/141110 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] d4f0f43 - [Clang] Fix the access checking for non-aggregates in default arguments (#141207)

2025-05-23 Thread via cfe-commits
Author: Younan Zhang Date: 2025-05-23T20:11:22+08:00 New Revision: d4f0f43506d04807e82e5c722c6362bcf112a2a0 URL: https://github.com/llvm/llvm-project/commit/d4f0f43506d04807e82e5c722c6362bcf112a2a0 DIFF: https://github.com/llvm/llvm-project/commit/d4f0f43506d04807e82e5c722c6362bcf112a2a0.diff

[clang] [Clang] Fix the access checking for non-aggregates in default arguments (PR #141207)

2025-05-23 Thread Younan Zhang via cfe-commits
https://github.com/zyn0217 closed https://github.com/llvm/llvm-project/pull/141207 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [KeyInstr][Clang] Catch variable init atom (PR #134641)

2025-05-23 Thread Orlando Cazalet-Hyams via cfe-commits
@@ -5055,6 +5055,7 @@ void ItaniumCXXABI::emitBeginCatch(CodeGenFunction &CGF, // Emit the local. CodeGenFunction::AutoVarEmission var = CGF.EmitAutoVarAlloca(*CatchParam); + ApplyAtomGroup Grp(CGF.getDebugInfo()); InitCatchParam(CGF, *CatchParam, var.getObjectAddress(

[clang] [KeyInstr][Clang] Catch variable init atom (PR #134641)

2025-05-23 Thread Orlando Cazalet-Hyams via cfe-commits
https://github.com/OCHyams ready_for_review https://github.com/llvm/llvm-project/pull/134641 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clang-tidy] Disable bugprone-multi-level-pointer-conversion in C code (PR #141209)

2025-05-23 Thread Carlos Galvez via cfe-commits
carlosgalvezp wrote: You can check the discussion in the linked issue. The argument is that these casts are idiomatic in C, and does not make sense to pollute the code with casts. There's no other safer alternative either. > then probably they should just disable a check There doesn't appear

[clang-tools-extra] [clang-tidy] Don't run use-nullptr check on objective-c code. (PR #141229)

2025-05-23 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang-tools-extra Author: Haojian Wu (hokein) Changes `nullptr` doesn't exist in objective-c. Right now, if an objective-c file compiled with `-std=c23` flag, it will trigger this check. This patch suppresses the check warning. --- Full diff: https:

[clang-tools-extra] [clang-tidy] Don't run use-nullptr check on objective-c code. (PR #141229)

2025-05-23 Thread Haojian Wu via cfe-commits
https://github.com/hokein updated https://github.com/llvm/llvm-project/pull/141229 >From 7d6ba7c08e260d75ad33c06f069898a528fe6159 Mon Sep 17 00:00:00 2001 From: Haojian Wu Date: Fri, 23 May 2025 14:16:52 +0200 Subject: [PATCH] [clang-tidy] Don't run use-nullptr check on objective-c code. If we

[clang-tools-extra] [clang-tidy] Don't run use-nullptr check on objective-c code. (PR #141229)

2025-05-23 Thread Haojian Wu via cfe-commits
https://github.com/hokein created https://github.com/llvm/llvm-project/pull/141229 `nullptr` doesn't exist in objective-c. Right now, if an objective-c file compiled with `-std=c23` flag, it will trigger this check. This patch suppresses the check warning. >From 07e1b67719f10d3fb31e2cdbe944

[clang-tools-extra] [clang-doc] fix names of conversions for template parameters (PR #140856)

2025-05-23 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `llvm-clang-x86_64-gcc-ubuntu` running on `sie-linux-worker3` while building `clang-tools-extra` at step 6 "test-build-unified-tree-check-all". Full details are available at: https://lab.llvm.org/buildbot/#/builders/174/builds

[clang] [llvm] [NFCI][Sanitizer] Convert Matcher::Globs from StringMap to vector. (PR #140964)

2025-05-23 Thread Vitaly Buka via cfe-commits
@@ -53,24 +53,27 @@ Error SpecialCaseList::Matcher::insert(StringRef Pattern, unsigned LineNumber, return Error::success(); } - auto [It, DidEmplace] = Globs.try_emplace(Pattern); - if (DidEmplace) { -// We must be sure to use the string in the map rather than the

[clang] [Clang][CodeGen] Add metadata for load from reference (PR #98746)

2025-05-23 Thread Eli Friedman via cfe-commits
https://github.com/efriedma-quic approved this pull request. LGTM https://github.com/llvm/llvm-project/pull/98746 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Modules] Don't fail when an unused textual header is missing. (PR #138227)

2025-05-23 Thread Reid Kleckner via cfe-commits
rnk wrote: (This is mostly written wearing my "Googler" hat, not the clang area team lead hat) There's been a lot of discussion of what the exact use case is. I think it's helpful to share the context that Google essentially uses Clang header modules to wrap generated proto headers, and this

[clang] [Clang][CMake][MSVC] Install PDBs alongside executables (PR #126961)

2025-05-23 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `openmp-s390x-linux` running on `systemz-1` while building `clang` at step 6 "test-openmp". Full details are available at: https://lab.llvm.org/buildbot/#/builders/88/builds/11989 Here is the relevant piece of the build log f

[clang] [llvm] [NFCI][Sanitizer] Convert Matcher::Globs from StringMap to vector. (PR #140964)

2025-05-23 Thread Vitaly Buka via cfe-commits
@@ -125,7 +125,7 @@ class SpecialCaseList { // Returns zero if no match is found. LLVM_ABI unsigned match(StringRef Query) const; -StringMap> Globs; +std::vector>> Globs; vitalybuka wrote: Could be easier with: ``` class Glob { name patt

[clang] [llvm] [NFCI][Sanitizer] Convert Matcher::Globs from StringMap to vector. (PR #140964)

2025-05-23 Thread Vitaly Buka via cfe-commits
@@ -125,7 +125,7 @@ class SpecialCaseList { // Returns zero if no match is found. LLVM_ABI unsigned match(StringRef Query) const; -StringMap> Globs; +std::vector>> Globs; vitalybuka wrote: Glob(Glob&&) = delete; // no copy, no move, because

[clang] [alpha.webkit.NoUnretainedMemberChecker] Recocgnize NS_REQUIRES_PROPERTY_DEFINITIONS (PR #141293)

2025-05-23 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang-static-analyzer-1 Author: Ryosuke Niwa (rniwa) Changes Allow @property of a raw pointer when NS_REQUIRES_PROPERTY_DEFINITIONS is specified on the interface since such an interface does not automatically synthesize raw pointer ivars. --- Full dif

[clang] [clang-format][NFC] FormatTokenLexer.cpp cleanup (PR #141202)

2025-05-23 Thread Owen Pan via cfe-commits
https://github.com/owenca closed https://github.com/llvm/llvm-project/pull/141202 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] eb341f0 - [clang-format][NFC] FormatTokenLexer.cpp cleanup (#141202)

2025-05-23 Thread via cfe-commits
Author: Owen Pan Date: 2025-05-23T13:49:28-07:00 New Revision: eb341f0b044c41dd8313365efbfc45cd587cb5a0 URL: https://github.com/llvm/llvm-project/commit/eb341f0b044c41dd8313365efbfc45cd587cb5a0 DIFF: https://github.com/llvm/llvm-project/commit/eb341f0b044c41dd8313365efbfc45cd587cb5a0.diff LOG:

[clang] [alpha.webkit.NoUnretainedMemberChecker] Recocgnize NS_REQUIRES_PROPERTY_DEFINITIONS (PR #141293)

2025-05-23 Thread via cfe-commits
github-actions[bot] wrote: :warning: C/C++ code formatter, clang-format found issues in your code. :warning: You can test this locally with the following command: ``bash git-clang-format --diff HEAD~1 HEAD --extensions h,cpp -- clang/lib/StaticAnalyzer/Checkers/WebKit/PtrTypesSe

[clang] [llvm] [NFCI][Sanitizer] Convert Matcher::Globs from StringMap to vector. (PR #140964)

2025-05-23 Thread Qinkun Bao via cfe-commits
https://github.com/qinkunbao updated https://github.com/llvm/llvm-project/pull/140964 >From 534d49d489476ffd1aa8f23d21a02f29d3b823fc Mon Sep 17 00:00:00 2001 From: Qinkun Bao Date: Wed, 21 May 2025 22:12:13 + Subject: [PATCH 1/8] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20in?= =?UT

[clang] [CIR] Allow use different Int types together in Vec Shift Op (PR #141111)

2025-05-23 Thread Bruno Cardoso Lopes via cfe-commits
https://github.com/bcardosolopes approved this pull request. LGTM https://github.com/llvm/llvm-project/pull/14 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clang-doc] Avoid reading files in unit tests (PR #141269)

2025-05-23 Thread Nico Weber via cfe-commits
nico wrote: Thanks! Can we stop generating config.h from config.h.cmake again too? That should no longer be needed now. https://github.com/llvm/llvm-project/pull/141269 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi

[clang-tools-extra] a8be7a7 - [clang-doc] Extract Info into JSON values (#138063)

2025-05-23 Thread via cfe-commits
Author: Paul Kirth Date: 2025-05-23T15:56:44-07:00 New Revision: a8be7a74893e82d23d9cd2a8f9fb08181b81aac7 URL: https://github.com/llvm/llvm-project/commit/a8be7a74893e82d23d9cd2a8f9fb08181b81aac7 DIFF: https://github.com/llvm/llvm-project/commit/a8be7a74893e82d23d9cd2a8f9fb08181b81aac7.diff LO

[clang-tools-extra] 26fe803 - [clang-doc] Implement setupTemplateValue for HTMLMustacheGenerator (#138064)

2025-05-23 Thread via cfe-commits
Author: Paul Kirth Date: 2025-05-23T15:59:42-07:00 New Revision: 26fe803b9fcc555ce7f719b542a05690805d5140 URL: https://github.com/llvm/llvm-project/commit/26fe803b9fcc555ce7f719b542a05690805d5140 DIFF: https://github.com/llvm/llvm-project/commit/26fe803b9fcc555ce7f719b542a05690805d5140.diff LO

[clang-tools-extra] [clang-doc] Update serializer for improved template handling (PR #138065)

2025-05-23 Thread Paul Kirth via cfe-commits
https://github.com/ilovepi updated https://github.com/llvm/llvm-project/pull/138065 >From 515b014f4a3b39ff6fa742de095cddbc375f5a3e Mon Sep 17 00:00:00 2001 From: Paul Kirth Date: Tue, 29 Apr 2025 18:31:54 -0700 Subject: [PATCH] [clang-doc] Update serializer for improved template handling This

[clang-tools-extra] [clang-doc] Implement setupTemplateValue for HTMLMustacheGenerator (PR #138064)

2025-05-23 Thread Paul Kirth via cfe-commits
https://github.com/ilovepi closed https://github.com/llvm/llvm-project/pull/138064 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clang-doc] Update serializer for improved template handling (PR #138065)

2025-05-23 Thread Paul Kirth via cfe-commits
https://github.com/ilovepi edited https://github.com/llvm/llvm-project/pull/138065 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] suggest headers on undeclared errors (PR #140247)

2025-05-23 Thread Jongmyeong Choi via cfe-commits
jongmyeong-choi wrote: Thanks for all your reviews. I learned a lot from your comments. I'll try to rewrite it from scratch, but honestly, it'll take me a while to figure out the TableGen structure. If anyone wants to make a change to this topic, please feel free to jump in. https://github.co

[clang] [Modules] Don't fail when an unused textual header is missing. (PR #138227)

2025-05-23 Thread Cyndy Ishida via cfe-commits
cyndyishida wrote: > And, in the end, the folks debugging this (Ilya & co) are busy working on > clang header module reproduction and reduction tools: > https://github.com/emaxx-google/cvise/branches @emaxx-google Interestingly enough, @vsapsai is also working on reproducers for explicitly bu

[clang] [llvm] [NFCI][Sanitizer] Convert Matcher::Globs from StringMap to vector. (PR #140964)

2025-05-23 Thread Qinkun Bao via cfe-commits
https://github.com/qinkunbao updated https://github.com/llvm/llvm-project/pull/140964 >From 534d49d489476ffd1aa8f23d21a02f29d3b823fc Mon Sep 17 00:00:00 2001 From: Qinkun Bao Date: Wed, 21 May 2025 22:12:13 + Subject: [PATCH 01/10] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20?= =?UT

[clang] [llvm] [NFCI][Sanitizer] Convert Matcher::Globs from StringMap to vector. (PR #140964)

2025-05-23 Thread Qinkun Bao via cfe-commits
https://github.com/qinkunbao ready_for_review https://github.com/llvm/llvm-project/pull/140964 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [alpha.webkit.NoUnretainedMemberChecker] Recocgnize NS_REQUIRES_PROPERTY_DEFINITIONS (PR #141293)

2025-05-23 Thread Ryosuke Niwa via cfe-commits
https://github.com/rniwa updated https://github.com/llvm/llvm-project/pull/141293 >From bd7daa355f5c1924475c0818dd4963af7bbffcb1 Mon Sep 17 00:00:00 2001 From: Ryosuke Niwa Date: Fri, 23 May 2025 13:53:36 -0700 Subject: [PATCH 1/2] [alpha.webkit.NoUnretainedMemberChecker] Recocgnize NS_REQUIRE

[clang] Include [[clang::require_explicit_initialization]] warnings in system headers (PR #141133)

2025-05-23 Thread Eli Friedman via cfe-commits
https://github.com/efriedma-quic commented: Looks fine, but I'd like a second approval; I haven't really looked at this template instantiation stuff recently. https://github.com/llvm/llvm-project/pull/141133 ___ cfe-commits mailing list cfe-commits@li

[clang] [llvm] [NFCI][Sanitizer] Convert Matcher::Globs from StringMap to vector. (PR #140964)

2025-05-23 Thread Qinkun Bao via cfe-commits
@@ -53,24 +53,27 @@ Error SpecialCaseList::Matcher::insert(StringRef Pattern, unsigned LineNumber, return Error::success(); } - auto [It, DidEmplace] = Globs.try_emplace(Pattern); - if (DidEmplace) { -// We must be sure to use the string in the map rather than the

[clang] [clang] Add managarm support (PR #139271)

2025-05-23 Thread Brad Smith via cfe-commits
brad0 wrote: @no92 Ping. https://github.com/llvm/llvm-project/pull/139271 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [clang][RISCV] Emit RISCV function-signature-based CFI label in llvm::Function metadata (PR #111661)

2025-05-23 Thread Eli Friedman via cfe-commits
efriedma-quic wrote: The issues I've raised are tied up with quality of implementation, even if the spec doesn't change. For missing prototypes, we can statically detect calls which pass arguments to function without a prototype. Generating code we know will never work, without a diagnostic,

[clang] [CIR] Allow use different Int types together in Vec Shift Op (PR #141111)

2025-05-23 Thread Amr Hesham via cfe-commits
https://github.com/AmrDeveloper updated https://github.com/llvm/llvm-project/pull/14 >From 875db6ee30e8075be50914f62a508511157e8a6f Mon Sep 17 00:00:00 2001 From: AmrDeveloper Date: Thu, 22 May 2025 19:38:08 +0200 Subject: [PATCH 1/4] [CIR] Allow use different Int types together in Vec Shif

[clang] Include [[clang::require_explicit_initialization]] warnings in system headers (PR #141133)

2025-05-23 Thread via cfe-commits
https://github.com/higher-performance updated https://github.com/llvm/llvm-project/pull/141133 >From 1006e27f02b1e6e94b4712d9ccbc5005a616d670 Mon Sep 17 00:00:00 2001 From: higher-performance Date: Thu, 22 May 2025 16:30:29 -0400 Subject: [PATCH] Include [[clang::require_explicit_initialization

[clang] [llvm] [NFCI][Sanitizer] Convert Matcher::Globs from StringMap to vector. (PR #140964)

2025-05-23 Thread Qinkun Bao via cfe-commits
https://github.com/qinkunbao updated https://github.com/llvm/llvm-project/pull/140964 >From 534d49d489476ffd1aa8f23d21a02f29d3b823fc Mon Sep 17 00:00:00 2001 From: Qinkun Bao Date: Wed, 21 May 2025 22:12:13 + Subject: [PATCH 1/6] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20in?= =?UT

[clang] [llvm] [NFCI][Sanitizer] Convert Matcher::Globs from StringMap to vector. (PR #140964)

2025-05-23 Thread Qinkun Bao via cfe-commits
https://github.com/qinkunbao updated https://github.com/llvm/llvm-project/pull/140964 >From 534d49d489476ffd1aa8f23d21a02f29d3b823fc Mon Sep 17 00:00:00 2001 From: Qinkun Bao Date: Wed, 21 May 2025 22:12:13 + Subject: [PATCH 1/7] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20in?= =?UT

[clang] [alpha.webkit.NoUnretainedMemberChecker] Recocgnize NS_REQUIRES_PROPERTY_DEFINITIONS (PR #141293)

2025-05-23 Thread Ryosuke Niwa via cfe-commits
https://github.com/rniwa created https://github.com/llvm/llvm-project/pull/141293 Allow @property of a raw pointer when NS_REQUIRES_PROPERTY_DEFINITIONS is specified on the interface since such an interface does not automatically synthesize raw pointer ivars. >From bd7daa355f5c1924475c0818dd4

[clang] [alpha.webkit.NoUnretainedMemberChecker] Recocgnize NS_REQUIRES_PROPERTY_DEFINITIONS (PR #141293)

2025-05-23 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Ryosuke Niwa (rniwa) Changes Allow @property of a raw pointer when NS_REQUIRES_PROPERTY_DEFINITIONS is specified on the interface since such an interface does not automatically synthesize raw pointer ivars. --- Full diff: https://github.

[clang] [CIR] Add alignment support for global, store, and load ops (PR #141163)

2025-05-23 Thread Bruno Cardoso Lopes via cfe-commits
https://github.com/bcardosolopes approved this pull request. LGTM, long needed. Thanks for changing the incubator too. https://github.com/llvm/llvm-project/pull/141163 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-b

[clang] [llvm] [NFCI][Sanitizer] Convert Matcher::Globs from StringMap to vector. (PR #140964)

2025-05-23 Thread Qinkun Bao via cfe-commits
https://github.com/qinkunbao updated https://github.com/llvm/llvm-project/pull/140964 >From 534d49d489476ffd1aa8f23d21a02f29d3b823fc Mon Sep 17 00:00:00 2001 From: Qinkun Bao Date: Wed, 21 May 2025 22:12:13 + Subject: [PATCH 01/11] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20?= =?UT

[clang] [alpha.webkit.NoUnretainedMemberChecker] Recocgnize NS_REQUIRES_PROPERTY_DEFINITIONS (PR #141293)

2025-05-23 Thread Ryosuke Niwa via cfe-commits
https://github.com/rniwa updated https://github.com/llvm/llvm-project/pull/141293 >From bd7daa355f5c1924475c0818dd4963af7bbffcb1 Mon Sep 17 00:00:00 2001 From: Ryosuke Niwa Date: Fri, 23 May 2025 13:53:36 -0700 Subject: [PATCH 1/3] [alpha.webkit.NoUnretainedMemberChecker] Recocgnize NS_REQUIRE

[clang] [alpha.webkit.NoUnretainedMemberChecker] Recognize NS_REQUIRES_PROPERTY_DEFINITIONS (PR #141293)

2025-05-23 Thread Ryosuke Niwa via cfe-commits
https://github.com/rniwa edited https://github.com/llvm/llvm-project/pull/141293 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [NFCI][Sanitizer] Convert Matcher::Globs from StringMap to vector. (PR #140964)

2025-05-23 Thread Qinkun Bao via cfe-commits
https://github.com/qinkunbao updated https://github.com/llvm/llvm-project/pull/140964 >From 534d49d489476ffd1aa8f23d21a02f29d3b823fc Mon Sep 17 00:00:00 2001 From: Qinkun Bao Date: Wed, 21 May 2025 22:12:13 + Subject: [PATCH 1/9] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20in?= =?UT

[clang-tools-extra] [clang-tidy] Don't run use-nullptr check on objective-c code. (PR #141229)

2025-05-23 Thread Sean McBride via cfe-commits
seanm wrote: Obj-C, being a superset of C, often has C code within it too, would this change mean that NULL -> nullptr won't be suggested at all? Also, isn't Obj-C nil defined as nullptr? I'm pretty sure it is in Obj-C++ at least. https://github.com/llvm/llvm-project/pull/141229

[clang-tools-extra] [clang-tidy] properly handle private move constructors in `modernize-pass-by-value` check (PR #141304)

2025-05-23 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang-tools-extra @llvm/pr-subscribers-clang-tidy Author: Baranov Victor (vbvictor) Changes Fixed false positives when class passed by const-reference had a private move constructor, which could not be used for a fix-it. --- Full diff: https://github.c

[clang] [llvm] [NFCI][Sanitizer] Convert Matcher::Globs from StringMap to vector. (PR #140964)

2025-05-23 Thread Qinkun Bao via cfe-commits
https://github.com/qinkunbao updated https://github.com/llvm/llvm-project/pull/140964 >From 534d49d489476ffd1aa8f23d21a02f29d3b823fc Mon Sep 17 00:00:00 2001 From: Qinkun Bao Date: Wed, 21 May 2025 22:12:13 + Subject: [PATCH 01/11] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20?= =?UT

[clang-tools-extra] [clang-tidy] properly handle private move constructors in `modernize-pass-by-value` check (PR #141304)

2025-05-23 Thread Baranov Victor via cfe-commits
https://github.com/vbvictor created https://github.com/llvm/llvm-project/pull/141304 Fixed false positives when class passed by const-reference had a private move constructor, which could not be used for a fix-it. >From fea5b6609391e6dc9d33777640ff98df1b54fecf Mon Sep 17 00:00:00 2001 From: Ba

[clang] [Clang] add fix-it hints for unknown attributes (PR #141305)

2025-05-23 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Oleksandr T. (a-tarasyuk) Changes This patch adds fix-it hints for unknown attribute names when Clang suggests a correction --- Full diff: https://github.com/llvm/llvm-project/pull/141305.diff 3 Files Affected: - (modified) clang/docs/

[clang-tools-extra] [clang-tidy] properly handle private move constructors in `modernize-pass-by-value` check (PR #141304)

2025-05-23 Thread via cfe-commits
github-actions[bot] wrote: :warning: C/C++ code formatter, clang-format found issues in your code. :warning: You can test this locally with the following command: ``bash git-clang-format --diff HEAD~1 HEAD --extensions cpp -- clang-tools-extra/clang-tidy/modernize/PassByValueChe

[clang-tools-extra] [clang-tidy] properly handle private move constructors in `modernize-pass-by-value` check (PR #141304)

2025-05-23 Thread Baranov Victor via cfe-commits
https://github.com/vbvictor updated https://github.com/llvm/llvm-project/pull/141304 >From fea5b6609391e6dc9d33777640ff98df1b54fecf Mon Sep 17 00:00:00 2001 From: Baranov Victor Date: Sat, 17 May 2025 18:40:54 +0300 Subject: [PATCH 1/5] [clang-tidy] Add isFriendOfClass helper function for Pass

[clang] b208016 - CodeGen: Fix implementation of __builtin_trivially_relocate.

2025-05-23 Thread via cfe-commits
Author: Peter Collingbourne Date: 2025-05-23T17:02:49-07:00 New Revision: b20801646a72b936983261b249e2bc97987445e4 URL: https://github.com/llvm/llvm-project/commit/b20801646a72b936983261b249e2bc97987445e4 DIFF: https://github.com/llvm/llvm-project/commit/b20801646a72b936983261b249e2bc97987445e4

[clang] [Clang] add fix-it hints for unknown attributes (PR #141305)

2025-05-23 Thread Oleksandr T. via cfe-commits
https://github.com/a-tarasyuk created https://github.com/llvm/llvm-project/pull/141305 This patch adds fix-it hints for unknown attribute names when Clang suggests a correction >From 2b84b21462a4e66b8681e165fcd24e3107612448 Mon Sep 17 00:00:00 2001 From: Oleksandr Tarasiuk Date: Sat, 24 May 2

[clang] [CIR] Add alignment support for global, store, and load ops (PR #141163)

2025-05-23 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `sanitizer-aarch64-linux-bootstrap-msan` running on `sanitizer-buildbot9` while building `clang` at step 2 "annotate". Full details are available at: https://lab.llvm.org/buildbot/#/builders/94/builds/7408 Here is the releva

[clang-tools-extra] [clang-tidy] properly handle private move constructors in `modernize-pass-by-value` check (PR #141304)

2025-05-23 Thread Baranov Victor via cfe-commits
https://github.com/vbvictor edited https://github.com/llvm/llvm-project/pull/141304 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [NFCI][Sanitizer] Convert Matcher::Globs from StringMap to vector. (PR #140964)

2025-05-23 Thread Qinkun Bao via cfe-commits
https://github.com/qinkunbao closed https://github.com/llvm/llvm-project/pull/140964 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] e9dbf31 - [NFCI][Sanitizer] Convert Matcher::Globs from StringMap to vector. (#140964)

2025-05-23 Thread via cfe-commits
Author: Qinkun Bao Date: 2025-05-23T20:23:13-04:00 New Revision: e9dbf31be5475a691fca26e56ba8b27337ba31b6 URL: https://github.com/llvm/llvm-project/commit/e9dbf31be5475a691fca26e56ba8b27337ba31b6 DIFF: https://github.com/llvm/llvm-project/commit/e9dbf31be5475a691fca26e56ba8b27337ba31b6.diff LO

[clang] [CIR] Add alignment support for global, store, and load ops (PR #141163)

2025-05-23 Thread Andy Kaylor via cfe-commits
https://github.com/andykaylor closed https://github.com/llvm/llvm-project/pull/141163 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clang-doc] Extract Info into JSON values (PR #138063)

2025-05-23 Thread Petr Hosek via cfe-commits
https://github.com/petrhosek approved this pull request. https://github.com/llvm/llvm-project/pull/138063 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [NFCI][Sanitizer] Convert Matcher::Globs from StringMap to vector. (PR #140964)

2025-05-23 Thread Vitaly Buka via cfe-commits
https://github.com/vitalybuka approved this pull request. https://github.com/llvm/llvm-project/pull/140964 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [alpha.webkit.NoUnretainedMemberChecker] Recognize NS_REQUIRES_PROPERTY_DEFINITIONS (PR #141293)

2025-05-23 Thread Ryosuke Niwa via cfe-commits
https://github.com/rniwa updated https://github.com/llvm/llvm-project/pull/141293 >From bcee06004f24f8488bcc8e84170bf3509daa5fd9 Mon Sep 17 00:00:00 2001 From: Ryosuke Niwa Date: Fri, 23 May 2025 13:53:36 -0700 Subject: [PATCH] [alpha.webkit.NoUnretainedMemberChecker] Recognize NS_REQUIRES_PRO

[clang-tools-extra] [clang-doc] Extract Info into JSON values (PR #138063)

2025-05-23 Thread Paul Kirth via cfe-commits
https://github.com/ilovepi updated https://github.com/llvm/llvm-project/pull/138063 >From 03f3e44aa97edb5cceda728d7318251ccca19cd0 Mon Sep 17 00:00:00 2001 From: Paul Kirth Date: Wed, 30 Apr 2025 08:11:39 -0700 Subject: [PATCH] [clang-doc] Extract Info into JSON values Split from #133161. This

[libcxx] [libcxxabi] [libunwind] [llvm] [libcxxabi][libunwind] Support for using LLVM libc (PR #134893)

2025-05-23 Thread Petr Hosek via cfe-commits
petrhosek wrote: @ldionne Would it be possible to take another look? https://github.com/llvm/llvm-project/pull/134893 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

<    1   2   3   4   >