https://github.com/Alcaro created
https://github.com/llvm/llvm-project/pull/133593
None
>From e8036df84bf7ea8d544814ca52dc963c036a1e69 Mon Sep 17 00:00:00 2001
From: Alcaro
Date: Sat, 29 Mar 2025 20:24:00 +0100
Subject: [PATCH] [NFC][analyzer] Fix typo in VirtualCall checker docs
---
clang/d
Author: Kazu Hirata
Date: 2025-03-29T12:22:13-07:00
New Revision: 884b19ab40c8b6e5d1fb54894c0418fd27bb93f9
URL:
https://github.com/llvm/llvm-project/commit/884b19ab40c8b6e5d1fb54894c0418fd27bb93f9
DIFF:
https://github.com/llvm/llvm-project/commit/884b19ab40c8b6e5d1fb54894c0418fd27bb93f9.diff
L
@@ -242,3 +242,9 @@
// NO-WARN-ATOMIC: clang{{.*}} "-triple" "amdgcn-amd-amdhsa" {{.*}}
"-Werror=atomic-alignment" {{.*}} "-Wno-error=atomic-alignment"
// NO-WARN-ATOMIC-NOT: clang{{.*}} "-triple" "x86_64-unknown-linux-gnu" {{.*}}
"-Werror=atomic-alignment"
// NO-WARN-ATOMIC-
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Kazu Hirata (kazutakahirata)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/133594.diff
1 Files Affected:
- (modified) clang/include/clang/Basic/TargetInfo.h (+1-2)
``diff
diff --git a/clang/include/clang/B
https://github.com/kazutakahirata created
https://github.com/llvm/llvm-project/pull/133594
None
>From 379db5ff7e08973d2e5a6c08de1a475ce84313f5 Mon Sep 17 00:00:00 2001
From: Kazu Hirata
Date: Sat, 29 Mar 2025 12:23:18 -0700
Subject: [PATCH] [Basic] Use SmallSet::insert_range (NFC)
---
clang/
@@ -127,9 +127,19 @@ void aix::Linker::ConstructJob(Compilation &C, const
JobAction &JA,
}
DanielCChen wrote:
Thanks for the comment!
Yes. I will add test.
https://github.com/llvm/llvm-project/pull/131822
___
cfe-c
https://github.com/cor3ntin closed
https://github.com/llvm/llvm-project/pull/133558
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvmbot wrote:
@llvm/pr-subscribers-flang-driver
Author: Paul Osmialowski (pawosm-arm)
Changes
Although combining -fveclib=ArmPL with -nostdlib is a rare situation, it should
still be supported correctly and should effect in avoidance of linking against
libm.
---
Full diff: https://git
pawosm-arm wrote:
Seriously, this is something CI people signaled to me. Our CI builds everything
with `-fveclib=ArmPL` and one of the packages failed on unexpected linker
behavior when libm was artificially added by `-fveclib=ArmPL` despite the
`-nostdlib` flag being used. My suggestion to us
macurtis-amd wrote:
> Missing new tests? I'd expect to see a few new targeted tests stressing
> different vector sizes and alignments, and not just updates of existing tests
Added a new test.
Thanks for the review!
https://github.com/llvm/llvm-project/pull/133301
_
@@ -1170,10 +1191,23 @@ class AllocaSlices::SliceBuilder : public
PtrUseVisitor {
if (!IsOffsetKnown)
return PI.setAborted(&II);
+auto IsSplittable = [&]() {
+ FixedVectorType *VTy = getVectorTypeFor(II, DL);
+ Type *ATy = AS.AI.getAllocatedType();
+
+
@@ -2316,12 +2362,15 @@ static VectorType *isVectorPromotionViable(Partition
&P, const DataLayout &DL) {
// Put load and store types into a set for de-duplication.
for (const Slice &S : P) {
-Type *Ty;
+Type *Ty = nullptr;
if (auto *LI = dyn_cast(S.getUse()->
@@ -1170,10 +1191,23 @@ class AllocaSlices::SliceBuilder : public
PtrUseVisitor {
if (!IsOffsetKnown)
return PI.setAborted(&II);
+auto IsSplittable = [&]() {
macurtis-amd wrote:
Cleaned up code removes the helper altogether. Hopefully better no
mgorny wrote:
This change seems to have broken the test suite:
```
ctest -j 32 --test-load 32
Test project /var/tmp/portage/llvm-core/libclc-21.0.0./work/libclc_build
Start 1: external-calls-tahiti-amdgcn--.bc
Start 2: external-calls-amdgcn--amdhsa.bc
Start 3: external-calls-tahi
@@ -61,7 +61,7 @@ void addCXXDeallocatorChecker(AnalysisASTConsumer
&AnalysisConsumer,
}
// TODO: What we should really be testing here is all the different varieties
-// of delete operators, and wether the retrieval of their arguments works as
+// of delete operators, and we
https://github.com/philnik777 created
https://github.com/llvm/llvm-project/pull/133587
Fixes #132672
>From 1c0a267544c43235d0004edb9beb127a124abd7a Mon Sep 17 00:00:00 2001
From: Nikolas Klauser
Date: Sat, 29 Mar 2025 15:21:10 +0100
Subject: [PATCH] [Clang] Make enums trivially equality comp
github-actions[bot] wrote:
:warning: C/C++ code formatter, clang-format found issues in your code.
:warning:
You can test this locally with the following command:
``bash
git-clang-format --diff HEAD~1 HEAD --extensions cpp --
clang/lib/Sema/SemaExprCXX.cpp clang/test/SemaCXX/typ
@@ -53,4 +53,4 @@ Options
If set to non-zero, the check does not suggest edits that will transform
EugeneZelenko wrote:
Ditto.
https://github.com/llvm/llvm-project/pull/133525
___
cfe-commits mailing list
cfe-com
@@ -53,4 +53,4 @@ Options
If set to non-zero, the check does not suggest edits that will transform
EugeneZelenko wrote:
`false`?
https://github.com/llvm/llvm-project/pull/133525
___
cfe-commits mailing list
cfe-c
@@ -1011,6 +1011,26 @@ static Value *foldPHINodeOrSelectInst(Instruction &I) {
return foldSelectInst(cast(I));
}
+/// Returns a fixed vector type equivalent to the memory set by II or nullptr
if
+/// unable to do so.
+static FixedVectorType *getVectorTypeFor(const MemSetIns
https://github.com/carlosgalvezp updated
https://github.com/llvm/llvm-project/pull/133582
>From 2ecbd54fbc82bbfad2e190529efd3a4cc71bd8df Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Carlos=20G=C3=A1lvez?=
Date: Sat, 29 Mar 2025 11:55:02 +
Subject: [PATCH] [clang-tidy] Fix broken HeaderFilterReg
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Paul Osmialowski (pawosm-arm)
Changes
Although combining -fveclib=ArmPL with -nostdlib is a rare situation, it should
still be supported correctly and should effect in avoidance of linking against
libm.
---
Full diff: https://github.com
https://github.com/macurtis-amd updated
https://github.com/llvm/llvm-project/pull/133301
>From c0525fd7bd2c740b5b969e8e2913a878792a377c Mon Sep 17 00:00:00 2001
From: Matthew Curtis
Date: Thu, 27 Mar 2025 14:04:40 -0500
Subject: [PATCH 1/3] [SROA] Vector promote some memsets
---
clang/test/Co
https://github.com/arsenm closed
https://github.com/llvm/llvm-project/pull/132813
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/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/Sema/Overload.h clang/include/c
https://github.com/HerrCai0907 approved this pull request.
https://github.com/llvm/llvm-project/pull/133525
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Mythreya
Date: 2025-03-20T02:22:29-04:00
New Revision: 9cdbc47144f28785133592b1ae219cb49b9ac61d
URL:
https://github.com/llvm/llvm-project/commit/9cdbc47144f28785133592b1ae219cb49b9ac61d
DIFF:
https://github.com/llvm/llvm-project/commit/9cdbc47144f28785133592b1ae219cb49b9ac61d.diff
LOG:
https://github.com/cor3ntin updated
https://github.com/llvm/llvm-project/pull/133426
>From 464aaf0944cac75ccc689ebb82e08554c12c6340 Mon Sep 17 00:00:00 2001
From: Corentin Jabot
Date: Thu, 27 Mar 2025 16:25:07 +0100
Subject: [PATCH 1/2] [Clang][WIP][RFC] Bypass TAD during overload resolution
i
https://github.com/philnik777 updated
https://github.com/llvm/llvm-project/pull/121199
>From 19a76935988b306201ed536e99e9094b18006a0f Mon Sep 17 00:00:00 2001
From: Nikolas Klauser
Date: Fri, 27 Sep 2024 22:11:14 +0200
Subject: [PATCH] [Clang] Add __builtin_common_reference
---
clang/docs/Lan
tearfur wrote:
Tested by applying this PR on top of 3e742b517a0606bfed329dfcb8c34c614ed73ea7.
Works great AFAICT!
```console
# curl -sL 'https://github.com/llvm/llvm-project/pull/133582.diff' | git apply
```
https://github.com/llvm/llvm-project/pull/133582
_
llvmbot wrote:
@llvm/pr-subscribers-clang-tools-extra
Author: Carlos Galvez (carlosgalvezp)
Changes
PR https://github.com/llvm/llvm-project/pull/91400 broke the usage of
HeaderFilterRegex via config file, because it is now created at a different
point in the execution and leads to a diff
llvmbot wrote:
@llvm/pr-subscribers-clang-tidy
Author: Carlos Galvez (carlosgalvezp)
Changes
PR https://github.com/llvm/llvm-project/pull/91400 broke the usage of
HeaderFilterRegex via config file, because it is now created at a different
point in the execution and leads to a different v
https://github.com/carlosgalvezp updated
https://github.com/llvm/llvm-project/pull/133582
>From 7f2b6448acee91deef7ff73fd36a92bb9787a674 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Carlos=20G=C3=A1lvez?=
Date: Sat, 29 Mar 2025 11:55:02 +
Subject: [PATCH] [clang-tidy] Fix broken HeaderFilterReg
https://github.com/carlosgalvezp updated
https://github.com/llvm/llvm-project/pull/133582
>From 99abf89c1ed177be1b9a3f7f2acd85ffb726a757 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Carlos=20G=C3=A1lvez?=
Date: Sat, 29 Mar 2025 11:55:02 +
Subject: [PATCH] [clang-tidy] Fix broken HeaderFilterReg
https://github.com/philnik777 updated
https://github.com/llvm/llvm-project/pull/116709
>From ddcc9187ffe1aa07988145ab91c0d038aef5800c Mon Sep 17 00:00:00 2001
From: Nikolas Klauser
Date: Tue, 1 Oct 2024 11:08:02 +0200
Subject: [PATCH] [Clang] Add __builtin_invoke and recognize std::invoke as a
https://github.com/qinkunbao updated
https://github.com/llvm/llvm-project/pull/133558
>From 12ce53e698857003de757cfecd0ff2b4f4a55e8f Mon Sep 17 00:00:00 2001
From: Qinkun Bao
Date: Sat, 29 Mar 2025 03:00:38 +
Subject: [PATCH] Fix some typos under clang
---
clang/docs/BoundsSafety.rst
https://github.com/cor3ntin approved this pull request.
LGTM. Do you need me to merge that for you?
https://github.com/llvm/llvm-project/pull/133558
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo
llvmbot wrote:
@llvm/pr-subscribers-clangd
Author: Kazu Hirata (kazutakahirata)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/133589.diff
4 Files Affected:
- (modified) clang-tools-extra/clang-tidy/bugprone/SignalHandlerCheck.cpp
(+4-7)
- (modified)
clang-tools-e
llvmbot wrote:
@llvm/pr-subscribers-clang-tidy
Author: Kazu Hirata (kazutakahirata)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/133589.diff
4 Files Affected:
- (modified) clang-tools-extra/clang-tidy/bugprone/SignalHandlerCheck.cpp
(+4-7)
- (modified)
clang-too
https://github.com/kazutakahirata created
https://github.com/llvm/llvm-project/pull/133589
None
>From c6e9d5f59267dc346794e539fcbcd293fda84f53 Mon Sep 17 00:00:00 2001
From: Kazu Hirata
Date: Fri, 28 Mar 2025 20:56:24 -0700
Subject: [PATCH] [clang-tools-extra] Use *Set::insert_range (NFC)
---
https://github.com/jhuber6 approved this pull request.
https://github.com/llvm/llvm-project/pull/133522
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
qinkunbao wrote:
> LGTM. Do you need me to merge that for you?
Yes, please. I don't have the write access. Thank you.
https://github.com/llvm/llvm-project/pull/133558
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-b
https://github.com/yxsamliu created
https://github.com/llvm/llvm-project/pull/133590
CUDA defines min/max functions for host in global namespace. HIP header needs
to define them too to be compatible. Currently only min/max(int, int) is
defined. This causes wrong result for arguments that are o
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Yaxun (Sam) Liu (yxsamliu)
Changes
CUDA defines min/max functions for host in global namespace. HIP header needs
to define them too to be compatible. Currently only min/max(int, int) is
defined. This causes wrong result for arguments that
yxsamliu wrote:
This is an effort to reland https://github.com/llvm/llvm-project/pull/82956
Since we kept encountering regressions, we plan to add the change conditionally
under a macro so that we can deliver the fix to intended users without causing
regressions. Then we will try making the ch
tearfur wrote:
P.S. Just a heads up that `Fixes #118009, #121969, #133453` is not linking this
PR to all 3 issues.
They need to be split to 3 `Fixes #xxx` lines.
https://github.com/llvm/llvm-project/pull/133582
___
cfe-commits mailing list
cfe-commit
https://github.com/DanielCChen updated
https://github.com/llvm/llvm-project/pull/131822
>From 21f3ec3b644ea201e3aa15d18f74c795ccf120f3 Mon Sep 17 00:00:00 2001
From: Daniel Chen
Date: Tue, 18 Mar 2025 11:09:41 -0400
Subject: [PATCH 1/2] [Driver] Add linker options to support statical linking
t
https://github.com/RiverDave updated
https://github.com/llvm/llvm-project/pull/131969
>From 9972aa8e1720f7e6378c63551c853ee504193000 Mon Sep 17 00:00:00 2001
From: David Rivera
Date: Sun, 16 Mar 2025 16:20:16 -0400
Subject: [PATCH] [clang-tidy] Add support for Initialization Forwarding in
Nest
llvmbot wrote:
@llvm/pr-subscribers-clang-tools-extra
Author: Kazu Hirata (kazutakahirata)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/133589.diff
4 Files Affected:
- (modified) clang-tools-extra/clang-tidy/bugprone/SignalHandlerCheck.cpp
(+4-7)
- (modified)
cl
@@ -1289,7 +1305,11 @@ void testBracedInitTemporaries() {
v2.push_back(NonTrivialWithVector{{0}});
v2.push_back({{0}});
v2.push_back(NonTrivialWithVector{std::vector{0}});
+ // CHECK-MESSAGES-CPP20: :[[@LINE-1]]:6: warning: use emplace_back instead
of push_back
https://github.com/carlosgalvezp created
https://github.com/llvm/llvm-project/pull/133582
PR https://github.com/llvm/llvm-project/pull/91400 broke the usage of
HeaderFilterRegex via config file, because it is now created at a different
point in the execution and leads to a different value.
Th
llvmbot wrote:
@llvm/pr-subscribers-backend-x86
Author: Yaxun (Sam) Liu (yxsamliu)
Changes
CUDA defines min/max functions for host in global namespace. HIP header needs
to define them too to be compatible. Currently only min/max(int, int) is
defined. This causes wrong result for argument
https://github.com/kuhar approved this pull request.
https://github.com/llvm/llvm-project/pull/133589
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/RiverDave updated
https://github.com/llvm/llvm-project/pull/131969
>From f0b31f80f452c80926822ea279e5aff37b84b840 Mon Sep 17 00:00:00 2001
From: David Rivera
Date: Sun, 16 Mar 2025 16:20:16 -0400
Subject: [PATCH] [clang-tidy] Add support for Initialization Forwarding in
Nest
https://github.com/steakhal closed
https://github.com/llvm/llvm-project/pull/133593
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Alcaro
Date: 2025-03-29T21:34:15+01:00
New Revision: be7f185ca16e990614113519d003bfe3685f489b
URL:
https://github.com/llvm/llvm-project/commit/be7f185ca16e990614113519d003bfe3685f489b
DIFF:
https://github.com/llvm/llvm-project/commit/be7f185ca16e990614113519d003bfe3685f489b.diff
LOG: [
https://github.com/kuhar approved this pull request.
https://github.com/llvm/llvm-project/pull/133594
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/irymarchyk created
https://github.com/llvm/llvm-project/pull/133598
Currently, the `ShortFunctionStyle` option in clang-format lacks the
granularity to specifically control the single-line formatting of `static
inline` C functions independently from other function types like
https://github.com/mizvekov edited
https://github.com/llvm/llvm-project/pull/133597
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/GeorgeKA created
https://github.com/llvm/llvm-project/pull/133597
Alias template class template argument deduction is a C++20 feature. Also
updated relevant CTAD test cases.
>From 5b6d72ce1503196bf1c06a8b3dbc3f913265d76c Mon Sep 17 00:00:00 2001
From: George Asante
Date: Sa
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: None (GeorgeKA)
Changes
Alias template class template argument deduction is a C++20 feature. Also
updated relevant CTAD test cases.
---
Full diff: https://github.com/llvm/llvm-project/pull/133597.diff
4 Files Affected:
- (modified) cla
https://github.com/GeorgeKA edited
https://github.com/llvm/llvm-project/pull/133597
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/steakhal approved this pull request.
https://github.com/llvm/llvm-project/pull/133593
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvmbot wrote:
@llvm/pr-subscribers-clang-format
Author: Ivan (irymarchyk)
Changes
Currently, the `ShortFunctionStyle` option in clang-format lacks the
granularity to specifically control the single-line formatting of `static
inline` C functions independently from other function types li
https://github.com/DanielCChen updated
https://github.com/llvm/llvm-project/pull/131822
>From 21f3ec3b644ea201e3aa15d18f74c795ccf120f3 Mon Sep 17 00:00:00 2001
From: Daniel Chen
Date: Tue, 18 Mar 2025 11:09:41 -0400
Subject: [PATCH] [Driver] Add linker options to support statical linking to
sh
https://github.com/GeorgeKA edited
https://github.com/llvm/llvm-project/pull/133597
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Ivan (irymarchyk)
Changes
Currently, the `ShortFunctionStyle` option in clang-format lacks the
granularity to specifically control the single-line formatting of `static
inline` C functions independently from other function types like regu
@@ -335,6 +336,32 @@ class LineJoiner {
}
}
+ if (Style.AllowShortFunctionsOnASingleLine ==
+ FormatStyle::SFS_StaticInlineOnly ||
+ Style.AllowShortFunctionsOnASingleLine ==
+ FormatStyle::SFS_StaticInline) {
+// C
@@ -15120,6 +15120,85 @@ TEST_F(FormatTest,
PullInlineFunctionDefinitionsIntoSingleLine) {
MergeInlineOnly);
}
+TEST_F(FormatTest, PullStaticInlineFunctionDefinitionsIntoSingleLine) {
+ FormatStyle MergeStaticInlineOnly = getLLVMStyle();
+ MergeStaticInlineOn
@@ -863,6 +870,12 @@ struct FormatStyle {
/// void f() {}
/// \endcode
SFS_Inline,
+/// Only merge functions defined as static inline. Implies ``empty``.
HazardyKnusperkeks wrote:
This should have a different text.
https://github.com/llvm/ll
@@ -335,6 +336,32 @@ class LineJoiner {
}
}
+ if (Style.AllowShortFunctionsOnASingleLine ==
+ FormatStyle::SFS_StaticInlineOnly ||
+ Style.AllowShortFunctionsOnASingleLine ==
+ FormatStyle::SFS_StaticInline) {
+// C
GeorgeKA wrote:
> We do tend to offer newer features into older standards when that isn't a
> burden, as long as we have a warning advertising the incompatibility with the
> current standard, as we do.
>
> What is the motivation for this change?
It was originally detailed in
[125913](https:/
@@ -335,6 +336,32 @@ class LineJoiner {
}
}
+ if (Style.AllowShortFunctionsOnASingleLine ==
+ FormatStyle::SFS_StaticInlineOnly ||
+ Style.AllowShortFunctionsOnASingleLine ==
+ FormatStyle::SFS_StaticInline) {
+// C
https://github.com/cyndyishida created
https://github.com/llvm/llvm-project/pull/132237
When a module is being scanned, it can depend on modules that have already been
built from a pch dependency. When this happens, the pcm files are reused for
the module dependencies. When this is the case, c
Author: Kazu Hirata
Date: 2025-03-29T14:26:36-07:00
New Revision: 3d43739753f3b50824a09343efe40a2a476488df
URL:
https://github.com/llvm/llvm-project/commit/3d43739753f3b50824a09343efe40a2a476488df
DIFF:
https://github.com/llvm/llvm-project/commit/3d43739753f3b50824a09343efe40a2a476488df.diff
L
https://github.com/MaskRay approved this pull request.
https://github.com/llvm/llvm-project/pull/131230
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -229,6 +229,12 @@ class CodeGenTypes {
const CGFunctionInfo &arrangeBuiltinFunctionCall(QualType resultType,
const CallArgList &args);
+ /// A SYCL device kernel function is a free standing function with
+ /// spir_kern
101 - 177 of 177 matches
Mail list logo
