[clang-tools-extra] [clang-doc] Make `--repository` change the HTML output (PR #122566)

2025-02-06 Thread Paul Kirth via cfe-commits
https://github.com/ilovepi updated https://github.com/llvm/llvm-project/pull/122566 >From 29788ea8109c838e2ec3a62d2d14f396ba2f9ee6 Mon Sep 17 00:00:00 2001 From: Paul Kirth Date: Sat, 11 Jan 2025 01:21:54 + Subject: [PATCH] [clang-doc] Make `--repository` change the HTML output The current

[clang] [AMDGPU] Do not enable GPU sanitizers by default (PR #126090)

2025-02-06 Thread Yaxun Liu via cfe-commits
yxsamliu wrote: This may break ROCm santizer builds since whether santizer is on should be determined by -fsanitize only, i.e. by default -fgpu-sanitize should be on. https://github.com/llvm/llvm-project/pull/126090 ___ cfe-commits mailing list cfe-co

[clang] [llvm] [CaptureTracking][FunctionAttrs] Add support for CaptureInfo (PR #125880)

2025-02-06 Thread Nikita Popov via cfe-commits
@@ -438,18 +448,21 @@ void llvm::PointerMayBeCaptured(const Value *V, CaptureTracker *Tracker, }; while (!Worklist.empty()) { const Use *U = Worklist.pop_back_val(); -switch (DetermineUseCaptureKind(*U, IsDereferenceableOrNull)) { -case UseCaptureKind::NO_CAPTU

[clang] [clang][Sema] Move computing enum bits into a separate function (PR #126096)

2025-02-06 Thread Michael Buch via cfe-commits
Michael137 wrote: > This is just moving stuff around so that seems fine. Though, I wonder, could > we just pass e.g. an `ArrayRef` to `computeEnumBits()`? The LLDB pr > you linked seems to have an `SmallVector`, so that should > be possible (I think it does require a `reinterpret_cast` from a

[clang] [clang][Sema] Move computing enum bits into a separate function (PR #126096)

2025-02-06 Thread via cfe-commits
Sirraide wrote: > > > This is just moving stuff around so that seems fine. Though, I wonder, > > > could we just pass e.g. an `ArrayRef` to `computeEnumBits()`? The > > > LLDB pr you linked seems to have an `SmallVector`, so > > > that should be possible (I think it does require a `reinterpret

[clang-tools-extra] f8e53a9 - [clang-doc] Precommit end-to-end test for `--repository` option (#122565)

2025-02-06 Thread via cfe-commits
Author: Paul Kirth Date: 2025-02-06T11:06:47-08:00 New Revision: f8e53a93be91299818e761afa8effd8301999336 URL: https://github.com/llvm/llvm-project/commit/f8e53a93be91299818e761afa8effd8301999336 DIFF: https://github.com/llvm/llvm-project/commit/f8e53a93be91299818e761afa8effd8301999336.diff LO

[clang-tools-extra] [clang-doc] Precommit end-to-end test for `--repository` option (PR #122565)

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

[clang-tools-extra] [clang-doc] Make `--repository` change the HTML output (PR #122566)

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

[clang-tools-extra] [clang-doc] Make `--repository` change the HTML output (PR #122566)

2025-02-06 Thread Paul Kirth via cfe-commits
https://github.com/ilovepi updated https://github.com/llvm/llvm-project/pull/122566 >From 5a3e1df2bc73c314c4d919aa81a034c4efcf15b3 Mon Sep 17 00:00:00 2001 From: Paul Kirth Date: Sat, 11 Jan 2025 01:21:54 + Subject: [PATCH] [clang-doc] Make `--repository` change the HTML output The current

[clang] [clang][Sema] Move computing enum bits into a separate function (PR #126096)

2025-02-06 Thread Ilia Kuklin via cfe-commits
https://github.com/kuilpd edited https://github.com/llvm/llvm-project/pull/126096 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [compiler-rt] [Sanitizers] the access size (8 bytes) exceeds the max lock-free size (4 bytes) for 32-bit (PR #125388)

2025-02-06 Thread Hubert Tong via cfe-commits
https://github.com/hubert-reinterpretcast requested changes to this pull request. https://github.com/llvm/llvm-project/pull/125388 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [compiler-rt] [Sanitizers] the access size (8 bytes) exceeds the max lock-free size (4 bytes) for 32-bit (PR #125388)

2025-02-06 Thread Hubert Tong via cfe-commits
@@ -1120,3 +1120,19 @@ // RUN:-c \ // RUN: | FileCheck --check-prefixes=CHECK-K-UNUSED %s // CHECK-K-UNUSED: clang: warning: -K: 'linker' input unused [-Wunused-command-line-argument] + +// Check No Sanitizer on 32-bit AIX +// RUN: %clang -target powerpc-ibm-aix -m32

[clang] [lldb] Reland: [clang] fix P3310 overload resolution flag propagation (PR #125791)

2025-02-06 Thread Matheus Izvekov via cfe-commits
mizvekov wrote: > > > Actually, @mizvekov, can we make the > > > `ClassTemplateSpecializationDecl::ClassTemplateSpecializationDecl(ASTContext > > > &C, Kind DK)` constructor default initialize the `StrictPackMatch` > > > member to false? In that case users of `CreateDeserialized` wouldn't need

[clang] [llvm] [CaptureTracking][FunctionAttrs] Add support for CaptureInfo (PR #125880)

2025-02-06 Thread via cfe-commits
@@ -358,49 +363,54 @@ UseCaptureKind llvm::DetermineUseCaptureKind( // Volatile stores make the address observable. auto *ACXI = cast(I); if (U.getOperandNo() == 1 || U.getOperandNo() == 2 || ACXI->isVolatile()) - return UseCaptureKind::MAY_CAPTURE; -return

[clang] [RISCV][Sema] Add feature check for target attribute to VSETVL intrinsics (PR #126064)

2025-02-06 Thread Craig Topper via cfe-commits
@@ -623,13 +623,37 @@ bool SemaRISCV::CheckBuiltinFunctionCall(const TargetInfo &TI, } } + auto checkVsetvl = [&](unsigned SEWOffset, +unsigned LMULOffset) -> bool { topperc wrote: Is this formatted correctly? ht

[clang] [lldb] Reland: [clang] fix P3310 overload resolution flag propagation (PR #125791)

2025-02-06 Thread Michael Buch via cfe-commits
Michael137 wrote: > > > > Actually, @mizvekov, can we make the > > > > `ClassTemplateSpecializationDecl::ClassTemplateSpecializationDecl(ASTContext > > > > &C, Kind DK)` constructor default initialize the `StrictPackMatch` > > > > member to false? In that case users of `CreateDeserialized` wou

[clang] [compiler-rt] [Sanitizers] the access size (8 bytes) exceeds the max lock-free size (4 bytes) for 32-bit (PR #125388)

2025-02-06 Thread Honey Goyal via cfe-commits
https://github.com/honeygoyal updated https://github.com/llvm/llvm-project/pull/125388 >From f9d8e7f9c0df6beb8b4a63a01ebbc3b3ab93d091 Mon Sep 17 00:00:00 2001 From: Honey Goyal Date: Sun, 2 Feb 2025 14:27:01 +0530 Subject: [PATCH 01/18] Test Cases for adding -latomic (the access size (8 bytes)

[clang] [flang] [lld] [Flang] Don't use FortranDecimal for runtime (PR #121997)

2025-02-06 Thread Michael Kruse via cfe-commits
https://github.com/Meinersbur edited https://github.com/llvm/llvm-project/pull/121997 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[libclc] [libclc] Allow default path when looking for llvm-spirv (PR #126071)

2025-02-06 Thread Nikita Popov via cfe-commits
nikic wrote: Good point, I've changed this to use HINTS instead of PATHS. I'm not really sure which one is more "correct" in this context, but sticking closer to the previous behavior seems like a good idea... https://github.com/llvm/llvm-project/pull/126071 __

[clang] f7b3559 - [clang-linker-wrapper] Add ELF packaging for spirv64-intel OpenMP images (#125737)

2025-02-06 Thread via cfe-commits
Author: Nick Sarnie Date: 2025-02-06T15:46:44Z New Revision: f7b3559ce07c83625bbe81a30a4da8ccef9ab53f URL: https://github.com/llvm/llvm-project/commit/f7b3559ce07c83625bbe81a30a4da8ccef9ab53f DIFF: https://github.com/llvm/llvm-project/commit/f7b3559ce07c83625bbe81a30a4da8ccef9ab53f.diff LOG: [

[libclc] [libclc] Allow default path when looking for llvm-spirv (PR #126071)

2025-02-06 Thread Nikita Popov via cfe-commits
https://github.com/nikic updated https://github.com/llvm/llvm-project/pull/126071 >From 2badb9749472014dc02610a3a511b548bfe8a2ae Mon Sep 17 00:00:00 2001 From: Nikita Popov Date: Thu, 6 Feb 2025 14:56:25 +0100 Subject: [PATCH 1/2] [libclc] Allow default path when looking for llvm-spirv This is

[clang] Add clang atomic control options and attribute (PR #114841)

2025-02-06 Thread Yaxun Liu via cfe-commits
@@ -5404,6 +5404,151 @@ third argument, can only occur at file scope. a = b[i] * c[i] + e; } +Extensions for controlling atomic code generation += + +The ``[[clang::atomic]]`` statement attribute enables users to control ho

[clang] [llvm] [OpenMP][OpenMPIRBuilder] Add initial changes for SPIR-V target frontend support (PR #125920)

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

[clang] [Clang][P1061] Consolidate ResolvedUnpexandedPackExpr into FunctionParmPackExpr (PR #125394)

2025-02-06 Thread Jason Rice via cfe-commits
https://github.com/ricejasonf updated https://github.com/llvm/llvm-project/pull/125394 >From a323e058b2c8adf97f7f9a55a9187f74de9b8d17 Mon Sep 17 00:00:00 2001 From: Jason Rice Date: Sun, 2 Feb 2025 00:52:47 -0800 Subject: [PATCH 1/5] [Clang][P1061] Consolidate ResolvedUnexpandedPackExpr into F

[clang] [Clang] Forward arguments to the device compiler better (PR #125957)

2025-02-06 Thread Nick Sarnie via cfe-commits
@@ -9270,11 +9260,6 @@ void LinkerWrapper::ConstructJob(Compilation &C, const JobAction &JA, }); } - // If we disable the GPU C library support it needs to be forwarded to the - // link job. - if (!Args.hasFlag(options::OPT_gpulibc, options::OPT_nogpulibc, true)) ---

[clang] [llvm] [RISCV] Add a generic OOO CPU (PR #120712)

2025-02-06 Thread Yingwei Zheng via cfe-commits
https://github.com/dtcxzyw edited https://github.com/llvm/llvm-project/pull/120712 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [RISCV] Add a generic OOO CPU (PR #120712)

2025-02-06 Thread Yingwei Zheng via cfe-commits
https://github.com/dtcxzyw approved this pull request. LGTM. Thank you! Please wait for additional approval from other reviewers :) https://github.com/llvm/llvm-project/pull/120712 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.

[clang] [llvm] [RISCV] Add a generic OOO CPU (PR #120712)

2025-02-06 Thread Yingwei Zheng via cfe-commits
@@ -0,0 +1,500 @@ +//===-- RISCVSchedGenericOOO.td - Generic O3 Processor -*- tablegen -*-===// +// +// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. +// See https://llvm.org/LICENSE.txt for license information. +// SPDX-License-Identifier: Ap

[clang] [llvm] [RISCV] Add a generic OOO CPU (PR #120712)

2025-02-06 Thread Yingwei Zheng via cfe-commits
@@ -0,0 +1,500 @@ +//===-- RISCVSchedGenericOOO.td - Generic O3 Processor -*- tablegen -*-===// +// +// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. +// See https://llvm.org/LICENSE.txt for license information. +// SPDX-License-Identifier: Ap

[clang] [Clang] Forward arguments to the device compiler better (PR #125957)

2025-02-06 Thread Joseph Huber via cfe-commits
@@ -9270,11 +9260,6 @@ void LinkerWrapper::ConstructJob(Compilation &C, const JobAction &JA, }); } - // If we disable the GPU C library support it needs to be forwarded to the - // link job. - if (!Args.hasFlag(options::OPT_gpulibc, options::OPT_nogpulibc, true)) ---

[clang] [Clang] Forward arguments to the device compiler better (PR #125957)

2025-02-06 Thread Nick Sarnie via cfe-commits
https://github.com/sarnex approved this pull request. https://github.com/llvm/llvm-project/pull/125957 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [AArch64][SelectionDAG] Add CodeGen support for scalar FEAT_CPA (PR #105669)

2025-02-06 Thread Rodolfo Wottrich via cfe-commits
@@ -0,0 +1,451 @@ +; RUN: llc -mtriple=aarch64 -verify-machineinstrs --mattr=+cpa -O0 -global-isel=0 -fast-isel=0 %s -o - 2>&1 | FileCheck %s --check-prefixes=CHECK-CPA-O0 +; RUN: llc -mtriple=aarch64 -verify-machineinstrs --mattr=+cpa -O3 -global-isel=0 -fast-isel=0 %s -o - 2>

[clang] [llvm] [Clang][Driver] Add an option to control loop-interchange (PR #125830)

2025-02-06 Thread Madhur Amilkanthwar via cfe-commits
https://github.com/madhur13490 edited https://github.com/llvm/llvm-project/pull/125830 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [analyzer] Add -ftime-trace scopes for region-store bindings and removeDead (PR #125884)

2025-02-06 Thread Donát Nagy via cfe-commits
https://github.com/NagyDonat edited https://github.com/llvm/llvm-project/pull/125884 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [analyzer] Add option assume-one-iteration (PR #125494)

2025-02-06 Thread Endre Fülöp via cfe-commits
@@ -294,6 +294,16 @@ ANALYZER_OPTION( bool, ShouldUnrollLoops, "unroll-loops", "Whether the analysis should try to unroll loops with known bounds.", false) +ANALYZER_OPTION( +bool, ShouldAssumeOneIteration, "assume-one-iteration", +"Whether the analyzer should

[clang] [analyzer] Add option assume-one-iteration (PR #125494)

2025-02-06 Thread Endre Fülöp via cfe-commits
https://github.com/gamesh411 commented: IMO. This enables a heuristic that is closer to real-world code usage, and having this option is valuable. I left one comment inline. https://github.com/llvm/llvm-project/pull/125494 ___ cfe-commits mailing list

[clang] [analyzer] Add option assume-one-iteration (PR #125494)

2025-02-06 Thread Endre Fülöp via cfe-commits
https://github.com/gamesh411 edited https://github.com/llvm/llvm-project/pull/125494 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [analyzer] Add option assume-one-iteration (PR #125494)

2025-02-06 Thread Endre Fülöp via cfe-commits
https://github.com/gamesh411 edited https://github.com/llvm/llvm-project/pull/125494 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [analyzer] Add option assume-one-iteration (PR #125494)

2025-02-06 Thread Endre Fülöp via cfe-commits
https://github.com/gamesh411 edited https://github.com/llvm/llvm-project/pull/125494 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [analyzer] Add -ftime-trace scopes for region-store bindings and removeDead (PR #125884)

2025-02-06 Thread Donát Nagy via cfe-commits
https://github.com/NagyDonat edited https://github.com/llvm/llvm-project/pull/125884 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clang-tidy] Address false positives in misc-redundant-expression checker (PR #122841)

2025-02-06 Thread via cfe-commits
earnol wrote: Hello maintainers, It seems already week passed with no additional comments or approvals. Could you please review the commit and take an action? https://github.com/llvm/llvm-project/pull/122841 ___ cfe-commits mailing list cfe-commits@li

[clang] [Clang] Forward arguments to the device compiler better (PR #125957)

2025-02-06 Thread Joseph Huber via cfe-commits
https://github.com/jhuber6 updated https://github.com/llvm/llvm-project/pull/125957 >From 4a457c531ccd3766cc14352e517b0b99ae1e6dd8 Mon Sep 17 00:00:00 2001 From: Joseph Huber Date: Wed, 5 Feb 2025 17:21:02 -0600 Subject: [PATCH] [Clang] Forward arguments to the device compiler better Summary:

[clang] [llvm] [analyzer] Consolidate array bound checkers (PR #125534)

2025-02-06 Thread Donát Nagy via cfe-commits
NagyDonat wrote: > I assume you intentionally did _not_ rename the ArrayBoundCheckerV2.cpp to > ArrayBoundChecker.cpp to avoid the generating a diff that is much harder to > understand than just the modification and the deletion of the old one. So > consistency will be restored in trivial foll

[clang] [clang][doc] Provide documentation for FixedPointLiteral matcher (PR #125122)

2025-02-06 Thread via cfe-commits
earnol wrote: A gentle reminder to request a review for this PR since it is a week without a review. https://github.com/llvm/llvm-project/pull/125122 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listin

[libclc] [libclc] Move __clc_ldexp to CLC library (PR #126078)

2025-02-06 Thread Fraser Cormack via cfe-commits
https://github.com/frasercrmck created https://github.com/llvm/llvm-project/pull/126078 This function was already conceptually in the CLC namespace - this just formally moves it over. Note however that this commit marks a change in how libclc functions may be overridden by targets. Until now

[clang] [OpenMP][SIMD][FIX] Use conservative "omp simd ordered" lowering (PR #123867)

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

[clang] [compiler-rt] [Sanitizers] the access size (8 bytes) exceeds the max lock-free size (4 bytes) for 32-bit (PR #125388)

2025-02-06 Thread Honey Goyal via cfe-commits
@@ -1120,3 +1120,23 @@ // RUN:-c \ // RUN: | FileCheck --check-prefixes=CHECK-K-UNUSED %s // CHECK-K-UNUSED: clang: warning: -K: 'linker' input unused [-Wunused-command-line-argument] + + +// This test verifies that the linker doesn't include '-latomic' when no sani

[libclc] [libclc] Allow default path when looking for llvm-spirv (PR #126071)

2025-02-06 Thread Fraser Cormack via cfe-commits
https://github.com/frasercrmck approved this pull request. LGTM, thanks https://github.com/llvm/llvm-project/pull/126071 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [AArch64][SelectionDAG] Add CodeGen support for scalar FEAT_CPA (PR #105669)

2025-02-06 Thread Rodolfo Wottrich via cfe-commits
@@ -0,0 +1,451 @@ +; RUN: llc -mtriple=aarch64 -verify-machineinstrs --mattr=+cpa -O0 -global-isel=0 -fast-isel=0 %s -o - 2>&1 | FileCheck %s --check-prefixes=CHECK-CPA-O0 +; RUN: llc -mtriple=aarch64 -verify-machineinstrs --mattr=+cpa -O3 -global-isel=0 -fast-isel=0 %s -o - 2>

[clang] [llvm] [AArch64][SelectionDAG] Add CodeGen support for scalar FEAT_CPA (PR #105669)

2025-02-06 Thread Rodolfo Wottrich via cfe-commits
@@ -10382,6 +10382,26 @@ let Predicates = [HasCPA] in { // Scalar multiply-add/subtract def MADDPT : MulAccumCPA<0, "maddpt">; def MSUBPT : MulAccumCPA<1, "msubpt">; + + // Rules to use CPA instructions in pointer arithmetic patterns which are not + // folded into loads

[clang] [analyzer] Add -ftime-trace scopes for region-store bindings and removeDead (PR #125884)

2025-02-06 Thread Balazs Benics via cfe-commits
@@ -1031,6 +1032,7 @@ void ExprEngine::removeDead(ExplodedNode *Pred, ExplodedNodeSet &Out, const LocationContext *LC, const Stmt *DiagnosticStmt, ProgramPoint::Kind K) { + llvm::TimeTraceScop

[clang] [clang][Sema] Fix type of an statement expression ending with an atomic type (PR #119711)

2025-02-06 Thread Aaron Ballman via cfe-commits
Alejandro =?utf-8?q?Álvarez_Ayllón?=, Alejandro =?utf-8?q?Álvarez_Ayllón?=, Alejandro =?utf-8?q?Álvarez_Ayllón?Message-ID: In-Reply-To: @@ -15919,10 +15919,17 @@ ExprResult Sema::ActOnStmtExprResult(ExprResult ER) { if (Cast && Cast->getCastKind() == CK_ARCConsumeObject)

[clang] [flang] [flang] Add -f[no-]vectorize flags (PR #119718)

2025-02-06 Thread David Truby via cfe-commits
https://github.com/DavidTruby updated https://github.com/llvm/llvm-project/pull/119718 >From 0dc613d94560cbe4e8a57eed35d985e9d6dae752 Mon Sep 17 00:00:00 2001 From: David Truby Date: Thu, 12 Dec 2024 14:50:19 + Subject: [PATCH] [flang] Add -f[no-]vectorize flags --- clang/include/clang/Dr

[clang] [analyzer] Add option assume-one-iteration (PR #125494)

2025-02-06 Thread Balazs Benics via cfe-commits
https://github.com/steakhal approved this pull request. Reviewed the changes and it looks correct. Please change the flag name (along with the comments to `assume-at-least-one-iteration`. Please also review Edre's comments, those appear relevant. https://github.com/llvm/llvm-project/pull/12549

[clang] [flang] [flang] Add -f[no-]vectorize flags (PR #119718)

2025-02-06 Thread David Truby via cfe-commits
DavidTruby wrote: Ugh it looks like my editor just clang-formatted everything instead of only changes... https://github.com/llvm/llvm-project/pull/119718 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/li

[clang] [clang] Implement __attribute__((format_matches)) (PR #116708)

2025-02-06 Thread Erich Keane via cfe-commits
=?utf-8?q?Félix?= Cloutier , =?utf-8?q?Félix?= Cloutier , =?utf-8?q?Félix?= Cloutier , =?utf-8?q?Félix?= Cloutier , =?utf-8?q?Félix?= Cloutier , =?utf-8?q?Félix?= Cloutier , =?utf-8?q?Félix?= Cloutier , =?utf-8?q?Félix?= Cloutier , =?utf-8?q?Félix?= Cloutier Message-ID: In-Reply-To: https://git

[clang] [clang] Implement __attribute__((format_matches)) (PR #116708)

2025-02-06 Thread Erich Keane via cfe-commits
=?utf-8?q?Félix?= Cloutier , =?utf-8?q?Félix?= Cloutier , =?utf-8?q?Félix?= Cloutier , =?utf-8?q?Félix?= Cloutier , =?utf-8?q?Félix?= Cloutier , =?utf-8?q?Félix?= Cloutier , =?utf-8?q?Félix?= Cloutier , =?utf-8?q?Félix?= Cloutier , =?utf-8?q?Félix?= Cloutier Message-ID: In-Reply-To: ===

[clang] [clang] Implement __attribute__((format_matches)) (PR #116708)

2025-02-06 Thread Erich Keane via cfe-commits
=?utf-8?q?Félix?= Cloutier , =?utf-8?q?Félix?= Cloutier , =?utf-8?q?Félix?= Cloutier , =?utf-8?q?Félix?= Cloutier , =?utf-8?q?Félix?= Cloutier , =?utf-8?q?Félix?= Cloutier , =?utf-8?q?Félix?= Cloutier , =?utf-8?q?Félix?= Cloutier , =?utf-8?q?Félix?= Cloutier Message-ID: In-Reply-To: ===

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-02-06 Thread Ryan Mast via cfe-commits
@@ -0,0 +1,36 @@ +[build-system] +requires = ["setuptools>=42", "setuptools_scm==8.1.0"] +build-backend = "setuptools.build_meta" + +[project] +name = "clang" +description = "libclang python bindings" +readme = {file = "README.txt", content-type = "text/plain"} + +license = {text

[clang] [verify] Improve the error messages with multiple active prefixes (PR #126068)

2025-02-06 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 39be2d0266f1aa229a79c47d81b004c8c2079362 105e5467662248ea2eb41edcfe0ce9a0cc97d044 --e

[clang] [llvm] [Clang][Driver] Add an option to control loop-interchange (PR #125830)

2025-02-06 Thread Madhur Amilkanthwar via cfe-commits
https://github.com/madhur13490 approved this pull request. LGTM https://github.com/llvm/llvm-project/pull/125830 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Remove __is_referenceable builtin (PR #123078)

2025-02-06 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `lldb-remote-linux-ubuntu` running on `as-builder-9` while building `clang` at step 16 "test-check-lldb-api". Full details are available at: https://lab.llvm.org/buildbot/#/builders/195/builds/4539 Here is the relevant piece

[clang] [clang] Implement __attribute__((format_matches)) (PR #116708)

2025-02-06 Thread via cfe-commits
=?utf-8?q?Félix?= Cloutier , =?utf-8?q?Félix?= Cloutier , =?utf-8?q?Félix?= Cloutier , =?utf-8?q?Félix?= Cloutier , =?utf-8?q?Félix?= Cloutier , =?utf-8?q?Félix?= Cloutier , =?utf-8?q?Félix?= Cloutier , =?utf-8?q?Félix?= Cloutier , =?utf-8?q?Félix?= Cloutier Message-ID: In-Reply-To: ===

[clang] [clang] Implement __attribute__((format_matches)) (PR #116708)

2025-02-06 Thread via cfe-commits
=?utf-8?q?Félix?= Cloutier , =?utf-8?q?Félix?= Cloutier , =?utf-8?q?Félix?= Cloutier , =?utf-8?q?Félix?= Cloutier , =?utf-8?q?Félix?= Cloutier , =?utf-8?q?Félix?= Cloutier , =?utf-8?q?Félix?= Cloutier , =?utf-8?q?Félix?= Cloutier , =?utf-8?q?Félix?= Cloutier Message-ID: In-Reply-To: https://git

[clang] [llvm] [RISCV] Add a generic OOO CPU (PR #120712)

2025-02-06 Thread Craig Topper via cfe-commits
@@ -0,0 +1,500 @@ +//===-- RISCVSchedGenericOOO.td - Generic O3 Processor -*- tablegen -*-===// +// +// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. +// See https://llvm.org/LICENSE.txt for license information. +// SPDX-License-Identifier: Ap

[clang] [llvm] [RISCV] Add a generic OOO CPU (PR #120712)

2025-02-06 Thread Craig Topper via cfe-commits
@@ -0,0 +1,494 @@ +//===-- RISCVSchedGenericOOO.td - Generic O3 Processor -*- tablegen -*-===// +// +// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. +// See https://llvm.org/LICENSE.txt for license information. +// SPDX-License-Identifier: Ap

[clang] [llvm] [clang-linker-wrapper] Add ELF packaging for spirv64-intel OpenMP images (PR #125737)

2025-02-06 Thread Nick Sarnie via cfe-commits
sarnex wrote: what should i do in that case? can i just ignore the failure or is there a way to clear the build cache? https://github.com/llvm/llvm-project/pull/125737 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-

[clang] [llvm] [clang-linker-wrapper] Add ELF packaging for spirv64-intel OpenMP images (PR #125737)

2025-02-06 Thread Joseph Huber via cfe-commits
jhuber6 wrote: > yeah my test should only run if `LLVM_INCLUDE_SPIRV_TOOLS_TESTS` is set, > which does not appear to be the case, so i'm surprised it's running. it > indeed did not run in the clean build. > > ``` > UNSUPPORTED: Clang :: Tooling/clang-linker-wrapper-spirv-elf.cpp (20383 of > 8

[clang] [llvm] [clang-linker-wrapper] Add ELF packaging for spirv64-intel OpenMP images (PR #125737)

2025-02-06 Thread Nick Sarnie via cfe-commits
sarnex wrote: yeah my test should only run if `LLVM_INCLUDE_SPIRV_TOOLS_TESTS` is set, which does not appear to be the case, so i'm surprised it's running. it indeed did not run in the clean build. ``` UNSUPPORTED: Clang :: Tooling/clang-linker-wrapper-spirv-elf.cpp (20383 of 84127) ``` the

[clang] [HLSL] Appropriately set function attribute optnone (PR #125937)

2025-02-06 Thread S. Bharadwaj Yadavalli via cfe-commits
https://github.com/bharadwajy updated https://github.com/llvm/llvm-project/pull/125937 >From 63a728dd7f5efff32a2f90608adfdc13d540f34f Mon Sep 17 00:00:00 2001 From: "S. Bharadwaj Yadavalli" Date: Tue, 4 Feb 2025 12:48:09 -0500 Subject: [PATCH] [HLSL] Set function optnone attribute appropriately

[clang] [HLSL] Appropriately set function attribute optnone (PR #125937)

2025-02-06 Thread S. Bharadwaj Yadavalli via cfe-commits
https://github.com/bharadwajy ready_for_review https://github.com/llvm/llvm-project/pull/125937 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [clang-linker-wrapper] Add ELF packaging for spirv64-intel OpenMP images (PR #125737)

2025-02-06 Thread Joseph Huber via cfe-commits
jhuber6 wrote: > what should i do in that case? can i just ignore the failure or is there a > way to clear the build cache? I got the failure locally as well, unsure, maybe there's a dependency missing somewhere. https://github.com/llvm/llvm-project/pull/125737 ___

[clang] [llvm] [clang-linker-wrapper] Add ELF packaging for spirv64-intel OpenMP images (PR #125737)

2025-02-06 Thread Nick Sarnie via cfe-commits
sarnex wrote: ok cool if you can repro it it's almost surely a real issue. can you share the repro steps, my local repro has it as unsupported as i expect https://github.com/llvm/llvm-project/pull/125737 ___ cfe-commits mailing list cfe-commits@lists.

[clang] [clang] Implement __attribute__((format_matches)) (PR #116708)

2025-02-06 Thread via cfe-commits
=?utf-8?q?Félix?= Cloutier , =?utf-8?q?Félix?= Cloutier , =?utf-8?q?Félix?= Cloutier , =?utf-8?q?Félix?= Cloutier , =?utf-8?q?Félix?= Cloutier , =?utf-8?q?Félix?= Cloutier , =?utf-8?q?Félix?= Cloutier , =?utf-8?q?Félix?= Cloutier , =?utf-8?q?Félix?= Cloutier Message-ID: In-Reply-To: https://git

[clang] [llvm] [clang-linker-wrapper] Add ELF packaging for spirv64-intel OpenMP images (PR #125737)

2025-02-06 Thread Nick Sarnie via cfe-commits
sarnex wrote: yes i will revert if i dont have a fix pr in 30 minutes https://github.com/llvm/llvm-project/pull/125737 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [HLSL] Appropriately set function attribute optnone (PR #125937)

2025-02-06 Thread S. Bharadwaj Yadavalli via cfe-commits
bharadwajy wrote: > Need to verify a couple more function attribute combinations. Changed it to > draft mode till that is done. > > Please hold off reviews. PR ready for review. Thanks! https://github.com/llvm/llvm-project/pull/125937 ___ cfe-commit

[clang] [llvm] [HLSL] [DXIL] Implement the `AddUint64` HLSL function and the `UAddc` DXIL op (PR #125319)

2025-02-06 Thread Deric Cheung via cfe-commits
@@ -19105,6 +19105,51 @@ Value *CodeGenFunction::EmitHLSLBuiltinExpr(unsigned BuiltinID, return nullptr; switch (BuiltinID) { + case Builtin::BI__builtin_hlsl_adduint64: { Icohedron wrote: Do you think it is something I should do for this implementati

[clang] [llvm] [clang-linker-wrapper] Add ELF packaging for spirv64-intel OpenMP images (PR #125737)

2025-02-06 Thread David Spickett via cfe-commits
DavidSpickett wrote: > I have seen this before https://github.com/llvm/llvm-project/commit/a2bd5db00d07bc28fcfb3350b9d50e72e6350cba https://github.com/llvm/llvm-project/pull/125737 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.

[clang] [Clang][P1061] Consolidate ResolvedUnpexandedPackExpr into FunctionParmPackExpr (PR #125394)

2025-02-06 Thread Jason Rice via cfe-commits
@@ -3492,10 +3492,13 @@ VarDecl *BindingDecl::getHoldingVar() const { return VD; } -llvm::ArrayRef BindingDecl::getBindingPackExprs() const { +llvm::ArrayRef BindingDecl::getBindingPackDecls() const { assert(Binding && "expecting a pack expr"); - auto *RP = cast(Binding)

[clang] [llvm] [clang-linker-wrapper] Add ELF packaging for spirv64-intel OpenMP images (PR #125737)

2025-02-06 Thread Nick Sarnie via cfe-commits
sarnex wrote: thanks a lot, i think i see the problem, but would help if i could repro :P https://github.com/llvm/llvm-project/pull/125737 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commi

[clang] [llvm] [clang-linker-wrapper] Add ELF packaging for spirv64-intel OpenMP images (PR #125737)

2025-02-06 Thread via cfe-commits
dyung wrote: If you cannot fix this soon, can you revert to get the bots back to green while you investigate? https://github.com/llvm/llvm-project/pull/125737 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailm

[clang] [compiler-rt] [Sanitizers] the access size (8 bytes) exceeds the max lock-free size (4 bytes) for 32-bit (PR #125388)

2025-02-06 Thread Hubert Tong via cfe-commits
@@ -1120,3 +1120,19 @@ // RUN:-c \ // RUN: | FileCheck --check-prefixes=CHECK-K-UNUSED %s // CHECK-K-UNUSED: clang: warning: -K: 'linker' input unused [-Wunused-command-line-argument] + +// Check No Sanitizer on 32-bit AIX +// RUN: %clang -target powerpc-ibm-aix -m32

[clang] [Clang][NFC] Clean up fetching the offloading toolchain (PR #125095)

2025-02-06 Thread Joseph Huber via cfe-commits
https://github.com/jhuber6 updated https://github.com/llvm/llvm-project/pull/125095 >From 11ae495099dac5536e1cfbae77385fca41983577 Mon Sep 17 00:00:00 2001 From: Joseph Huber Date: Thu, 30 Jan 2025 11:28:50 -0600 Subject: [PATCH 1/2] [Clang] Clean up fetching the offloading toolchain Summary:

[clang] [clang] [test] Adjust XFAILs for windows in unique_object_duplication.cpp (PR #126139)

2025-02-06 Thread Martin Storsjö via cfe-commits
https://github.com/mstorsjo created https://github.com/llvm/llvm-project/pull/126139 This test does succeed in MinGW configurations. From 30a0f8b46a55965a0ebbab066cf62b77f6ab9e5c Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Martin=20Storsj=C3=B6?= Date: Thu, 6 Feb 2025 23:18:07 +0200 Subject: [PAT

[clang] [clang] [test] Adjust XFAILs for windows in unique_object_duplication.cpp (PR #126139)

2025-02-06 Thread Martin Storsjö via cfe-commits
mstorsjo wrote: CC @DKLoehr https://github.com/llvm/llvm-project/pull/126139 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang] [test] Adjust XFAILs for windows in unique_object_duplication.cpp (PR #126139)

2025-02-06 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Martin Storsjö (mstorsjo) Changes This test does succeed in MinGW configurations. --- Full diff: https://github.com/llvm/llvm-project/pull/126139.diff 1 Files Affected: - (modified) clang/test/SemaCXX/unique_object_duplication.cpp (+2-2

[clang] default clause replaced by otherwise clause for metadirective in OpenMP 5.2 (PR #125648)

2025-02-06 Thread Urvi Rav via cfe-commits
https://github.com/ravurvi20 updated https://github.com/llvm/llvm-project/pull/125648 >From 189dd3cc2230ea5752969f02f119b6ee30e3df69 Mon Sep 17 00:00:00 2001 From: Urvi Rav Date: Tue, 4 Feb 2025 01:35:41 -0600 Subject: [PATCH 1/4] default clause replaced by otherwise clause for metadirective

[clang] [RISCV][Sema] Add feature check for target attribute to VSETVL intrinsics (PR #126064)

2025-02-06 Thread Brandon Wu via cfe-commits
@@ -623,13 +623,37 @@ bool SemaRISCV::CheckBuiltinFunctionCall(const TargetInfo &TI, } } + auto checkVsetvl = [&](unsigned SEWOffset, 4vtomat wrote: I see, let me change the case https://github.com/llvm/llvm-project/pull/126064

[clang] [clang] Add 'instantiated_from' for enums to the output of TextNodeDumper (PR #124409)

2025-02-06 Thread André Brand via cfe-commits
thebrandre wrote: @shafik @Sirraide I rechecked everything and I fixed some non-relative line numbers. I only just noticed now that I probably should have checked that by inserting some line numbers just before my test code to see if this doesn't break the test. 🙈 If there is still something

[clang] [compiler-rt] [profile] Add `%b` `LLVM_PROFILE_FILE` option for binary ID (PR #123963)

2025-02-06 Thread via cfe-commits
nataraj-hates-MS-for-stealing-github wrote: Hi! I see this patch is already merged, but sine the most testing work is already done, I will still write my report here. I am original reporter of #51560. @ArtSin asked me to test the patch. TL;TR: `%b` solves my problem, but `%p` still breaks eve

[clang] [clang-format] Improve function pointer CastRParen detection. (PR #126019)

2025-02-06 Thread Owen Pan via cfe-commits
@@ -874,6 +874,11 @@ TEST_F(TokenAnnotatorTest, UnderstandsCasts) { EXPECT_TOKEN(Tokens[14], tok::r_paren, TT_CastRParen); EXPECT_TOKEN(Tokens[15], tok::amp, TT_UnaryOperator); + Tokens = annotate("func((foo(bar::*)(void))&a);"); owenca wrote: What's `fo

[clang] [clang] Add 'instantiated_from' for enums to the output of TextNodeDumper (PR #124409)

2025-02-06 Thread André Brand via cfe-commits
https://github.com/thebrandre updated https://github.com/llvm/llvm-project/pull/124409 From 1c829c6abcf6d2152088dce7799253bf94b77d7d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Andr=C3=A9=20Brand?= Date: Wed, 22 Jan 2025 19:23:06 +0100 Subject: [PATCH 1/5] [clang] Add 'instantiated_from' for enums

[clang] [clang] Add 'instantiated_from' for enums to the output of TextNodeDumper (PR #124409)

2025-02-06 Thread André Brand via cfe-commits
https://github.com/thebrandre updated https://github.com/llvm/llvm-project/pull/124409 From 1c829c6abcf6d2152088dce7799253bf94b77d7d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Andr=C3=A9=20Brand?= Date: Wed, 22 Jan 2025 19:23:06 +0100 Subject: [PATCH 1/5] [clang] Add 'instantiated_from' for enums

[clang] Revert "[HLSL] Implement HLSL Flat casting (excluding splat cases)" (PR #126149)

2025-02-06 Thread Sarah Spall via cfe-commits
spall wrote: > Hi, > > We usually ask to include the reason for the revert in the commit message. I apologize, there was a buildbot failure which seemed to be caused by the reverted commit. https://github.com/llvm/llvm-project/pull/126149 ___ cfe-c

[clang] [flang] [flang][OpenMP] Upstream first part of `do concurrent` mapping (PR #126026)

2025-02-06 Thread Damian Rouson via cfe-commits
rouson wrote: Thanks for submitting this, @ergawy. I'll test this. The mentioned performance results are in the slides [here](https://go.lbl.gov/just-write-fortran). https://github.com/llvm/llvm-project/pull/126026 ___ cfe-commits mailing list cfe-

[clang] [llvm] [mlir] [NVPTX] Convert scalar function nvvm.annotations to attributes (PR #125908)

2025-02-06 Thread Alex MacLean via cfe-commits
@@ -179,6 +179,13 @@ static bool argHasNVVMAnnotation(const Value &Val, return false; } +static std::optional getFnAttrParsedInt(const Function &F, + StringRef Attr) { + return F.hasFnAttribute(Attr) + ? std::opti

[clang] [clang] Assert the enum FPOpts and LangOpts fit into the storage (PR #126166)

2025-02-06 Thread Florian Mayer via cfe-commits
https://github.com/fmayer created https://github.com/llvm/llvm-project/pull/126166 Fix existing failure >From 85540ede2e0ba1b6f0b8a69ac6d5a6ddf56fdfc8 Mon Sep 17 00:00:00 2001 From: Florian Mayer Date: Thu, 6 Feb 2025 17:41:59 -0800 Subject: [PATCH] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=9

[clang] [clang] Assert the enum FPOpts and LangOpts fit into the storage (PR #126166)

2025-02-06 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 163ccfac33201948eb430db6fac38d265958cfaa 85540ede2e0ba1b6f0b8a69ac6d5a6ddf56fdfc8 --e

[clang] [clang] Assert the enum FPOpts and LangOpts fit into the storage (PR #126166)

2025-02-06 Thread Florian Mayer via cfe-commits
https://github.com/fmayer updated https://github.com/llvm/llvm-project/pull/126166 >From 85540ede2e0ba1b6f0b8a69ac6d5a6ddf56fdfc8 Mon Sep 17 00:00:00 2001 From: Florian Mayer Date: Thu, 6 Feb 2025 17:41:59 -0800 Subject: [PATCH 1/2] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20in?= =?UTF

[clang] [llvm] [analyzer] Consolidate array bound checkers (PR #125534)

2025-02-06 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `lld-x86_64-win` running on `as-worker-93` while building `clang,llvm` at step 7 "test-build-unified-tree-check-all". Full details are available at: https://lab.llvm.org/buildbot/#/builders/146/builds/2243 Here is the releva

<    1   2   3   4   5   6   >