[clang-tools-extra] [clang-tidy] Add an option in 'readability-named-parameter' to print names without comment (PR #147953)

https://github.com/dmpolukhin closed https://github.com/llvm/llvm-project/pull/147953 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [X86] Remove WIDEKL feature from Pantherlake and Clearwaterforest (PR #148184)

llvmbot wrote: @llvm/pr-subscribers-backend-x86 Author: Phoebe Wang (phoebewang) Changes According to Intel [SDM088](https://cdrdv2.intel.com/v1/dl/getContent/671200) Volume 1, Chapter 2.4, Key Locker will be removed from 2025 onwards products. --- Full diff: https://github.com/llvm/llvm

[clang] [llvm] [X86] Remove WIDEKL feature from Pantherlake and Clearwaterforest (PR #148184)

llvmbot wrote: @llvm/pr-subscribers-clang Author: Phoebe Wang (phoebewang) Changes According to Intel [SDM088](https://cdrdv2.intel.com/v1/dl/getContent/671200) Volume 1, Chapter 2.4, Key Locker will be removed from 2025 onwards products. --- Full diff: https://github.com/llvm/llvm-proje

[clang] [llvm] [X86] Remove WIDEKL feature from Pantherlake and Clearwaterforest (PR #148184)

https://github.com/phoebewang created https://github.com/llvm/llvm-project/pull/148184 According to Intel [SDM088](https://cdrdv2.intel.com/v1/dl/getContent/671200) Volume 1, Chapter 2.4, Key Locker will be removed from 2025 onwards products. >From 9a8224c22535912a1fa95caa41f8fc2c3f7a76d0 Mon

[clang] 0e5f9f6 - [clang][bytecode] Keep a list of initializing blocks in InterpState (#148120)

Author: Timm Baeder Date: 2025-07-11T09:49:47+02:00 New Revision: 0e5f9f67cd990e55aaa64472c5fcc2117ac7ae33 URL: https://github.com/llvm/llvm-project/commit/0e5f9f67cd990e55aaa64472c5fcc2117ac7ae33 DIFF: https://github.com/llvm/llvm-project/commit/0e5f9f67cd990e55aaa64472c5fcc2117ac7ae33.diff L

[clang] [clang][bytecode] Keep a list of initializing blocks in InterpState (PR #148120)

https://github.com/tbaederr closed https://github.com/llvm/llvm-project/pull/148120 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang][bytecode] Allocate Record fields and bases via Program (PR #147909)

https://github.com/tbaederr edited https://github.com/llvm/llvm-project/pull/147909 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clang-tidy][NFC] add '.clang-tidy' config for clang-tidy project (PR #147793)

carlosgalvezp wrote: > I will add it in a separate PR. Don't feel right to make kind-of unrelated > changes after most of the people gave their consent. Ok! https://github.com/llvm/llvm-project/pull/147793 ___ cfe-commits mailing list cfe-commits@li

[clang-tools-extra] [clang-tidy][NFC] add '.clang-tidy' config for clang-tidy project (PR #147793)

https://github.com/carlosgalvezp approved this pull request. https://github.com/llvm/llvm-project/pull/147793 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang][Driver] Revise Cygwin ToolChain to call linker directly (PR #147960)

https://github.com/tyan0 updated https://github.com/llvm/llvm-project/pull/147960 >From efab30c8f540d6aff0bcb0ebec22b633e690de8c Mon Sep 17 00:00:00 2001 From: Takashi Yano Date: Thu, 10 Jul 2025 21:55:05 +0900 Subject: [PATCH] [Clang][Driver] Revise Cygwin ToolChain to call linker directly .

[clang] [llvm] [OptBisect][IR] Adding a new OptPassGate for disabling passes via name (PR #145059)

https://github.com/cristianassaiante updated https://github.com/llvm/llvm-project/pull/145059 >From ab6063493744ef5a1ee92fd249bf8d86b7299fad Mon Sep 17 00:00:00 2001 From: Cristian Assaiante Date: Fri, 20 Jun 2025 16:56:23 +0200 Subject: [PATCH 01/15] Adding -opt-disable and a test for it ---

[clang] [Clang][Driver] Revise Cygwin ToolChain to call linker directly (PR #147960)

@@ -107,3 +110,289 @@ void Cygwin::AddClangSystemIncludeArgs(const ArgList &DriverArgs, addExternCSystemInclude(DriverArgs, CC1Args, SysRoot + "/usr/include"); addExternCSystemInclude(DriverArgs, CC1Args, SysRoot + "/usr/include/w32api"); } + +static bool getStaticPIE(con

[clang] [Clang][Driver] Revise Cygwin ToolChain to call linker directly (PR #147960)

@@ -1504,6 +1506,7 @@ void ToolChain::AddCXXStdlibLibArgs(const ArgList &Args, switch (Type) { case ToolChain::CST_Libcxx: CmdArgs.push_back("-lc++"); +CmdArgs.push_back("-lc++abi"); tyan0 wrote: I'm not sure. `libc++.dll.a` depends `libc++abi.dll.

[clang] [Clang][Driver] Revise Cygwin ToolChain to call linker directly (PR #147960)

@@ -107,3 +110,289 @@ void Cygwin::AddClangSystemIncludeArgs(const ArgList &DriverArgs, addExternCSystemInclude(DriverArgs, CC1Args, SysRoot + "/usr/include"); addExternCSystemInclude(DriverArgs, CC1Args, SysRoot + "/usr/include/w32api"); } + +static bool getStaticPIE(con

[clang] [llvm] [Clang-Repl] Add custom function as lambda in launchExecutor and fetch PID of launched executor (PR #147478)

@@ -91,7 +95,8 @@ createSharedMemoryManager(SimpleRemoteEPC &SREPC, Expected> launchExecutor(StringRef ExecutablePath, bool UseSharedMemory, - llvm::StringRef SlabAllocateSizeString) { + llvm::StringRef SlabAllocateSizeString, + std::f

[clang] [KeyInstr] Disable key-instructions for coroutine scopes (PR #147551)

https://github.com/OCHyams edited https://github.com/llvm/llvm-project/pull/147551 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [KeyInstr] Disable key-instructions for coroutine scopes (PR #147551)

@@ -0,0 +1,81 @@ +// RUN: %clang_cc1 -disable-llvm-optzns -std=c++20 \ +// RUN:-triple=x86_64 -dwarf-version=4 -debug-info-kind=limited \ +// RUN:-emit-llvm -o - %s -gkey-instructions | \ +// RUN:FileCheck %s + +// Check that for the coroutine b

[clang] [KeyInstr] Disable key-instructions for coroutine scopes (PR #147551)

https://github.com/OCHyams edited https://github.com/llvm/llvm-project/pull/147551 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [KeyInstr] Disable key-instructions for coroutine scopes (PR #147551)

https://github.com/OCHyams approved this pull request. LGTM, thanks https://github.com/llvm/llvm-project/pull/147551 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang-tools-extra] [lldb] [Clang][AST][NFC] (`RecordDecl` -> `CXXRecordDecl`)`::isInjectedClassName` (PR #148195)

llvmbot wrote: @llvm/pr-subscribers-lldb Author: Yanzuo Liu (zwuis) Changes Co-authored-by: Matheus Izvekov --- Full diff: https://github.com/llvm/llvm-project/pull/148195.diff 10 Files Affected: - (modified) clang-tools-extra/clangd/CodeComplete.cpp (+1-1) - (mod

[clang] [clang-tools-extra] [lldb] [Clang][AST][NFC] (`RecordDecl` -> `CXXRecordDecl`)`::isInjectedClassName` (PR #148195)

llvmbot wrote: @llvm/pr-subscribers-clang-tools-extra @llvm/pr-subscribers-clangd Author: Yanzuo Liu (zwuis) Changes Co-authored-by: Matheus Izvekov --- Full diff: https://github.com/llvm/llvm-project/pull/148195.diff 10 Files Affected: - (modified) clang-tools-ext

[clang] [clang-tools-extra] [lldb] [Clang][AST][NFC] (`RecordDecl` -> `CXXRecordDecl`)`::isInjectedClassName` (PR #148195)

https://github.com/zwuis created https://github.com/llvm/llvm-project/pull/148195 Co-authored-by: Matheus Izvekov >From 306049aa7dd17f6683935d631b3ad222b268a3f2 Mon Sep 17 00:00:00 2001 From: Yanzuo Liu Date: Fri, 11 Jul 2025 18:17:05 +0800 Subject: [PATCH] (`RecordDecl` -> `CXXRecordDecl`)`:

[clang] [Clang][Driver] Revise Cygwin ToolChain to call linker directly (PR #147960)

kikairoya wrote: Thanks. It looks like some linker options might still be missing or need adjustments: - `--enable-auto-import`, `--disable-high-entropy-va`, and `--dll-search-prefix=cyg` are generally required on Cygwin. - `-pie` has no effect when targeting PE/COFF and can be ignored. Also,

[clang] [clang-tools-extra] [lldb] [Clang][AST][NFC] (`RecordDecl` -> `CXXRecordDecl`)`::isInjectedClassName` (PR #148195)

zwuis wrote: I'm not sure if I should update `llvm/include/llvm/Testing/Demangle/DemangleTestCases.inc` and `libcxxabi/test/DemangleTestCases.inc`. https://github.com/llvm/llvm-project/pull/148195 ___ cfe-commits mailing list cfe-commits@lists.llvm.o

[clang] [clang-tools-extra] [lldb] [Clang][AST][NFC] (`RecordDecl` -> `CXXRecordDecl`)`::isInjectedClassName` (PR #148195)

cor3ntin wrote: Can you provide a more detailed description? Thanks! https://github.com/llvm/llvm-project/pull/148195 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] AMDGPU: Remove "gws" from the “read-only” target feature list (PR #148141)

llvmbot wrote: @llvm/pr-subscribers-clang @llvm/pr-subscribers-backend-amdgpu Author: Changpeng Fang (changpeng) Changes Since this feature is no longer universally available, we may need it in the IR. Fixes: SWDEV-541399 --- Patch is 24.65 KiB, truncated to 20.00 KiB below, full vers

[clang] AMDGPU: Remove "gws" from the “read-only” target feature list (PR #148141)

https://github.com/changpeng created https://github.com/llvm/llvm-project/pull/148141 Since this feature is no longer universally available, we may need it in the IR. Fixes: SWDEV-541399 >From c9cf9873e35205f9715acd545680713c0dd912aa Mon Sep 17 00:00:00 2001 From: Changpeng Fang Date: Fri,

[clang] [llvm] [OptBisect][IR] Adding a new OptPassGate for disabling passes via name (PR #145059)

https://github.com/cristianassaiante updated https://github.com/llvm/llvm-project/pull/145059 >From ab6063493744ef5a1ee92fd249bf8d86b7299fad Mon Sep 17 00:00:00 2001 From: Cristian Assaiante Date: Fri, 20 Jun 2025 16:56:23 +0200 Subject: [PATCH 01/15] Adding -opt-disable and a test for it ---

[clang] [Clang][Driver] Revise Cygwin ToolChain to call linker directly (PR #147960)

tyan0 wrote: > Could you let I know output of `echo | clang -xc - -v` (and some option > variants) with patch? ``` $ echo | clang -xc - -v clang version 20.1.7 (ssh://ty...@cygwin.com/git/cygwin-packages/clang 9fadb5f0ea7d31bddc623bbd3c6a9beff4d235fc) Target: x86_64-pc-windows-cygnus Thread mo

[clang] [Clang][CodeGen] Emit “trap reasons” on UBSan traps (PR #145967)

@@ -17,8 +18,11 @@ void bar(volatile int a) __attribute__((optnone)) { } // With optimisations enabled the traps are merged and need to share a debug location -// CHECK: [[LOC]] = !DILocation(line: 0 +// CHECK: [[LOC]] = !DILocation(line: 0, anthonyhatran wro

[clang] [Clang][CodeGen] Emit “trap reasons” on UBSan traps (PR #145967)

@@ -87,38 +87,42 @@ void baz(void (*fn)(int, int, int), int arg1, int arg2, int arg3) { // CHECK: [[DBG24]] = !DILocation(line: 25, column: 5, scope: [[DBG7]]) // CHECK: [[META25]] = !{} // CHECK: [[PROF26]] = !{!"branch_weights", i32 1048575, i32 1} -// CHECK: [[DBG27]] = !DI

[clang] [Clang][CodeGen] Emit “trap reasons” on UBSan traps (PR #145967)

@@ -0,0 +1,18 @@ +// RUN: %clang_cc1 -triple arm64-apple-macosx14.0.0 -O0 -debug-info-kind=standalone -dwarf-version=5 \ +// RUN: -fsanitize=nullability-return -fsanitize-trap=nullability-return -emit-llvm %s -o - | FileCheck %s + +#include +#include + +int* _Nonnull nullabili

[clang] [Clang][CodeGen] Emit “trap reasons” on UBSan traps (PR #145967)

@@ -17,8 +18,11 @@ void bar(volatile int a) __attribute__((optnone)) { } // With optimisations enabled the traps are merged and need to share a debug location -// CHECK: [[LOC]] = !DILocation(line: 0 +// CHECK: [[LOC]] = !DILocation(line: 0, anthonyhatran wro

[clang] [Clang] Consider default template arguments when synthesizing CTAD guides (PR #147675)

https://github.com/zyn0217 updated https://github.com/llvm/llvm-project/pull/147675 >From f8b1894ad39560edb5bb0c14194a3a2d54a911ff Mon Sep 17 00:00:00 2001 From: Younan Zhang Date: Wed, 9 Jul 2025 16:29:45 +0800 Subject: [PATCH] [Clang] Consider default template arguments when synthesizing CTA

[clang] [clang-tools-extra] [lldb] [llvm] [mlir] A couple of grammar fixes (PR #144368)

llvmbot wrote: @llvm/pr-subscribers-mlir-core Author: Roman (GameRoMan) Changes --- Full diff: https://github.com/llvm/llvm-project/pull/144368.diff 10 Files Affected: - (modified) clang-tools-extra/clang-tidy/modernize/UseUsingCheck.cpp (+1-1) - (modified) clang/test/AST/HLSL/RootSi

[clang] [llvm] [HIPSTDPAR] Add support for globals (PR #146813)

https://github.com/jmmartinez edited https://github.com/llvm/llvm-project/pull/146813 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [HIPSTDPAR] Add support for globals (PR #146813)

https://github.com/jmmartinez approved this pull request. It looks good to me, but I'd like for Sam to take an extra look since I'm not familiar with the bigger picture here. https://github.com/llvm/llvm-project/pull/146813 ___ cfe-commits mailing lis

[clang] [clang-tools-extra] [Clang] [Diagnostics] Simplify filenames that contain '..' (PR #143520)

justincady wrote: Instead of operating in place, could you avoid the network mount entirely by creating the required structure under /tmp and running the test there? https://github.com/llvm/llvm-project/pull/143520 ___ cfe-commits mailing list cfe-com

[clang] [clang-tools-extra] [lldb] [llvm] [mlir] A couple of grammar fixes (PR #144368)

https://github.com/GameRoMan updated https://github.com/llvm/llvm-project/pull/144368 >From 2b50682f230efa03c3b9a1f5c5e48e708734bf4d Mon Sep 17 00:00:00 2001 From: Roman A <121314722+gamero...@users.noreply.github.com> Date: Mon, 16 Jun 2025 15:59:01 +0100 Subject: [PATCH] A couple of grammar fi

[clang] [clang] Build argument string for clang::warn_unused_result (PR #148090)

https://github.com/zebullax updated https://github.com/llvm/llvm-project/pull/148090 >From 11909560ed6cb4e56192fbbfe4d8b1cdf58e1cb1 Mon Sep 17 00:00:00 2001 From: zebullax Date: Fri, 11 Jul 2025 09:12:44 +0900 Subject: [PATCH 1/6] Build argument string for clang::warn_unused_result Preserve th

[clang] [clang] Build argument string for clang::warn_unused_result (PR #148090)

https://github.com/zebullax updated https://github.com/llvm/llvm-project/pull/148090 >From 11909560ed6cb4e56192fbbfe4d8b1cdf58e1cb1 Mon Sep 17 00:00:00 2001 From: zebullax Date: Fri, 11 Jul 2025 09:12:44 +0900 Subject: [PATCH 1/6] Build argument string for clang::warn_unused_result Preserve th

[clang] [LifetimeSafety] Implement dataflow analysis for loan propagation (PR #148065)

@@ -493,7 +496,247 @@ class FactGenerator : public ConstStmtVisitor { }; // = // -// TODO: Run dataflow analysis to propagate loans, analyse and error reporting. +// The Dat

[clang] [llvm] [X86] Remove WIDEKL feature from Pantherlake and Clearwaterforest (PR #148184)

https://github.com/RKSimon approved this pull request. LGTM https://github.com/llvm/llvm-project/pull/148184 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang][Driver] Revise Cygwin ToolChain to call linker directly (PR #147960)

@@ -1504,6 +1506,7 @@ void ToolChain::AddCXXStdlibLibArgs(const ArgList &Args, switch (Type) { case ToolChain::CST_Libcxx: CmdArgs.push_back("-lc++"); +CmdArgs.push_back("-lc++abi"); tyan0 wrote: Then, how about: ```diff @@ -1504,6 +1506,8 @@ void

[clang] [llvm] [X86] Remove WIDEKL feature from Pantherlake and Clearwaterforest (PR #148184)

https://github.com/e-kud approved this pull request. LGTM. Thanks! https://github.com/llvm/llvm-project/pull/148184 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang][CodeGen][X86] don't coerce int128 into `{i64,i64}` for SysV-like ABIs (PR #135230)

@@ -2595,6 +2595,14 @@ GetX86_64ByValArgumentPair(llvm::Type *Lo, llvm::Type *Hi, ABIArgInfo X86_64ABIInfo:: classifyReturnType(QualType RetTy) const { + // return int128 as i128 + if (const BuiltinType *BT = RetTy->getAs()) { +BuiltinType::Kind k = BT->getKind(); +

[clang] [Clang] Improve diagnostics for 'placement new' with const storage argument (PR #144270)

https://github.com/vbvictor updated https://github.com/llvm/llvm-project/pull/144270 >From ba7db62070dc669d5ee0e54df075abb35772faee Mon Sep 17 00:00:00 2001 From: Victor Baranov Date: Sun, 15 Jun 2025 19:56:05 +0300 Subject: [PATCH 1/4] [Clang] Improve diagnostics when 'placement new' was call

[clang] [clang-tools-extra] [Clang] [Diagnostics] Simplify filenames that contain '..' (PR #143520)

Sirraide wrote: > The only suggestion I have might be terrible, but you could write some python > to enumerate mount points, determine which ones are network drives, and then > check the test path against the list of mount points. I will say that I don’t think I know enough Python (basically n

[clang] [clang][bytecode] Remove needless global check (PR #148163)

https://github.com/tbaederr closed https://github.com/llvm/llvm-project/pull/148163 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [WIP][DO NOT MERGE][Clang][Driver] Emit warning when -fsanitize-trap=<...> is passed without associated -fsanitize=<...> (PR #147997)

https://github.com/delcypher edited https://github.com/llvm/llvm-project/pull/147997 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] 38b9c66 - [clang][bytecode] Remove needless global check (#148163)

Author: Timm Baeder Date: 2025-07-11T15:47:32+02:00 New Revision: 38b9c660a7a3b8947b23acc344521d770c340e48 URL: https://github.com/llvm/llvm-project/commit/38b9c660a7a3b8947b23acc344521d770c340e48 DIFF: https://github.com/llvm/llvm-project/commit/38b9c660a7a3b8947b23acc344521d770c340e48.diff L

[clang] [clang-tools-extra] [lldb] [llvm] [mlir] A couple of grammar fixes (PR #144368)

vbvictor wrote: Please write PR description more specific and add (NFC) at the end. Could be: "Fix typos 'seperate' -> 'separate' (NFC)" https://github.com/llvm/llvm-project/pull/144368 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://li

[clang] [WIP][DO NOT MERGE][Clang][Driver] Emit warning when -fsanitize-trap=<...> is passed without associated -fsanitize=<...> (PR #147997)

https://github.com/delcypher edited https://github.com/llvm/llvm-project/pull/147997 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang-tools-extra] [Clang] [Diagnostics] Simplify filenames that contain '..' (PR #143520)

AaronBallman wrote: > > I suppose another option is to use a regex in the test to accept either > > form of canonicalization with a comment that lit tests are sometimes run > > from network mounts and that's why the test is the way it is. > > I mean, at that point we can just delete the test e

[clang] [WIP][DO NOT MERGE][Clang][Driver] Emit warning when -fsanitize-trap=<...> is passed without associated -fsanitize=<...> (PR #147997)

delcypher wrote: @MaskRay > The current -fsanitize-trap behavior is intentional. See https://discourse.llvm.org/t/clang-gsoc-2025-usability-improvements-for-trapping-undefined-behavior-sanitizer/84568/11 Yes we aware the current behavior is intentional. That however doesn't mean it's the right

[clang] [Clang] fix crash in codegen caused by deferred asm diagnostics under -fopenmp (PR #147163)

https://github.com/a-tarasyuk updated https://github.com/llvm/llvm-project/pull/147163 >From 4e0cf4e00d4cfd837e9dfd9e6aed88aca1de295a Mon Sep 17 00:00:00 2001 From: Oleksandr Tarasiuk Date: Sun, 6 Jul 2025 00:35:48 +0300 Subject: [PATCH 1/4] [Clang] fix crash in codegen caused by deferred asm

[clang] [clang-tools-extra] [lldb] [Clang][AST][NFC] (`RecordDecl` -> `CXXRecordDecl`)`::isInjectedClassName` (PR #148195)

Michael137 wrote: > I'm not sure if I should update > `llvm/include/llvm/Testing/Demangle/DemangleTestCases.inc` and > `libcxxabi/test/DemangleTestCases.inc`. Why would you need to? There are no changes to mangling/demangling here AFAICT https://github.com/llvm/llvm-project/pull/148195 __

[clang] [clang][scan-deps] Report a scanned TU's visible modules (PR #147969)

@@ -0,0 +1,116 @@ +// This test verifies that the modules visible to the translation unit are computed in dependency scanning. +// "client" in the first scan represents the translation unit that imports an explicit submodule, +//that only exports one other module. +// In t

[clang] [clang][scan-deps] Report a scanned TU's visible modules (PR #147969)

https://github.com/jansvoboda11 edited https://github.com/llvm/llvm-project/pull/147969 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang][scan-deps] Report a scanned TU's visible modules (PR #147969)

https://github.com/jansvoboda11 approved this pull request. LGTM https://github.com/llvm/llvm-project/pull/147969 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang][Dependency Scanning] Report only Regular File Size Changes When Computing Out-of-Date File System Cache Entries (PR #148082)

https://github.com/jansvoboda11 approved this pull request. LGTM, thanks! https://github.com/llvm/llvm-project/pull/148082 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [NFC][C++][Modules] Refine test/CXX/basic.link/p3.cpp with split-file (PR #147945)

https://github.com/yronglin closed https://github.com/llvm/llvm-project/pull/147945 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] d679b9a - [NFC][C++][Modules] Refine test/CXX/basic.link/p3.cpp with split-file (#147945)

Author: yronglin Date: 2025-07-11T15:12:37+08:00 New Revision: d679b9a8226fdb1204f8022b78b761fde44fea0b URL: https://github.com/llvm/llvm-project/commit/d679b9a8226fdb1204f8022b78b761fde44fea0b DIFF: https://github.com/llvm/llvm-project/commit/d679b9a8226fdb1204f8022b78b761fde44fea0b.diff LOG:

[clang-tools-extra] [clang-tidy][readability-named-parameter] Add an option to print names without comment (PR #147953)

vbvictor wrote: nit: we typically not write check name in `[]` in the title, the title should be `[clang-tidy] Add an option in 'readability-named-parameter' to print names without comment` https://github.com/llvm/llvm-project/pull/147953 ___ cfe-comm

[clang] [clang] Build argument string for clang::warn_unused_result (PR #148090)

https://github.com/zebullax updated https://github.com/llvm/llvm-project/pull/148090 >From 11909560ed6cb4e56192fbbfe4d8b1cdf58e1cb1 Mon Sep 17 00:00:00 2001 From: zebullax Date: Fri, 11 Jul 2025 09:12:44 +0900 Subject: [PATCH 1/3] Build argument string for clang::warn_unused_result Preserve th

[clang] AMDGPU: Remove "gws" from the “read-only” target feature list (PR #148141)

@@ -1,6 +0,0 @@ -// RUN: %clang_cc1 -triple amdgcn -target-feature +gws -o /dev/null %s 2>&1 \ -// RUN: | FileCheck --check-prefix=GWS %s - -// GWS: warning: feature flag '+gws' is ignored since the feature is read only [-Winvalid-command-line-argument] arsenm

[clang] AMDGPU: Remove "gws" from the “read-only” target feature list (PR #148141)

@@ -816,12 +816,12 @@ kernel void test_target_features_kernel(global int *i) { // NOCPU: attributes #[[ATTR10]] = { convergent nounwind } //. // GFX900: attributes #[[ATTR0:[0-9]+]] = { "objc_arc_inert" } -// GFX900: attributes #[[ATTR1]] = { convergent norecurse nounwind "den

[clang] [clang][bytecode] Remove needless global check (PR #148163)

https://github.com/tbaederr created https://github.com/llvm/llvm-project/pull/148163 Remove the call to D->hasGlobalStorage(), since we never reach this point for local variables. >From 39bd7c5dff416b3ae4c156835aa27aadcaea704c Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Timm=20B=C3=A4der?= Date:

[clang] [clang][bytecode] Remove needless global check (PR #148163)

llvmbot wrote: @llvm/pr-subscribers-clang Author: Timm Baeder (tbaederr) Changes Remove the call to D->hasGlobalStorage(), since we never reach this point for local variables. --- Full diff: https://github.com/llvm/llvm-project/pull/148163.diff 1 Files Affected: - (modified) clang/lib

[clang] [Clang] Respect MS layout attributes during CUDA/HIP device compilation (PR #146620)

llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `llvm-x86_64-debian-dylib` running on `gribozavr4` while building `clang` at step 5 "build-unified-tree". Full details are available at: https://lab.llvm.org/buildbot/#/builders/60/builds/32566 Here is the relevant piece of t

[clang] a2246ee - [C23] Accept an _Atomic underlying type (#147802)

Author: Aaron Ballman Date: 2025-07-11T07:28:03-04:00 New Revision: a2246eebcae47c5cb92c524ee96191edb358922d URL: https://github.com/llvm/llvm-project/commit/a2246eebcae47c5cb92c524ee96191edb358922d DIFF: https://github.com/llvm/llvm-project/commit/a2246eebcae47c5cb92c524ee96191edb358922d.diff

[clang] [C23] Accept an _Atomic underlying type (PR #147802)

https://github.com/AaronBallman closed https://github.com/llvm/llvm-project/pull/147802 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang][Doc] Add Changelog line for (#144886) (PR #147978)

https://github.com/cor3ntin approved this pull request. https://github.com/llvm/llvm-project/pull/147978 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] 77861b3 - [OpenMP][clang] 6.0: parsing/sema for message/severity for parallel (#146093)

Author: Robert Imschweiler Date: 2025-07-11T11:22:06+02:00 New Revision: 77861b3a8f26bb31446db79979a92aeee86ed20f URL: https://github.com/llvm/llvm-project/commit/77861b3a8f26bb31446db79979a92aeee86ed20f DIFF: https://github.com/llvm/llvm-project/commit/77861b3a8f26bb31446db79979a92aeee86ed20f.

[clang] [Clang][AArch64] Relax SVE bf16 requirement for opaque builtins. (PR #147795)

@@ -1,87 +0,0 @@ -// REQUIRES: aarch64-registered-target - -// RUN: %clang_cc1 -triple aarch64-none-linux-gnu -target-feature +sve -verify -verify-ignore-unexpected=error,note -emit-llvm -o - %s - -#include - -void test_bfloat(svbool_t pg, uint64_t u64, int64_t i64, const bfloat

[clang] [Clang][AArch64] Relax SVE bf16 requirement for opaque builtins. (PR #147795)

https://github.com/kmclaughlin-arm approved this pull request. https://github.com/llvm/llvm-project/pull/147795 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] fix crash in codegen caused by deferred asm diagnostics under -fopenmp (PR #147163)

https://github.com/a-tarasyuk updated https://github.com/llvm/llvm-project/pull/147163 >From 4e0cf4e00d4cfd837e9dfd9e6aed88aca1de295a Mon Sep 17 00:00:00 2001 From: Oleksandr Tarasiuk Date: Sun, 6 Jul 2025 00:35:48 +0300 Subject: [PATCH 1/2] [Clang] fix crash in codegen caused by deferred asm

[clang] [clang-tools-extra] [Clang] [Diagnostics] Simplify filenames that contain '..' (PR #143520)

AaronBallman wrote: I don't think we have a way to support that. I looked through lit to see what kind of `REQUIRES` support we have and we can target systems and such, but I don't see any built-in support for non-network mounts. The only suggestion I have might be terrible, but you could writ

[clang] [Clang][Driver] Revise Cygwin ToolChain to call linker directly (PR #147960)

@@ -1504,6 +1506,7 @@ void ToolChain::AddCXXStdlibLibArgs(const ArgList &Args, switch (Type) { case ToolChain::CST_Libcxx: CmdArgs.push_back("-lc++"); +CmdArgs.push_back("-lc++abi"); kikairoya wrote: It's packaging policy matter, but it would be be

[clang] [llvm] [HIPSTDPAR] Add support for globals (PR #146813)

https://github.com/AlexVlx updated https://github.com/llvm/llvm-project/pull/146813 >From d98e3785a144ada9881cdbe24c86f273850eca20 Mon Sep 17 00:00:00 2001 From: Alex Voicu Date: Thu, 3 Jul 2025 02:02:04 +0100 Subject: [PATCH 1/5] Add support for true globals. --- llvm/lib/Transforms/HipStdPa

[clang] [Multilib] Extend the Multilib system to support an IncludepPath field. (PR #146651)

@@ -0,0 +1,28 @@ + +# REQUIRES: shell +# UNSUPPORTED: system-windows + +# This test demonstrates the new "IncludePath" field. +# This field allows specifying include paths through +# multilib.yaml configuration using relative paths. +# Absolute paths should be rejected by the driv

[clang] [llvm] [HIPSTDPAR] Add support for globals (PR #146813)

https://github.com/AlexVlx updated https://github.com/llvm/llvm-project/pull/146813 >From d98e3785a144ada9881cdbe24c86f273850eca20 Mon Sep 17 00:00:00 2001 From: Alex Voicu Date: Thu, 3 Jul 2025 02:02:04 +0100 Subject: [PATCH 1/4] Add support for true globals. --- llvm/lib/Transforms/HipStdPa

[clang] [Multilib] Extend the Multilib system to support an IncludepPath field. (PR #146651)

https://github.com/simpal01 updated https://github.com/llvm/llvm-project/pull/146651 >From 30c353052c35f0117818aca3241e3dea6be2 Mon Sep 17 00:00:00 2001 From: Simi Pallipurath Date: Tue, 1 Jul 2025 21:25:00 +0100 Subject: [PATCH 1/4] [Multilib] Extend the Multilib system to support an Incl

[clang] [PATCH 1/4] [clang] Improve nested name specifier AST representation (PR #147835)

AaronBallman wrote: > Given the scale of the patch, it's not surprising if it undergoes some > revert-reapplies cycles after the initial commit. > > To reduce the churn, would it be possible to ask google (and other major > downstream clients) to test it internally before we merge? @AaronBallm

[clang] [llvm] [HIPSTDPAR] Add support for globals (PR #146813)

@@ -114,24 +115,221 @@ static inline void clearModule(Module &M) { // TODO: simplify. eraseFromModule(*M.ifuncs().begin()); } +static inline SmallVector> +collectIndirectableUses(GlobalVariable *G) { + // We are interested only in use chains that end in an Instruction. +

[clang] [llvm] [HIPSTDPAR] Add support for globals (PR #146813)

@@ -114,24 +115,221 @@ static inline void clearModule(Module &M) { // TODO: simplify. eraseFromModule(*M.ifuncs().begin()); } +static inline SmallVector> +collectIndirectableUses(GlobalVariable *G) { + // We are interested only in use chains that end in an Instruction. +

[clang] [Clang] Improve diagnostics for 'placement new' with const storage argument (PR #144270)

@@ -690,6 +690,9 @@ Improvements to Clang's diagnostics - Clang now tries to avoid printing file paths that contain ``..``, instead preferring the canonical file path if it ends up being shorter. +- Improve the diagnostics for placement new expression when const-qualified +

[clang] [Clang] Improve diagnostics for 'placement new' with const storage argument (PR #144270)

https://github.com/cor3ntin approved this pull request. LGTM once the changelog is updated https://github.com/llvm/llvm-project/pull/144270 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-comm

[clang] [Clang][Driver] Revise Cygwin ToolChain to call linker directly (PR #147960)

kikairoya wrote: If we should consider cross-compiling, it's needed to be aware that `$SYSROOT/usr/lib` doesn't appear in real filesystem. https://github.com/llvm/llvm-project/pull/147960 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https:/

[clang] [clang-tools-extra] [lldb] [llvm] [mlir] A couple of grammar fixes (PR #144368)

https://github.com/GameRoMan updated https://github.com/llvm/llvm-project/pull/144368 >From 2b50682f230efa03c3b9a1f5c5e48e708734bf4d Mon Sep 17 00:00:00 2001 From: Roman A <121314722+gamero...@users.noreply.github.com> Date: Mon, 16 Jun 2025 15:59:01 +0100 Subject: [PATCH] A couple of grammar fi

[clang] [clang-tools-extra] [lldb] [llvm] [mlir] A couple of grammar fixes (PR #144368)

https://github.com/GameRoMan ready_for_review https://github.com/llvm/llvm-project/pull/144368 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang-tools-extra] [lldb] [llvm] [mlir] A couple of grammar fixes (PR #144368)

llvmbot wrote: @llvm/pr-subscribers-debuginfo Author: Roman (GameRoMan) Changes --- Full diff: https://github.com/llvm/llvm-project/pull/144368.diff 10 Files Affected: - (modified) clang-tools-extra/clang-tidy/modernize/UseUsingCheck.cpp (+1-1) - (modified) clang/test/AST/HLSL/RootSi

[clang] [clang-tools-extra] [lldb] [llvm] [mlir] A couple of grammar fixes (PR #144368)

llvmbot wrote: @llvm/pr-subscribers-llvm-analysis Author: Roman (GameRoMan) Changes --- Full diff: https://github.com/llvm/llvm-project/pull/144368.diff 10 Files Affected: - (modified) clang-tools-extra/clang-tidy/modernize/UseUsingCheck.cpp (+1-1) - (modified) clang/test/AST/HLSL/Ro

[clang] [clang-tools-extra] [lldb] [llvm] [mlir] A couple of grammar fixes (PR #144368)

https://github.com/GameRoMan updated https://github.com/llvm/llvm-project/pull/144368 >From 2b50682f230efa03c3b9a1f5c5e48e708734bf4d Mon Sep 17 00:00:00 2001 From: Roman A <121314722+gamero...@users.noreply.github.com> Date: Mon, 16 Jun 2025 15:59:01 +0100 Subject: [PATCH] A couple of grammar fi

[clang] [CIR] Add support for non-virtual base class initialization (PR #148080)

https://github.com/erichkeane approved this pull request. https://github.com/llvm/llvm-project/pull/148080 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [HIPSTDPAR] Add support for globals (PR #146813)

@@ -114,24 +115,221 @@ static inline void clearModule(Module &M) { // TODO: simplify. eraseFromModule(*M.ifuncs().begin()); } +static inline SmallVector> +collectIndirectableUses(GlobalVariable *G) { + // We are interested only in use chains that end in an Instruction. +

[clang] [clang] Build argument string for clang::warn_unused_result (PR #148090)

zebullax wrote: @AaronBallman To bypass verification on the standard version used (*), which do not seem to make too much sense here as we were allowing albeit dropping the string until now. I have extracted the `if (AL.isClangScope()) {` code path out of the rest of that branch that focuses o

[clang] [clang-tools-extra] [Clang] [Diagnostics] Simplify filenames that contain '..' (PR #143520)

AaronBallman wrote: > We could try that (assuming that `/tmp` exists on every system but I sure > hope so); it might run into issues if we somehow pick a file/directory name > that something else is already using, but that’s probably not too likely. Windows doesn't have `/tmp` for example. I d

[clang] [clang-tools-extra] [Clang] [Diagnostics] Simplify filenames that contain '..' (PR #143520)

Sirraide wrote: > Windows doesn't have `/tmp` for example. I don't think we have any > [substitutions](https://llvm.org/docs/CommandGuide/lit.html#substitutions) > for getting the temp directory. Ah, I meant non-windows systems (I thought `REQUIRES: shell` already meant non-windows because th

[clang] d11fd66 - [X86] Remove WIDEKL feature from Pantherlake and Clearwaterforest (#148184)

Author: Phoebe Wang Date: 2025-07-11T21:27:42+08:00 New Revision: d11fd669598944b8f1204a7e42ae427fc4901ba9 URL: https://github.com/llvm/llvm-project/commit/d11fd669598944b8f1204a7e42ae427fc4901ba9 DIFF: https://github.com/llvm/llvm-project/commit/d11fd669598944b8f1204a7e42ae427fc4901ba9.diff L

<    1   2   3   4   5   >