[clang-tools-extra] [clang-tidy] Fix false positives with deducing this in `readability-convert-member-functions-to-static` check (PR #141391)

2025-06-07 Thread Baranov Victor via cfe-commits
https://github.com/vbvictor closed https://github.com/llvm/llvm-project/pull/141391 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang-format] Add options to control wrapped lambda brace indent. (PR #143249)

2025-06-07 Thread via cfe-commits
@@ -1547,6 +1547,38 @@ struct FormatStyle { bool BeforeWhile; /// Indent the wrapped braces themselves. bool IndentBraces; +/// Indent nested wrapped lambda braces. rmarker wrote: When it is inside some form of brackets. >From the documentation

[clang-tools-extra] [clang-tidy] Fix false positives with deducing this in `readability-convert-member-functions-to-static` check (PR #141391)

2025-06-07 Thread Baranov Victor via cfe-commits
vbvictor wrote: When merging, your new public email will be used, now it says "This commit will be authored by flb...@protonmail.com." https://github.com/llvm/llvm-project/pull/141391 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lis

[clang-tools-extra] [clang-tidy] Fix false positives with deducing this in `readability-convert-member-functions-to-static` check (PR #141391)

2025-06-07 Thread Carlos Galvez via cfe-commits
https://github.com/carlosgalvezp approved this pull request. LGTM! https://github.com/llvm/llvm-project/pull/141391 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clang-tidy] Fix false positives with deducing this in `readability-convert-member-functions-to-static` check (PR #141391)

2025-06-07 Thread via cfe-commits
flovent wrote: Rebased and changed my email to public. Different with this PR, I use github noreply email in my latest PR to commit, do I need to rebase to change it or it will automaticly update to my public email when merging like it will transfter to noreply email like [LLVM Discourse](htt

[clang-tools-extra] [clang-tidy] Fix false positives with deducing this in `readability-convert-member-functions-to-static` check (PR #141391)

2025-06-07 Thread via cfe-commits
https://github.com/flovent updated https://github.com/llvm/llvm-project/pull/141391 >From 5b8b6d4ce5e7faa32f1c26e57a71739e6440a2e1 Mon Sep 17 00:00:00 2001 From: fubowen Date: Sun, 25 May 2025 09:37:47 +0800 Subject: [PATCH 1/5] [clang-tidy] fix false positives with deducing this in `readabili

[clang-tools-extra] [llvm] [llvm][ADT] Add wrappers to `std::includes` (PR #143297)

2025-06-07 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clangd Author: Longsheng Mou (CoTinker) Changes Add `llvm::includes` that accepts a range instead of start/end iterator. --- Full diff: https://github.com/llvm/llvm-project/pull/143297.diff 8 Files Affected: - (modified) clang-tools-extra/clangd/refa

[clang-tools-extra] [llvm] [llvm][ADT] Add wrappers to `std::includes` (PR #143297)

2025-06-07 Thread Longsheng Mou via cfe-commits
https://github.com/CoTinker created https://github.com/llvm/llvm-project/pull/143297 Add `llvm::includes` that accepts a range instead of start/end iterator. >From dea346e4361644aa9fc1c66bea6d18b1a582d009 Mon Sep 17 00:00:00 2001 From: Longsheng Mou Date: Sun, 8 Jun 2025 12:24:44 +0800 Subject

[clang-tools-extra] [clang-tidy] Fix false positives with deducing this in `readability-convert-member-functions-to-static` check (PR #141391)

2025-06-07 Thread Baranov Victor via cfe-commits
vbvictor wrote: Could you please rebase on fresh main, and I will land the changes. https://github.com/llvm/llvm-project/pull/141391 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Support constexpr asm at global scope. (PR #143268)

2025-06-07 Thread Younan Zhang via cfe-commits
https://github.com/zyn0217 approved this pull request. https://github.com/llvm/llvm-project/pull/143268 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] 897ccdd - [clang-format][NFC] Clean up fillRanges() in ClangFormat.cpp (#143236)

2025-06-07 Thread via cfe-commits
Author: Owen Pan Date: 2025-06-07T20:59:53-07:00 New Revision: 897ccddcc30cacdfe04ddd622986b50ec963eabc URL: https://github.com/llvm/llvm-project/commit/897ccddcc30cacdfe04ddd622986b50ec963eabc DIFF: https://github.com/llvm/llvm-project/commit/897ccddcc30cacdfe04ddd622986b50ec963eabc.diff LOG:

[clang] [clang-format][NFC] Clean up fillRanges() in ClangFormat.cpp (PR #143236)

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

[clang] [clang-format][NFC] Clean up fillRanges() in ClangFormat.cpp (PR #143236)

2025-06-07 Thread Owen Pan via cfe-commits
@@ -284,27 +284,23 @@ static bool fillRanges(MemoryBuffer *Code, errs() << "error: number of -offset and -length arguments must match.\n"; return true; } - for (unsigned i = 0, e = Offsets.size(); i != e; ++i) { -if (Offsets[i] >= Code->getBufferSize()) { -

[clang] [clang] AST: fix dependency calculation for TypedefTypes (PR #143291)

2025-06-07 Thread Younan Zhang via cfe-commits
https://github.com/zyn0217 approved this pull request. Thanks https://github.com/llvm/llvm-project/pull/143291 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clang-tidy] Fix false positives with template in `misc-unconventional-assign-operator` check (PR #143292)

2025-06-07 Thread via cfe-commits
https://github.com/flovent updated https://github.com/llvm/llvm-project/pull/143292 >From 311bc80c0e5ab787aef8476dae51bc16805d728b Mon Sep 17 00:00:00 2001 From: flovent <144676429+flov...@users.noreply.github.com> Date: Sun, 8 Jun 2025 10:46:48 +0800 Subject: [PATCH 1/2] [clang-tidy] Fix false

[clang-tools-extra] [clang-tidy] Fix false positives with template in `misc-unconventional-assign-operator` check (PR #143292)

2025-06-07 Thread via cfe-commits
github-actions[bot] wrote: ⚠️ We detected that you are using a GitHub private e-mail address to contribute to the repo. Please turn off [Keep my email addresses private](https://github.com/settings/emails) setting in your account. See [LLVM Discourse](https://discourse.llvm.org/t/hidden-email

[clang-tools-extra] [clang-tidy] Fix false positives with template in `misc-unconventional-assign-operator` check (PR #143292)

2025-06-07 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang-tools-extra Author: None (flovent) Changes Fix false positives when copy assignment operator function in a template class returns the result of another assignment to `*this`, this check doesn't consider this situation that there will be a `Binary

[clang-tools-extra] [clang-tidy] Fix false positives with template in `misc-unconventional-assign-operator` check (PR #143292)

2025-06-07 Thread via cfe-commits
https://github.com/flovent created https://github.com/llvm/llvm-project/pull/143292 Fix false positives when copy assignment operator function in a template class returns the result of another assignment to `*this`, this check doesn't consider this situation that there will be a `BinaryOperato

[clang] [clang] AST: fix dependency calculation for TypedefTypes (PR #143291)

2025-06-07 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-debuginfo @llvm/pr-subscribers-clang Author: Matheus Izvekov (mizvekov) Changes The dependency from the type sugar of the underlying type of a Typedef were not being considered for the dependency of the TypedefType itself. A TypedefType should be insta

[clang] [clang] AST: fix dependency calculation for TypedefTypes (PR #143291)

2025-06-07 Thread Matheus Izvekov via cfe-commits
https://github.com/mizvekov created https://github.com/llvm/llvm-project/pull/143291 The dependency from the type sugar of the underlying type of a Typedef were not being considered for the dependency of the TypedefType itself. A TypedefType should be instantiation dependent if it involves non

[clang] [clang][NFC] Refactor replaceExternalDecls to use llvm::any_of (PR #143275)

2025-06-07 Thread Yanzuo Liu via cfe-commits
zwuis wrote: IIUC the `FIXME` comment means that the time complexity of this part of code is not good enough and we can make it faster. https://github.com/llvm/llvm-project/pull/143275 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://li

[clang-tools-extra] [clang-doc] add a JSON generator (PR #142483)

2025-06-07 Thread Petr Hosek via cfe-commits
@@ -0,0 +1,194 @@ +// RUN: rm -rf %t && mkdir -p %t +// RUN: clang-doc --output=%t --format=json --executor=standalone %s +// RUN: FileCheck %s < %t/GlobalNamespace/MyClass.json + +struct Foo; + +// This is a nice class. +// It has some nice methods and fields. +// @brief This is

[clang] [clang] Catch missing format attributes (PR #105479)

2025-06-07 Thread Aaron Puchert via cfe-commits
@@ -0,0 +1,251 @@ +// RUN: %clang_cc1 -fsyntax-only -verify=expected,c_diagnostics -Wmissing-format-attribute %s +// RUN: %clang_cc1 -fsyntax-only -Wmissing-format-attribute -fdiagnostics-parseable-fixits %s 2>&1 | FileCheck %s +// RUN: %clang_cc1 -fsyntax-only -x c++ -verify=ex

[clang] [clang] Catch missing format attributes (PR #105479)

2025-06-07 Thread Aaron Puchert via cfe-commits
@@ -5918,6 +5918,181 @@ static void handlePreferredTypeAttr(Sema &S, Decl *D, const ParsedAttr &AL) { D->addAttr(::new (S.Context) PreferredTypeAttr(S.Context, AL, ParmTSI)); } +// Diagnosing missing format attributes is implemented in two steps: +// 1. Detect missing forma

[clang] [clang] Catch missing format attributes (PR #105479)

2025-06-07 Thread Aaron Puchert via cfe-commits
@@ -5918,6 +5918,181 @@ static void handlePreferredTypeAttr(Sema &S, Decl *D, const ParsedAttr &AL) { D->addAttr(::new (S.Context) PreferredTypeAttr(S.Context, AL, ParmTSI)); } +// Diagnosing missing format attributes is implemented in two steps: +// 1. Detect missing forma

[clang] [clang] Catch missing format attributes (PR #105479)

2025-06-07 Thread Aaron Puchert via cfe-commits
@@ -5918,6 +5918,181 @@ static void handlePreferredTypeAttr(Sema &S, Decl *D, const ParsedAttr &AL) { D->addAttr(::new (S.Context) PreferredTypeAttr(S.Context, AL, ParmTSI)); } +// Diagnosing missing format attributes is implemented in two steps: +// 1. Detect missing forma

[clang] [clang] Catch missing format attributes (PR #105479)

2025-06-07 Thread Aaron Puchert via cfe-commits
@@ -0,0 +1,259 @@ +// RUN: %clang_cc1 -fsyntax-only -verify -Wmissing-format-attribute %s +// RUN: %clang_cc1 -fsyntax-only -Wmissing-format-attribute -fdiagnostics-parseable-fixits %s 2>&1 | FileCheck %s +// RUN: %clang_cc1 -fsyntax-only -x c++ -verify -Wmissing-format-attribute

[clang] [clang] Catch missing format attributes (PR #105479)

2025-06-07 Thread Aaron Puchert via cfe-commits
@@ -5335,6 +5335,230 @@ static void handlePreferredTypeAttr(Sema &S, Decl *D, const ParsedAttr &AL) { D->addAttr(::new (S.Context) PreferredTypeAttr(S.Context, AL, ParmTSI)); } +// Returns vector of format attributes. There are no two attributes with same +// arguments in r

[clang] [clang] Catch missing format attributes (PR #105479)

2025-06-07 Thread Aaron Puchert via cfe-commits
@@ -0,0 +1,259 @@ +// RUN: %clang_cc1 -fsyntax-only -verify -Wmissing-format-attribute %s +// RUN: %clang_cc1 -fsyntax-only -Wmissing-format-attribute -fdiagnostics-parseable-fixits %s 2>&1 | FileCheck %s +// RUN: %clang_cc1 -fsyntax-only -x c++ -verify -Wmissing-format-attribute

[clang] [clang] Catch missing format attributes (PR #105479)

2025-06-07 Thread Aaron Puchert via cfe-commits
@@ -5918,6 +5918,181 @@ static void handlePreferredTypeAttr(Sema &S, Decl *D, const ParsedAttr &AL) { D->addAttr(::new (S.Context) PreferredTypeAttr(S.Context, AL, ParmTSI)); } +// Diagnosing missing format attributes is implemented in two steps: +// 1. Detect missing forma

[clang] [clang] Catch missing format attributes (PR #105479)

2025-06-07 Thread Aaron Puchert via cfe-commits
@@ -3460,8 +3460,10 @@ void Sema::checkCall(NamedDecl *FDecl, const FunctionProtoType *Proto, } } - if (FD) + if (FD) { diagnoseArgDependentDiagnoseIfAttrs(FD, ThisArg, Args, Loc); +DetectMissingFormatAttributes(FD, Args, Loc); aaronpuchert w

[clang] [clang] Catch missing format attributes (PR #105479)

2025-06-07 Thread Aaron Puchert via cfe-commits
@@ -5918,6 +5918,181 @@ static void handlePreferredTypeAttr(Sema &S, Decl *D, const ParsedAttr &AL) { D->addAttr(::new (S.Context) PreferredTypeAttr(S.Context, AL, ParmTSI)); } +// Diagnosing missing format attributes is implemented in two steps: +// 1. Detect missing forma

[clang] [clang] Catch missing format attributes (PR #105479)

2025-06-07 Thread Aaron Puchert via cfe-commits
https://github.com/aaronpuchert edited https://github.com/llvm/llvm-project/pull/105479 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang] Catch missing format attributes (PR #105479)

2025-06-07 Thread Aaron Puchert via cfe-commits
https://github.com/aaronpuchert commented: > As I understand your change, it can only trip when `-Wformat-nonliteral` also > trips. I think that you should implement this as a note/fixit on > `-Wformat-nonliteral`. Good observation, and I agree. It's even documented that this is one way to fix

[clang] [clang][NFC] Refactor replaceExternalDecls to use llvm::any_of (PR #143275)

2025-06-07 Thread Samarth Narang via cfe-commits
snarang181 wrote: @ChuanqiXu9, requesting your review here. https://github.com/llvm/llvm-project/pull/143275 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang-format] Add options to control wrapped lambda brace indent. (PR #143249)

2025-06-07 Thread Björn Schäpers via cfe-commits
https://github.com/HazardyKnusperkeks commented: A general note, if you comment in the `Files changed` tab and `Start a review` it should only send one email for all your comments. https://github.com/llvm/llvm-project/pull/143249 ___ cfe-commits maili

[clang] [clang-format] Add options to control wrapped lambda brace indent. (PR #143249)

2025-06-07 Thread Björn Schäpers via cfe-commits
https://github.com/HazardyKnusperkeks edited https://github.com/llvm/llvm-project/pull/143249 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang-format] Add options to control wrapped lambda brace indent. (PR #143249)

2025-06-07 Thread Björn Schäpers via cfe-commits
@@ -1547,6 +1547,38 @@ struct FormatStyle { bool BeforeWhile; /// Indent the wrapped braces themselves. bool IndentBraces; +/// Indent nested wrapped lambda braces. HazardyKnusperkeks wrote: I don't really have something, but you should swap La

[clang] [clang-format] Add options to control wrapped lambda brace indent. (PR #143249)

2025-06-07 Thread Björn Schäpers via cfe-commits
@@ -1334,12 +1334,15 @@ unsigned ContinuationIndenter::getNewLineColumn(const LineState &State) { Style.IndentWidth; } - if (Style.BraceWrapping.BeforeLambdaBody && - Style.BraceWrapping.IndentBraces && Current.is(TT_LambdaLBrace)) { + if (Style.BraceWrapp

[clang] [clang-format][NFC] Clean up fillRanges() in ClangFormat.cpp (PR #143236)

2025-06-07 Thread Björn Schäpers via cfe-commits
@@ -284,27 +284,23 @@ static bool fillRanges(MemoryBuffer *Code, errs() << "error: number of -offset and -length arguments must match.\n"; return true; } - for (unsigned i = 0, e = Offsets.size(); i != e; ++i) { -if (Offsets[i] >= Code->getBufferSize()) { -

[clang] [clang-format][NFC] Clean up fillRanges() in ClangFormat.cpp (PR #143236)

2025-06-07 Thread Owen Pan via cfe-commits
@@ -284,27 +284,23 @@ static bool fillRanges(MemoryBuffer *Code, errs() << "error: number of -offset and -length arguments must match.\n"; owenca wrote: Actually, we can achieve that by handling `Offsets.size() == 1 && Lengths.empty()` separately. https://

[clang-tools-extra] 30d8aeb - [clang-tidy] Add option to disable bugprone-multi-level-pointer-conversion in C code (#141209)

2025-06-07 Thread via cfe-commits
Author: Carlos Galvez Date: 2025-06-07T21:52:23+02:00 New Revision: 30d8aebbe2c3bd4bdda7a7feefeb7cf7d556a458 URL: https://github.com/llvm/llvm-project/commit/30d8aebbe2c3bd4bdda7a7feefeb7cf7d556a458 DIFF: https://github.com/llvm/llvm-project/commit/30d8aebbe2c3bd4bdda7a7feefeb7cf7d556a458.diff

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

2025-06-07 Thread Carlos Galvez via cfe-commits
https://github.com/carlosgalvezp closed https://github.com/llvm/llvm-project/pull/141209 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clang-tools-extra] Use llvm::any_of (NFC) (PR #143281)

2025-06-07 Thread Carlos Galvez via cfe-commits
https://github.com/carlosgalvezp approved this pull request. https://github.com/llvm/llvm-project/pull/143281 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang-format][NFC] Clean up fillRanges() in ClangFormat.cpp (PR #143236)

2025-06-07 Thread Owen Pan via cfe-commits
https://github.com/owenca updated https://github.com/llvm/llvm-project/pull/143236 >From 1ece3adee540295efbe04fc74876efd39549e577 Mon Sep 17 00:00:00 2001 From: Owen Pan Date: Fri, 6 Jun 2025 23:10:18 -0700 Subject: [PATCH 1/3] [clang-format][NFC] Clean up fillRanges() in ClangFormat.cpp ---

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

2025-06-07 Thread Carlos Galvez via cfe-commits
carlosgalvezp wrote: Went for adding an option instead! https://github.com/llvm/llvm-project/pull/141209 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

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

2025-06-07 Thread Carlos Galvez via cfe-commits
https://github.com/carlosgalvezp updated https://github.com/llvm/llvm-project/pull/141209 >From 3e9f3b2d01343be72c1da3a4dafe9fc2f23c2e63 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Carlos=20G=C3=A1lvez?= Date: Fri, 23 May 2025 07:45:04 + Subject: [PATCH] [clang-tidy] Add option to disable bug

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

2025-06-07 Thread Carlos Galvez via cfe-commits
https://github.com/carlosgalvezp edited https://github.com/llvm/llvm-project/pull/141209 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

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

2025-06-07 Thread Carlos Galvez via cfe-commits
https://github.com/carlosgalvezp edited https://github.com/llvm/llvm-project/pull/141209 ___ 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-06-07 Thread Carlos Galvez via cfe-commits
https://github.com/carlosgalvezp updated https://github.com/llvm/llvm-project/pull/141209 >From 4de236aac2c75c1c092ac6b46591839678114365 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Carlos=20G=C3=A1lvez?= Date: Fri, 23 May 2025 07:45:04 + Subject: [PATCH] [clang-tidy] Add option to disable bug

[clang] [clang][NFC] Refactor replaceExternalDecls to use llvm::any_of (PR #143275)

2025-06-07 Thread Samarth Narang via cfe-commits
https://github.com/snarang181 ready_for_review https://github.com/llvm/llvm-project/pull/143275 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang][NFC] Refactor replaceExternalDecls to use llvm::any_of (PR #143275)

2025-06-07 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Samarth Narang (snarang181) Changes This patch simplifies the declaration replacement logic in replaceExternalDecls by replacing a manual loop with an idiomatic use of llvm::any_of. This improves code readability and aligns with common LL

[clang] [clang][NFC] Refactor replaceExternalDecls to use llvm::any_of (PR #143275)

2025-06-07 Thread Samarth Narang via cfe-commits
https://github.com/snarang181 edited https://github.com/llvm/llvm-project/pull/143275 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] Refactor replaceExternalDecls to use llvm::any_of (PR #143275)

2025-06-07 Thread Samarth Narang via cfe-commits
https://github.com/snarang181 edited https://github.com/llvm/llvm-project/pull/143275 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] Refactor replaceExternalDecls to use llvm::any_of (PR #143275)

2025-06-07 Thread Samarth Narang via cfe-commits
https://github.com/snarang181 edited https://github.com/llvm/llvm-project/pull/143275 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clang-move] Teach getDeclarationList to return ArrayRef (NFC) (PR #143278)

2025-06-07 Thread Shilei Tian via cfe-commits
https://github.com/shiltian approved this pull request. https://github.com/llvm/llvm-project/pull/143278 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clang-tools-extra] Use llvm::any_of (NFC) (PR #143281)

2025-06-07 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang-tidy Author: Kazu Hirata (kazutakahirata) Changes --- Full diff: https://github.com/llvm/llvm-project/pull/143281.diff 2 Files Affected: - (modified) clang-tools-extra/clang-doc/HTMLMustacheGenerator.cpp (+2-3) - (modified) clang-tools-extra/

[clang-tools-extra] [clang-tools-extra] Use llvm::any_of (NFC) (PR #143281)

2025-06-07 Thread Kazu Hirata via cfe-commits
https://github.com/kazutakahirata created https://github.com/llvm/llvm-project/pull/143281 None >From fb4b1787546b3a11bca7d79eae8900277748cdc5 Mon Sep 17 00:00:00 2001 From: Kazu Hirata Date: Sat, 7 Jun 2025 09:20:35 -0700 Subject: [PATCH] [clang-tools-extra] Use llvm::any_of (NFC) --- clang

[clang-tools-extra] [clang-move] Teach getDeclarationList to return ArrayRef (NFC) (PR #143278)

2025-06-07 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang-tools-extra Author: Kazu Hirata (kazutakahirata) Changes getDeclarationList is used only for read-only access to the array. I don't think it's actually meant to return by value. --- Full diff: https://github.com/llvm/llvm-project/pull/143278.dif

[clang-tools-extra] [clang-move] Teach getDeclarationList to return ArrayRef (NFC) (PR #143278)

2025-06-07 Thread Kazu Hirata via cfe-commits
https://github.com/kazutakahirata created https://github.com/llvm/llvm-project/pull/143278 getDeclarationList is used only for read-only access to the array. I don't think it's actually meant to return by value. >From d69392b80cdb2cb60aec005bcc4e18f5f9d4453a Mon Sep 17 00:00:00 2001 From: Kaz

[clang-tools-extra] [clang-tidy] Improve integer comparison by matching valid expressions outside implicitCastExpr (PR #134188)

2025-06-07 Thread David Rivera via cfe-commits
https://github.com/RiverDave deleted https://github.com/llvm/llvm-project/pull/134188 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [NFC][Clang] Adopt simplified `getTrailingObjects` in StmtObjC (PR #143256)

2025-06-07 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `clang-m68k-linux-cross` running on `suse-gary-m68k-cross` while building `clang` at step 5 "ninja check 1". Full details are available at: https://lab.llvm.org/buildbot/#/builders/27/builds/11218 Here is the relevant piece

[clang] Reapply "[Clang] Profile singly-resolved UnresolvedLookupExpr with the declaration" (PR #140680)

2025-06-07 Thread Corentin Jabot via cfe-commits
https://github.com/cor3ntin approved this pull request. we could merge that now to fix the bug, and introduce a new node later https://github.com/llvm/llvm-project/pull/140680 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.

[clang] [NFC][Clang] Adopt simplified `getTrailingObjects` in ParsedTemplate (PR #143255)

2025-06-07 Thread Rahul Joshi via cfe-commits
https://github.com/jurahul closed https://github.com/llvm/llvm-project/pull/143255 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] 7dbae40 - [NFC][Clang] Adopt simplified `getTrailingObjects` in ParsedTemplate (#143255)

2025-06-07 Thread via cfe-commits
Author: Rahul Joshi Date: 2025-06-07T09:56:45-07:00 New Revision: 7dbae40c94fc39727b847e3db0aed0cf19adc3ce URL: https://github.com/llvm/llvm-project/commit/7dbae40c94fc39727b847e3db0aed0cf19adc3ce DIFF: https://github.com/llvm/llvm-project/commit/7dbae40c94fc39727b847e3db0aed0cf19adc3ce.diff L

[clang] [NFC][Clang] Adopt simplified `getTrailingObjects` in ExprObjC (PR #143254)

2025-06-07 Thread Rahul Joshi via cfe-commits
https://github.com/jurahul closed https://github.com/llvm/llvm-project/pull/143254 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] 6615ec6 - [NFC][Clang] Adopt simplified `getTrailingObjects` in ExprObjC (#143254)

2025-06-07 Thread via cfe-commits
Author: Rahul Joshi Date: 2025-06-07T09:56:24-07:00 New Revision: 6615ec6084f05e609e00025f7f31216710e765cc URL: https://github.com/llvm/llvm-project/commit/6615ec6084f05e609e00025f7f31216710e765cc DIFF: https://github.com/llvm/llvm-project/commit/6615ec6084f05e609e00025f7f31216710e765cc.diff L

[clang] 3ebb91d - [NFC][Clang] Adopt simplified `getTrailingObjects` in ExprConcepts (#143252)

2025-06-07 Thread via cfe-commits
Author: Rahul Joshi Date: 2025-06-07T09:56:05-07:00 New Revision: 3ebb91d3c8013b7da577bab6db249ce5bea6d6ee URL: https://github.com/llvm/llvm-project/commit/3ebb91d3c8013b7da577bab6db249ce5bea6d6ee DIFF: https://github.com/llvm/llvm-project/commit/3ebb91d3c8013b7da577bab6db249ce5bea6d6ee.diff L

[clang] [NFC][Clang] Adopt simplified `getTrailingObjects` in ExprConcepts (PR #143252)

2025-06-07 Thread Rahul Joshi via cfe-commits
https://github.com/jurahul closed https://github.com/llvm/llvm-project/pull/143252 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [NFC][Clang] Adopt simplified `getTrailingObjects` in StmtCXX (PR #143251)

2025-06-07 Thread Rahul Joshi via cfe-commits
https://github.com/jurahul closed https://github.com/llvm/llvm-project/pull/143251 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] 3bf2aec - [NFC][Clang] Adopt simplified `getTrailingObjects` in StmtCXX (#143251)

2025-06-07 Thread via cfe-commits
Author: Rahul Joshi Date: 2025-06-07T09:55:44-07:00 New Revision: 3bf2aece835bff9de41759bb8993a32de57e0f06 URL: https://github.com/llvm/llvm-project/commit/3bf2aece835bff9de41759bb8993a32de57e0f06 DIFF: https://github.com/llvm/llvm-project/commit/3bf2aece835bff9de41759bb8993a32de57e0f06.diff L

[clang] [clang][AST] Fix spaces in TypePrinter for some calling convs (PR #143160)

2025-06-07 Thread Nick Sarnie via cfe-commits
https://github.com/sarnex closed https://github.com/llvm/llvm-project/pull/143160 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] 56b9844 - [clang][AST] Fix spaces in TypePrinter for some calling convs (#143160)

2025-06-07 Thread via cfe-commits
Author: Nick Sarnie Date: 2025-06-07T16:47:28Z New Revision: 56b98449c8f047c983508b13294a9610f8df0e2b URL: https://github.com/llvm/llvm-project/commit/56b98449c8f047c983508b13294a9610f8df0e2b DIFF: https://github.com/llvm/llvm-project/commit/56b98449c8f047c983508b13294a9610f8df0e2b.diff LOG: [

[clang] [NFC][Clang] Adopt simplified `getTrailingObjects` in ParsedTemplate (PR #143255)

2025-06-07 Thread Corentin Jabot via cfe-commits
https://github.com/cor3ntin approved this pull request. https://github.com/llvm/llvm-project/pull/143255 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [NFC][Clang] Adopt simplified `getTrailingObjects` in StmtObjC (PR #143256)

2025-06-07 Thread Rahul Joshi via cfe-commits
https://github.com/jurahul closed https://github.com/llvm/llvm-project/pull/143256 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] de256ac - [NFC][Clang] Adopt simplified `getTrailingObjects` in StmtObjC (#143256)

2025-06-07 Thread via cfe-commits
Author: Rahul Joshi Date: 2025-06-07T09:36:51-07:00 New Revision: de256ac8fb33ef1d5f38be278a8a81e0cc5236eb URL: https://github.com/llvm/llvm-project/commit/de256ac8fb33ef1d5f38be278a8a81e0cc5236eb DIFF: https://github.com/llvm/llvm-project/commit/de256ac8fb33ef1d5f38be278a8a81e0cc5236eb.diff L

[clang] [NFC][Clang] Adopt simplified `getTrailingObjects` in ExprObjC (PR #143254)

2025-06-07 Thread Corentin Jabot via cfe-commits
https://github.com/cor3ntin approved this pull request. https://github.com/llvm/llvm-project/pull/143254 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [NFC][Clang] Adopt simplified `getTrailingObjects` in StmtObjC (PR #143256)

2025-06-07 Thread Rahul Joshi via cfe-commits
jurahul wrote: Thanks https://github.com/llvm/llvm-project/pull/143256 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [NFC][Clang] Adopt simplified `getTrailingObjects` in ExprConcepts (PR #143252)

2025-06-07 Thread Corentin Jabot via cfe-commits
https://github.com/cor3ntin approved this pull request. https://github.com/llvm/llvm-project/pull/143252 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [NFC][Clang] Adopt simplified `getTrailingObjects` in StmtCXX (PR #143251)

2025-06-07 Thread Corentin Jabot via cfe-commits
https://github.com/cor3ntin approved this pull request. https://github.com/llvm/llvm-project/pull/143251 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [NFC][Clang] Adopt simplified `getTrailingObjects` in StmtObjC (PR #143256)

2025-06-07 Thread Corentin Jabot via cfe-commits
https://github.com/cor3ntin approved this pull request. https://github.com/llvm/llvm-project/pull/143256 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] Replace loop with llvm:any_of (PR #143275)

2025-06-07 Thread Samarth Narang via cfe-commits
https://github.com/snarang181 created https://github.com/llvm/llvm-project/pull/143275 None >From dc4e3c9afbef406ae1acbd572dfe9edd9d5efea3 Mon Sep 17 00:00:00 2001 From: Samarth Narang Date: Sat, 7 Jun 2025 12:21:17 -0400 Subject: [PATCH] Replace loop with llvm:any_of --- clang/include/clang

[clang] [Clang] Support constexpr asm at global scope. (PR #143268)

2025-06-07 Thread Fangrui Song via cfe-commits
https://github.com/MaskRay approved this pull request. https://github.com/llvm/llvm-project/pull/143268 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Support constexpr asm at global scope. (PR #143268)

2025-06-07 Thread Corentin Jabot via cfe-commits
https://github.com/cor3ntin updated https://github.com/llvm/llvm-project/pull/143268 >From e50565a0e4850eb66ba718be08324a9b5af73d5a Mon Sep 17 00:00:00 2001 From: Corentin Jabot Date: Sat, 7 Jun 2025 16:23:38 +0200 Subject: [PATCH] [Clang] Support constexpr asm at global scope. I previously fa

[clang-tools-extra] Refactor clang doc comment structure (PR #142273)

2025-06-07 Thread Samarth Narang via cfe-commits
snarang181 wrote: @ilovepi, this failure is probably related to the merged PR, right? https://github.com/llvm/llvm-project/pull/142273 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] Refactor clang doc comment structure (PR #142273)

2025-06-07 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `sanitizer-x86_64-linux-fast` running on `sanitizer-buildbot4` while building `clang-tools-extra` at step 2 "annotate". Full details are available at: https://lab.llvm.org/buildbot/#/builders/169/builds/12077 Here is the rel

[clang-tools-extra] Refactor clang doc comment structure (PR #142273)

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

[clang-tools-extra] d570409 - Refactor clang doc comment structure (#142273)

2025-06-07 Thread via cfe-commits
Author: Samarth Narang Date: 2025-06-07T08:04:56-07:00 New Revision: d5704097fcc2128bb8c97867e090543024a936d0 URL: https://github.com/llvm/llvm-project/commit/d5704097fcc2128bb8c97867e090543024a936d0 DIFF: https://github.com/llvm/llvm-project/commit/d5704097fcc2128bb8c97867e090543024a936d0.diff

[clang-tools-extra] [clangd] Improve Markup Rendering (PR #140498)

2025-06-07 Thread via cfe-commits
tcottin wrote: ping https://github.com/llvm/llvm-project/pull/140498 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [libclang/python] Add missing enum variants (PR #143264)

2025-06-07 Thread Jannick Kremer via cfe-commits
https://github.com/DeinAlptraum updated https://github.com/llvm/llvm-project/pull/143264 >From 71a404a5bbef81fa0aba48eb14671ed17ce4d8fb Mon Sep 17 00:00:00 2001 From: Jannick Kremer Date: Sat, 7 Jun 2025 21:57:17 +0900 Subject: [PATCH 1/2] [libclang/python] Add missing enum variants Add tests

[clang] [libclang/python] Add missing enum variants (PR #143264)

2025-06-07 Thread Jannick Kremer via cfe-commits
DeinAlptraum wrote: @Endilll this adds the missing enum variants, and also adds a test using libclang, parsing `Index.h` for all our used enums to check for missing variants. You can see what this looks like in case of failure on the first CI run for this PR. This uses only the numeric enum v

[clang-tools-extra] [WIP] Warn about misuse of sizeof operator in loops. (PR #143205)

2025-06-07 Thread via cfe-commits
https://github.com/EugeneZelenko edited https://github.com/llvm/llvm-project/pull/143205 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [WIP] Warn about misuse of sizeof operator in loops. (PR #143205)

2025-06-07 Thread via cfe-commits
@@ -316,3 +316,11 @@ Options When `true`, the check will warn on pointer arithmetic where the element count is obtained from a division with ``sizeof(...)``, e.g., ``Ptr + Bytes / sizeof(*T)``. Default is `true`. + +.. option:: WarnOnSizeOfInLoopTermination + + When

[clang-tools-extra] [WIP] Warn about misuse of sizeof operator in loops. (PR #143205)

2025-06-07 Thread via cfe-commits
https://github.com/EugeneZelenko commented: Please mention changes in Release Notes. https://github.com/llvm/llvm-project/pull/143205 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Support constexpr asm at global scope. (PR #143268)

2025-06-07 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/include/clang/AST/ASTNodeTraverser.h clang/in

[clang] [libclang/python] Add missing enum variants (PR #143264)

2025-06-07 Thread Jannick Kremer via cfe-commits
https://github.com/DeinAlptraum updated https://github.com/llvm/llvm-project/pull/143264 >From cee0067c7a0b35d2e1f15fd3fa698ec2afd6de88 Mon Sep 17 00:00:00 2001 From: Jannick Kremer Date: Sat, 7 Jun 2025 21:57:17 +0900 Subject: [PATCH 1/2] [libclang/python] Add missing enum variants Add tests

[clang] [Clang] Support constexpr asm at global scope. (PR #143268)

2025-06-07 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Corentin Jabot (cor3ntin) Changes I previously failed to realize this feature existed... Fixes #137459 Fixes #143242 --- Full diff: https://github.com/llvm/llvm-project/pull/143268.diff 9 Files Affected: - (modified) clang/include/clan

[clang] [Clang] Support constexpr asm at global scope. (PR #143268)

2025-06-07 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang-modules Author: Corentin Jabot (cor3ntin) Changes I previously failed to realize this feature existed... Fixes #137459 Fixes #143242 --- Full diff: https://github.com/llvm/llvm-project/pull/143268.diff 9 Files Affected: - (modified) clang/incl

[clang] [Clang] Support constexpr asm at global scope. (PR #143268)

2025-06-07 Thread Corentin Jabot via cfe-commits
https://github.com/cor3ntin created https://github.com/llvm/llvm-project/pull/143268 I previously failed to realize this feature existed... Fixes #137459 Fixes #143242 >From c09e49f84b189c51950842b4f79581ca78817919 Mon Sep 17 00:00:00 2001 From: Corentin Jabot Date: Sat, 7 Jun 2025 16:23:38 +

[clang] [Clang][Parse][NFC] Use `llvm::function_ref<>` instead of `std::optional>` (PR #142906)

2025-06-07 Thread Corentin Jabot via cfe-commits
https://github.com/cor3ntin closed https://github.com/llvm/llvm-project/pull/142906 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

  1   2   3   >