https://github.com/gedare updated
https://github.com/llvm/llvm-project/pull/93140
>From 27a2da876926d2157ea9f18c5fd71a2e81e097fc Mon Sep 17 00:00:00 2001
From: Gedare Bloom
Date: Tue, 21 May 2024 22:21:59 -0600
Subject: [PATCH 1/4] [clang-format] Improve BlockIndent at ColumnLimit
Fixes
https://github.com/ahmedbougacha edited
https://github.com/llvm/llvm-project/pull/93904
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ahmedbougacha edited
https://github.com/llvm/llvm-project/pull/93906
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ahmedbougacha closed
https://github.com/llvm/llvm-project/pull/93904
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Ahmed Bougacha
Date: 2024-06-20T12:09:54-07:00
New Revision: 7c814c13d0df6dbd0ef6a8b2be214d3f6edbb566
URL:
https://github.com/llvm/llvm-project/commit/7c814c13d0df6dbd0ef6a8b2be214d3f6edbb566
DIFF:
theuni wrote:
I tried out this WIP on our codebase after we ran into a problem where an
annotation was added to a definition rather than a declaration, leading to a
(silently) missed capability check.
This addition seems to work fine, but it triggers warnings when annotations are
added to
https://github.com/ahmedbougacha updated
https://github.com/llvm/llvm-project/pull/93904
>From 2767407e2970d6fb8b0e651d80162d9481129258 Mon Sep 17 00:00:00 2001
From: Ahmed Bougacha
Date: Thu, 30 May 2024 17:33:04 -0700
Subject: [PATCH 1/6] [clang] Define ptrauth_sign_constant builtin.
This
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Oleksandr T. (a-tarasyuk)
Changes
Fixes #96043
---
Full diff: https://github.com/llvm/llvm-project/pull/96228.diff
3 Files Affected:
- (modified) clang/docs/ReleaseNotes.rst (+1-1)
- (modified) clang/lib/Sema/SemaDeclCXX.cpp (+7-5)
-
https://github.com/a-tarasyuk created
https://github.com/llvm/llvm-project/pull/96228
Fixes #96043
>From 3345a48624a6f5795cbbdac21b5504322871ea90 Mon Sep 17 00:00:00 2001
From: Oleksandr T
Date: Thu, 20 Jun 2024 21:51:42 +0300
Subject: [PATCH] [Clang] resolve record declaration of defaulted
https://github.com/ahmedbougacha edited
https://github.com/llvm/llvm-project/pull/93904
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ahmedbougacha closed
https://github.com/llvm/llvm-project/pull/93903
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Ahmed Bougacha
Date: 2024-06-20T11:55:41-07:00
New Revision: 50b919378186ebb59e2dd402720f6ea8576d2477
URL:
https://github.com/llvm/llvm-project/commit/50b919378186ebb59e2dd402720f6ea8576d2477
DIFF:
gedare wrote:
> > I saw some comments on this. Looks like some were deleted maybe. This is
> > still in my queue, but unclear when I'll get back to it.
>
> Yeah, sorry, I posted a comment here but decided it was a bit irrelevant. :)
>
> But going to ask anyway then. Can this setting remove
Fznamznon wrote:
Thank you!
https://github.com/llvm/llvm-project/pull/95802
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -4650,10 +4650,23 @@ struct FormatStyle {
/// # Should be declared this way:
/// SpacesInParens: Custom
/// SpacesInParensOptions:
+ /// ExceptDoubleParentheses: false
/// InConditionalStatements: true
/// Other: true
/// \endcode
struct
jroelofs wrote:
I think this needs a release note.
https://github.com/llvm/llvm-project/pull/96221
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
gedare wrote:
> Do we need all these new test cases? If yes, consider moving the test (and
> other `ConfigurableSpacesIn...` tests) to a separate file.
I have simplified the additional test cases.
https://github.com/llvm/llvm-project/pull/77522
___
tbaederr wrote:
I just pushed
https://github.com/llvm/llvm-project/commit/99f5fcb0d1e04125daa404ff14c9cd14b7a2c40b
- I don't have time to run _all_ the tests though, so this is a bit of a long
shot. If that doesn't fix it, then disabling them for now sounds fine to me.
https://github.com/gedare updated
https://github.com/llvm/llvm-project/pull/77522
>From 4a7489251b842da778dd839bd5af3db12ba0fe0b Mon Sep 17 00:00:00 2001
From: Gedare Bloom
Date: Mon, 17 Jul 2023 18:24:30 -0600
Subject: [PATCH 1/4] Add SpaceInParensOption ExceptDoubleParentheses
This change
Author: Timm Bäder
Date: 2024-06-20T20:29:27+02:00
New Revision: 99f5fcb0d1e04125daa404ff14c9cd14b7a2c40b
URL:
https://github.com/llvm/llvm-project/commit/99f5fcb0d1e04125daa404ff14c9cd14b7a2c40b
DIFF:
https://github.com/llvm/llvm-project/commit/99f5fcb0d1e04125daa404ff14c9cd14b7a2c40b.diff
temyurchenko wrote:
> Do you need someone to land these changes on your behalf?
Yeah, I don't have the rights :(
https://github.com/llvm/llvm-project/pull/93913
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
Fznamznon wrote:
I wonder if that would be ok to disable interpreter tests for now?
https://github.com/llvm/llvm-project/pull/95802
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HazardyKnusperkeks approved this pull request.
https://github.com/llvm/llvm-project/pull/96128
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jroelofs wrote:
cc @tahonermann
https://github.com/llvm/llvm-project/pull/96221
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/labrinea edited
https://github.com/llvm/llvm-project/pull/96221
___
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: Alexandros Lamprineas (labrinea)
Changes
Long story short the interaction of two optimizations happening in GlobalOpt
results in a crash. For more details look at the issue
https://github.com/llvm/llvm-project/issues/96197. I will be
https://github.com/labrinea created
https://github.com/llvm/llvm-project/pull/96221
Long story short the interaction of two optimizations happening in GlobalOpt
results in a crash. For more details look at the issue
https://github.com/llvm/llvm-project/issues/96197. I will be fixing this in
@@ -0,0 +1,141 @@
+// RUN: %clang_cc1 -fsyntax-only -fblocks -fcxx-exceptions -verify %s
+// RUN: %clang_cc1 -fsyntax-only -fblocks -verify -x c -std=c23 %s
+
+#if !__has_attribute(nonblocking)
+#error "the 'nonblocking' attribute is not available"
+#endif
+
+// --- ATTRIBUTE
https://github.com/dougsonos edited
https://github.com/llvm/llvm-project/pull/84983
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -118,6 +124,37 @@ llvm::Type *CodeGenTypes::ConvertTypeForMem(QualType T,
bool ForBitField) {
return R;
}
+bool CodeGenTypes::LLVMTypeLayoutMatchesAST(QualType ASTTy,
+llvm::Type *LLVMTy) {
+ CharUnits ASTSize =
Fznamznon wrote:
I can't because I don't have a big endian to verify with. We can try to push
speculatively if it doesn't break existing tests.
https://github.com/llvm/llvm-project/pull/95802
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
tbaederr wrote:
Here's a quick patch with the cast inserted:
```diff
diff --git a/clang/lib/AST/Interp/ByteCodeExprGen.cpp
b/clang/lib/AST/Interp/ByteCodeExprGen.cpp
index 731153a6ead9..e7fa1a62c277 100644
--- a/clang/lib/AST/Interp/ByteCodeExprGen.cpp
+++
Fznamznon wrote:
I'm trying to insert a cast using emitCast:
```
--- a/clang/lib/AST/Interp/ByteCodeExprGen.cpp
+++ b/clang/lib/AST/Interp/ByteCodeExprGen.cpp
@@ -1347,6 +1347,13 @@ bool
ByteCodeExprGen::visitInitList(ArrayRef Inits,
}
auto Eval = [&](Expr *Init, unsigned ElemIndex)
Fznamznon wrote:
Yes, all bots are big endian. Reproducer is
```
clang -cc1 %s -fsyntax-only -verify -fexperimental-new-constant-interpreter
constexpr int value(int a, int b) {
return a + b;
}
constexpr int init_list_expr() {
int vals[] = {
#embed "jk.txt"
};
return value(vals[0],
rapidsna wrote:
@AaronBallman Thanks for your prompt feedback!
https://github.com/llvm/llvm-project/pull/95964
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
tbaederr wrote:
Do you have a smaller reproducer? Are all the failing build bots big endian?
https://github.com/llvm/llvm-project/pull/95802
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://github.com/rapidsna updated
https://github.com/llvm/llvm-project/pull/95964
>From 6ed8ac33952bc8f9a9b672740d45f25872ae1dbb Mon Sep 17 00:00:00 2001
From: Yeoul Na
Date: Tue, 18 Jun 2024 11:05:17 -0700
Subject: [PATCH 1/2] [BoundsSafety][doc] Make it clear that the feature is
https://github.com/Sirraide closed
https://github.com/llvm/llvm-project/pull/95580
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Mital Ashok
Date: 2024-06-20T19:44:06+02:00
New Revision: 482c41e992c1edf8833a4577b56ff9dda49fbc83
URL:
https://github.com/llvm/llvm-project/commit/482c41e992c1edf8833a4577b56ff9dda49fbc83
DIFF:
https://github.com/llvm/llvm-project/commit/482c41e992c1edf8833a4577b56ff9dda49fbc83.diff
@@ -0,0 +1,141 @@
+// RUN: %clang_cc1 -fsyntax-only -fblocks -fcxx-exceptions -verify %s
+// RUN: %clang_cc1 -fsyntax-only -fblocks -verify -x c -std=c23 %s
+
+#if !__has_attribute(nonblocking)
+#error "the 'nonblocking' attribute is not available"
+#endif
+
+// --- ATTRIBUTE
@@ -8,6 +8,9 @@
Overview
+**NOTE:** This is a design document and the feature is not available for users
yet.
+Please find :doc:`BoundsSafetyImplPlans` for more details.
AaronBallman wrote:
```suggestion
Please see :doc:`BoundsSafetyImplPlans` for
https://github.com/AaronBallman approved this pull request.
Thank you for updating the documentation, the changes LGTM (had one minor
suggestion you can take or leave as you'd like). I think it's better for us to
have the documentation than not, so I'm fine with this approach of letting
https://github.com/AaronBallman edited
https://github.com/llvm/llvm-project/pull/95964
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Aaron Ballman
Date: 2024-06-20T13:29:36-04:00
New Revision: 6bc71cdd32de0add80d620b1342b5549efff363a
URL:
https://github.com/llvm/llvm-project/commit/6bc71cdd32de0add80d620b1342b5549efff363a
DIFF:
https://github.com/llvm/llvm-project/commit/6bc71cdd32de0add80d620b1342b5549efff363a.diff
@@ -141,3 +141,15 @@ namespace r360308_regression {
return a == b;
}
}
+
+namespace GH95598 {
+template
+struct __is_pointer {};
+// expected-warning@-1 {{keyword '__is_pointer' will be made available as an
identifier for the remainder of the translation unit}}
yronglin wrote:
@cor3ntin @AaronBallman Thanks for the suggestions, and sorry for the slow
action. I've added an `annot_module_name` token in the latest update, WDYT?
https://github.com/llvm/llvm-project/pull/90574
___
cfe-commits mailing list
aemerson wrote:
Reverse ping
https://github.com/llvm/llvm-project/pull/88499
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
aemerson wrote:
Reverse ping
https://github.com/llvm/llvm-project/pull/88901
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
aemerson wrote:
Reverse ping
https://github.com/llvm/llvm-project/pull/88710
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Fznamznon wrote:
@tbaederr , I noticed that all buildbot failures relate to the run with the new
constant interpreter. I was wondering if you could see if I did something
wrong? For example, embed by default yields values of type `unsigned char`.
However when expanding in
https://github.com/Xazax-hun approved this pull request.
https://github.com/llvm/llvm-project/pull/96140
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
AaronBallman wrote:
Do you need someone to land these changes on your behalf?
https://github.com/llvm/llvm-project/pull/93913
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Sirraide wrote:
> Clang changes LGTM modulo the dependency scanner bits.
Alright, I’ll wait for a reply from @jansvoboda11 then
https://github.com/llvm/llvm-project/pull/88265
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://github.com/shiltian updated
https://github.com/llvm/llvm-project/pull/94576
>From 01443b5f9d0b8d96b1d9e874d08c0464e9114502 Mon Sep 17 00:00:00 2001
From: Shilei Tian
Date: Thu, 20 Jun 2024 12:35:35 -0400
Subject: [PATCH] [Clang][AMDGPU] Add builtins for instrinsic
https://github.com/AaronBallman approved this pull request.
LGTM; when we add `-fbounds-safety` in the future, we may want the language
option to become an enumeration so we can distinguish between the experimental,
non-experimental, and disabled bounds safety options, but we can cross that
AaronBallman wrote:
> @AaronBallman We haven't exposed the `-fbounds-safety` flag yet. The idea was
> to guard it under the experimental flag called `-fexperimental-bounds-safety`
> as a CC1 flag for testing until we have the full feature available (the
> feature is work-in-progress and is
farzonl wrote:
@efriedma-quic if you have time could you take a look? Thanks in advance!
https://github.com/llvm/llvm-project/pull/95999
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://github.com/AaronBallman approved this pull request.
Clang changes LGTM modulo the dependency scanner bits.
https://github.com/llvm/llvm-project/pull/88265
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
@@ -8086,23 +8086,21 @@ static void HandleNeonVectorTypeAttr(QualType ,
const ParsedAttr ,
// Target must have NEON (or MVE, whose vectors are similar enough
// not to need a separate attribute)
- if (!(S.Context.getTargetInfo().hasFeature("neon") ||
-
AaronBallman wrote:
> > Why gnu99 mode and not gnu89 mode? I see GCC has that behavior, but I'm not
> > certain why.
>
> We went over this a while back: [#88265
> (comment)](https://github.com/llvm/llvm-project/pull/88265#discussion_r1571088366)
THAT is why this was so familiar to me! :-D
AaronBallman wrote:
Thank you for the fix! FWIW, I don't think this is NFC as it is changing test
behavior (fixing a memory leak); please be sure to get a proper code review for
functional changes. The changes are reasonable enough, but one question I have
is: why not stack allocate the
etcwilde wrote:
@danielotero Thank you for your contribution.
https://github.com/llvm/llvm-project/pull/95979
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
github-actions[bot] wrote:
@danielotero Congratulations on having your first Pull Request (PR) merged into
the LLVM Project!
Your changes will be combined with recent changes from other authors, then
tested
by our [build bots](https://lab.llvm.org/buildbot/). If there is a problem with
a
https://github.com/etcwilde closed
https://github.com/llvm/llvm-project/pull/95979
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Daniel Otero
Date: 2024-06-20T09:19:54-07:00
New Revision: 651d44d3dac68192bc4fa37a53e13ae3211c9de0
URL:
https://github.com/llvm/llvm-project/commit/651d44d3dac68192bc4fa37a53e13ae3211c9de0
DIFF:
https://github.com/llvm/llvm-project/commit/651d44d3dac68192bc4fa37a53e13ae3211c9de0.diff
https://github.com/Endilll updated
https://github.com/llvm/llvm-project/pull/96168
>From 7ac89c2758965fec2dd007954735e59d793d6a59 Mon Sep 17 00:00:00 2001
From: Vlad Serebrennikov
Date: Thu, 20 Jun 2024 14:07:37 +0300
Subject: [PATCH 1/4] [clang] Add test for CWG2811 "Clarify "use" of main"
github-actions[bot] wrote:
@hdoc Congratulations on having your first Pull Request (PR) merged into the
LLVM Project!
Your changes will be combined with recent changes from other authors, then
tested
by our [build bots](https://lab.llvm.org/buildbot/). If there is a problem with
a build,
https://github.com/AaronBallman closed
https://github.com/llvm/llvm-project/pull/91100
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: hdoc
Date: 2024-06-20T12:14:51-04:00
New Revision: af6acd7442646fde56de919964bd52d7bb7922b2
URL:
https://github.com/llvm/llvm-project/commit/af6acd7442646fde56de919964bd52d7bb7922b2
DIFF:
https://github.com/llvm/llvm-project/commit/af6acd7442646fde56de919964bd52d7bb7922b2.diff
LOG:
https://github.com/compnerd approved this pull request.
https://github.com/llvm/llvm-project/pull/95979
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/paulwalker-arm approved this pull request.
https://github.com/llvm/llvm-project/pull/95787
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
labrinea wrote:
closing in favor of https://github.com/llvm/llvm-project/pull/92882
https://github.com/llvm/llvm-project/pull/92319
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/labrinea closed
https://github.com/llvm/llvm-project/pull/92319
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/MitalAshok updated
https://github.com/llvm/llvm-project/pull/95580
>From ef42900b5e3533839c126cb9e6b19d7dcc7806c6 Mon Sep 17 00:00:00 2001
From: Mital Ashok
Date: Fri, 14 Jun 2024 19:14:21 +0100
Subject: [PATCH 1/5] [Clang] [SemaCXX] Diagnose unknown std::initializer_list
@@ -0,0 +1,141 @@
+// RUN: %clang_cc1 -fsyntax-only -fblocks -fcxx-exceptions -verify %s
+// RUN: %clang_cc1 -fsyntax-only -fblocks -verify -x c -std=c23 %s
+
+#if !__has_attribute(nonblocking)
+#error "the 'nonblocking' attribute is not available"
+#endif
+
+// --- ATTRIBUTE
https://github.com/MitalAshok edited
https://github.com/llvm/llvm-project/pull/95580
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -17,7 +25,7 @@
// CPP-CHECK-NEXT:[[TMP1:%.*]] = shl nuw nsw i64 [[TMP0]], 4
// CPP-CHECK-NEXT:ret i64 [[TMP1]]
//
-uint64_t test_svcntb()
+uint64_t test_svcntb(void) MODE_ATTR
sdesmalen-arm wrote:
I don't think so.
The ACLE describes that the SME
https://github.com/nikic closed https://github.com/llvm/llvm-project/pull/96175
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Nikita Popov
Date: 2024-06-20T17:41:35+02:00
New Revision: 49ae2dcf369e0fd76e4bb18f1c84e6716ff0ec4b
URL:
https://github.com/llvm/llvm-project/commit/49ae2dcf369e0fd76e4bb18f1c84e6716ff0ec4b
DIFF:
https://github.com/llvm/llvm-project/commit/49ae2dcf369e0fd76e4bb18f1c84e6716ff0ec4b.diff
asb wrote:
My understanding from the sync-up call just now is you're planning to drop
lr/sc intrinsics due to the concerns listed in this thread (which I share), but
will keep pushing the zawrs intrinsics.
https://github.com/llvm/llvm-project/pull/94578
https://github.com/jroelofs closed
https://github.com/llvm/llvm-project/pull/96007
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/tmatheson-arm approved this pull request.
https://github.com/llvm/llvm-project/pull/96007
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -108,6 +108,12 @@ Changes to the AArch64 Backend
in ``standard`` being equal to ``bti+pac-ret+pc`` when ``+pauth-lr``
is passed as part of ``-mcpu=`` options.
+* SVE and SVE2 have been moved to the default extensions list for ARMv9.0,
+ making them optional per the
https://github.com/jroelofs updated
https://github.com/llvm/llvm-project/pull/96007
>From b18411c28ed9b89660fac6311596408dafb7f40e Mon Sep 17 00:00:00 2001
From: Jon Roelofs
Date: Tue, 18 Jun 2024 15:26:32 -0700
Subject: [PATCH 1/3] [llvm][AArch64] SVE2 is an optional feature in ARMv9.0a
---
=?utf-8?q?Donát?= Nagy ,
=?utf-8?q?Donát?= Nagy
Message-ID:
In-Reply-To:
@@ -342,31 +342,33 @@ void SizeofExpressionCheck::check(const
MatchFinder::MatchResult ) {
if (DenominatorSize > CharUnits::Zero() &&
!NumeratorSize.isMultipleOf(DenominatorSize)) {
-
https://github.com/HerrCai0907 updated
https://github.com/llvm/llvm-project/pull/96199
>From 03bc40a343227cb839ec908492c0d904be9af654 Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Thu, 20 Jun 2024 14:36:51 +
Subject: [PATCH 1/2] [clang-tidy] align all help message in run-clang-tidy
@@ -1781,7 +1781,13 @@ void SVEEmitter::createStreamingAttrs(raw_ostream ,
ACLEKind Kind) {
uint64_t VerifyRuntimeMode = getEnumValueForFlag("VerifyRuntimeMode");
uint64_t IsStreamingCompatibleFlag =
getEnumValueForFlag("IsStreamingCompatible");
+
for (auto :
@@ -2264,6 +2278,18 @@ let TargetGuard = "sve2p1" in {
defm SVPMOV_TO_VEC_LANE_D : PMOV_TO_VEC<"svpmov", "lUl",
"aarch64_sve_pmov_to_vector_lane" ,[], ImmCheck1_7>;
}
+let TargetGuard = "sve2p1|sme2p1" in {
+ // DUPQ
+ def SVDUP_LANEQ_B : SInst<"svdup_laneq[_{d}]",
https://github.com/paulwalker-arm commented:
With the change of default it's very hard to check everything but we've already
agreed there'll need to be a full audit once all the inflight work has landed.
I did spot one thing though:
Should the integer svclamp and svrevd builtins be
@@ -286,17 +290,20 @@ let TargetGuard = "sve,f64mm,bf16" in {
}
let TargetGuard = "sve,bf16" in {
+ def SVBFMMLA : SInst<"svbfmmla[_{0}]", "MMdd", "b", MergeNone,
"aarch64_sve_bfmmla", [IsOverloadNone]>;
+}
+
+let TargetGuard = "(sve|sme),bf16" in {
https://github.com/paulwalker-arm edited
https://github.com/llvm/llvm-project/pull/95787
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -17,7 +25,7 @@
// CPP-CHECK-NEXT:[[TMP1:%.*]] = shl nuw nsw i64 [[TMP0]], 4
// CPP-CHECK-NEXT:ret i64 [[TMP1]]
//
-uint64_t test_svcntb()
+uint64_t test_svcntb(void) MODE_ATTR
paulwalker-arm wrote:
Is there a problem we need to worry about with
ilya-biryukov wrote:
> Yeah, I think this patch may be conceptually good except the extra memory
> using. But benchmarking, I tried it locally but didn't find observable
> effects. So I'd like to land this after 19's branching to give it more baking
> time.
Do you mean that no large
llvmbot wrote:
@llvm/pr-subscribers-clang-tidy
Author: Congcong Cai (HerrCai0907)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/96203.diff
3 Files Affected:
- (modified) clang-tools-extra/clang-tidy/misc/UseInternalLinkageCheck.cpp
(+10-2)
- (modified)
https://github.com/HerrCai0907 created
https://github.com/llvm/llvm-project/pull/96203
None
>From 669205c0f659239c58a3bde3ddadabb0a8ecbad8 Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Thu, 20 Jun 2024 15:05:57 +
Subject: [PATCH] [clang-tidy] add fixhint for misc-use-internal-linkage
shiltian wrote:
> maybe add a test for non-constant offset?
Hmm, I thought I added all tests...My bad. Will add it right away.
https://github.com/llvm/llvm-project/pull/94576
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://github.com/shiltian closed
https://github.com/llvm/llvm-project/pull/95276
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Shilei Tian
Date: 2024-06-20T11:01:54-04:00
New Revision: e3eb12cce97fa75d1d2443bcc2c2b26aa660fe34
URL:
https://github.com/llvm/llvm-project/commit/e3eb12cce97fa75d1d2443bcc2c2b26aa660fe34
DIFF:
https://github.com/llvm/llvm-project/commit/e3eb12cce97fa75d1d2443bcc2c2b26aa660fe34.diff
yxsamliu wrote:
maybe add a test for non-constant offset?
https://github.com/llvm/llvm-project/pull/94576
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -964,7 +964,7 @@ def err_main_global_variable :
def warn_main_redefined : Warning<"variable named 'main' with external linkage
"
"has undefined behavior">, InGroup;
def ext_main_used : Extension<
- "ISO C++ does not allow 'main' to be used by a program">, InGroup;
+
1001 - 1100 of 443486 matches
Mail list logo