https://github.com/jroelofs approved this pull request.
https://github.com/llvm/llvm-project/pull/110297
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jroelofs wrote:
@mgorny https://github.com/llvm/llvm-project/pull/109601
https://github.com/llvm/llvm-project/pull/109306
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jroelofs wrote:
@compnerd @etcwilde WDYT about conditionalizing the new dependency with an `if
(TARGET vt_gen)` instead of doing the `add_subdirectory()` dance?
https://github.com/llvm/llvm-project/pull/109601
___
cfe-commits mailing list
cfe-commits@
https://github.com/jroelofs approved this pull request.
https://github.com/llvm/llvm-project/pull/109306
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -73,8 +73,6 @@ enum CPUFeatures {
FEAT_SSBS,
FEAT_SSBS2,
FEAT_BTI,
- FEAT_LS64,
- FEAT_LS64_V,
jroelofs wrote:
We could do that automatically every release by putting the version number in
its name.
https://github.com/llvm/llvm-project/pull/108024
@@ -33,7 +33,6 @@ enum CPUFeatures {
FEAT_FP,
FEAT_SIMD,
FEAT_CRC,
- FEAT_SHA1,
jroelofs wrote:
Here are the other threads:
https://github.com/llvm/llvm-project/pull/108024#discussion_r1757249346
https://github.com/llvm/llvm-project/pull/88965#discussio
@@ -73,8 +73,6 @@ enum CPUFeatures {
FEAT_SSBS,
FEAT_SSBS2,
FEAT_BTI,
- FEAT_LS64,
- FEAT_LS64_V,
jroelofs wrote:
https://github.com/llvm/llvm-project/pull/88965#discussion_r1567997481
> Did any implementation ship __aarch64_cpu_features in a way that
https://github.com/jroelofs deleted
https://github.com/llvm/llvm-project/pull/108024
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -73,8 +73,6 @@ enum CPUFeatures {
FEAT_SSBS,
FEAT_SSBS2,
FEAT_BTI,
- FEAT_LS64,
- FEAT_LS64_V,
jroelofs wrote:
https://github.com/llvm/llvm-project/pull/88965#discussion_r1567997481
https://github.com/llvm/llvm-project/pull/108024
_
Author: Jon Roelofs
Date: 2024-09-11T13:37:45-07:00
New Revision: b3f3c0c63358b412348022d10308b97332d02bcd
URL:
https://github.com/llvm/llvm-project/commit/b3f3c0c63358b412348022d10308b97332d02bcd
DIFF:
https://github.com/llvm/llvm-project/commit/b3f3c0c63358b412348022d10308b97332d02bcd.diff
L
https://github.com/jroelofs deleted
https://github.com/llvm/llvm-project/pull/100278
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -404,142 +370,174 @@ enum ArmSMEState : unsigned {
ArmZT0Mask = 0b11 << 2
};
-bool SemaARM::ParseSVEImmChecks(
-CallExpr *TheCall, SmallVector, 3> &ImmChecks) {
- // Perform all the immediate checks for this builtin call.
+bool SemaARM::CheckImmediateArg(CallExpr *Th
@@ -216,6 +216,36 @@ namespace clang {
}
bool isUnsigned() const { return (Flags & UnsignedFlag) != 0; }
bool isQuad() const { return (Flags & QuadFlag) != 0; }
+unsigned getEltSizeInBits() const {
+ switch (getEltType()) {
+ case Int8:
+ case Pol
@@ -216,6 +216,36 @@ namespace clang {
}
bool isUnsigned() const { return (Flags & UnsignedFlag) != 0; }
bool isQuad() const { return (Flags & QuadFlag) != 0; }
+unsigned getEltSizeInBits() const {
+ switch (getEltType()) {
+ case Int8:
+ case Pol
@@ -404,142 +370,174 @@ enum ArmSMEState : unsigned {
ArmZT0Mask = 0b11 << 2
};
-bool SemaARM::ParseSVEImmChecks(
-CallExpr *TheCall, SmallVector, 3> &ImmChecks) {
- // Perform all the immediate checks for this builtin call.
+bool SemaARM::CheckImmediateArg(CallExpr *Th
https://github.com/jroelofs approved this pull request.
https://github.com/llvm/llvm-project/pull/106312
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jroelofs approved this pull request.
https://github.com/llvm/llvm-project/pull/101712
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jroelofs wrote:
/cherry-pick 019ef522756886caa258daf68d877f84abc1b878
https://github.com/llvm/llvm-project/pull/103392
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jroelofs closed
https://github.com/llvm/llvm-project/pull/103392
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jroelofs updated
https://github.com/llvm/llvm-project/pull/103392
>From a6af41dd6da847237e79ce57b3847bbc241b2cba Mon Sep 17 00:00:00 2001
From: Jon Roelofs
Date: Tue, 13 Aug 2024 10:59:25 -0700
Subject: [PATCH 1/2] [clang][AArch64] Point the nofp ABI check diagnostics at
the
https://github.com/jroelofs milestoned
https://github.com/llvm/llvm-project/pull/103392
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jroelofs created
https://github.com/llvm/llvm-project/pull/103392
... whereever we have the Decl for it, and even when we don't keep the
SourceLocation of it aimed at the call site.
Fixes: #102983
>From a6af41dd6da847237e79ce57b3847bbc241b2cba Mon Sep 17 00:00:00 2001
From:
@@ -2397,6 +2397,1262 @@ class UnsafeBufferUsageReporter : public
UnsafeBufferUsageHandler {
};
} // namespace
+//
=
+
+namespace FXAnalysis {
+
+enum class DiagnosticID : uint8_t {
+ None = 0, // s
jroelofs wrote:
That's annoying. You can always put the diff from this thing's comment on your
clipboard, and then do `pbpaste | patch -p1` from the top-level source dir.
https://github.com/llvm/llvm-project/pull/99656#issuecomment-2241192368
https://github.com/llvm/llvm-project/pull/99656
___
@@ -0,0 +1,194 @@
+// RUN: %clang_cc1 -fsyntax-only -fblocks -fcxx-exceptions -verify %s
+// These are in a separate file because errors (e.g. incompatible attributes)
currently prevent
+// the AnalysisBasedWarnings pass from running at all.
+
+// This diagnostic is re-enabled an
@@ -2397,6 +2397,1262 @@ class UnsafeBufferUsageReporter : public
UnsafeBufferUsageHandler {
};
} // namespace
+//
=
+
+namespace FXAnalysis {
+
+enum class DiagnosticID : uint8_t {
+ None = 0, // s
@@ -2397,6 +2397,1262 @@ class UnsafeBufferUsageReporter : public
UnsafeBufferUsageHandler {
};
} // namespace
+//
=
+
+namespace FXAnalysis {
+
+enum class DiagnosticID : uint8_t {
+ None = 0, // s
https://github.com/jroelofs closed
https://github.com/llvm/llvm-project/pull/101243
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jroelofs updated
https://github.com/llvm/llvm-project/pull/101243
>From 61371af08b82e1229c34b43f56772813c2f74b1c Mon Sep 17 00:00:00 2001
From: Jon Roelofs
Date: Tue, 30 Jul 2024 13:56:21 -0700
Subject: [PATCH 1/7] [cmake][llvm] Limit the number of Xcode schemes created
by d
@@ -1423,3 +1423,11 @@ endif()
if (LLVM_INCLUDE_UTILS AND LLVM_INCLUDE_TOOLS)
add_subdirectory(utils/llvm-locstats)
endif()
+
+if (XCODE)
+ set(LLVM_XCODE_EXTRA_TARGET_SCHEMES "" CACHE STRING "Specifies an extra list
of targets to turn into schemes")
jroelo
https://github.com/jroelofs edited
https://github.com/llvm/llvm-project/pull/101243
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jroelofs updated
https://github.com/llvm/llvm-project/pull/101243
>From 61371af08b82e1229c34b43f56772813c2f74b1c Mon Sep 17 00:00:00 2001
From: Jon Roelofs
Date: Tue, 30 Jul 2024 13:56:21 -0700
Subject: [PATCH 1/6] [cmake][llvm] Limit the number of Xcode schemes created
by d
jroelofs wrote:
neat idea. that's doable.
https://github.com/llvm/llvm-project/pull/101243
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jroelofs updated
https://github.com/llvm/llvm-project/pull/101243
>From 61371af08b82e1229c34b43f56772813c2f74b1c Mon Sep 17 00:00:00 2001
From: Jon Roelofs
Date: Tue, 30 Jul 2024 13:56:21 -0700
Subject: [PATCH 1/5] [cmake][llvm] Limit the number of Xcode schemes created
by d
jroelofs wrote:
And you can always manually override them through the scheme editor thing, if
that is a one-off local need.
https://github.com/llvm/llvm-project/pull/101243
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org
https://github.com/jroelofs updated
https://github.com/llvm/llvm-project/pull/101243
>From 61371af08b82e1229c34b43f56772813c2f74b1c Mon Sep 17 00:00:00 2001
From: Jon Roelofs
Date: Tue, 30 Jul 2024 13:56:21 -0700
Subject: [PATCH 1/4] [cmake][llvm] Limit the number of Xcode schemes created
by d
https://github.com/jroelofs approved this pull request.
https://github.com/llvm/llvm-project/pull/100753
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -2397,6 +2397,1262 @@ class UnsafeBufferUsageReporter : public
UnsafeBufferUsageHandler {
};
} // namespace
+//
=
+
+namespace FXAnalysis {
+
+enum class DiagnosticID : uint8_t {
+ None = 0, // s
@@ -4699,7 +4699,7 @@ class FunctionEffect {
private:
LLVM_PREFERRED_TYPE(Kind)
- unsigned FKind : 3;
+ uint8_t FKind : 3;
jroelofs wrote:
"no functional change"
https://llvm.org/docs/Lexicon.html#n
https://github.com/llvm/llvm-project/pull/99656
__
@@ -0,0 +1,19 @@
+// RUN: %clang_cc1 -fsyntax-only -fblocks -fcxx-exceptions -verify %s
+
+#pragma clang diagnostic ignored "-Wperf-constraint-implies-noexcept"
+
+// Objective-C
+@interface OCClass
+- (void)method;
+@end
+
+void nl14(OCClass *oc) [[clang::nonblocking]] {
+
@@ -0,0 +1,194 @@
+// RUN: %clang_cc1 -fsyntax-only -fblocks -fcxx-exceptions -verify %s
+// These are in a separate file because errors (e.g. incompatible attributes)
currently prevent
+// the AnalysisBasedWarnings pass from running at all.
+
+// This diagnostic is re-enabled an
@@ -0,0 +1,19 @@
+// RUN: %clang_cc1 -fsyntax-only -fblocks -fcxx-exceptions -verify %s
+
+#pragma clang diagnostic ignored "-Wperf-constraint-implies-noexcept"
+
+// Objective-C
+@interface OCClass
+- (void)method;
+@end
+
+void nl14(OCClass *oc) [[clang::nonblocking]] {
+
@@ -0,0 +1,194 @@
+// RUN: %clang_cc1 -fsyntax-only -fblocks -fcxx-exceptions -verify %s
+// These are in a separate file because errors (e.g. incompatible attributes)
currently prevent
+// the AnalysisBasedWarnings pass from running at all.
+
+// This diagnostic is re-enabled an
@@ -0,0 +1,194 @@
+// RUN: %clang_cc1 -fsyntax-only -fblocks -fcxx-exceptions -verify %s
+// These are in a separate file because errors (e.g. incompatible attributes)
currently prevent
+// the AnalysisBasedWarnings pass from running at all.
+
+// This diagnostic is re-enabled an
https://github.com/jroelofs edited
https://github.com/llvm/llvm-project/pull/99656
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,194 @@
+// RUN: %clang_cc1 -fsyntax-only -fblocks -fcxx-exceptions -verify %s
+// These are in a separate file because errors (e.g. incompatible attributes)
currently prevent
+// the AnalysisBasedWarnings pass from running at all.
+
+// This diagnostic is re-enabled an
@@ -0,0 +1,194 @@
+// RUN: %clang_cc1 -fsyntax-only -fblocks -fcxx-exceptions -verify %s
+// These are in a separate file because errors (e.g. incompatible attributes)
currently prevent
+// the AnalysisBasedWarnings pass from running at all.
+
+// This diagnostic is re-enabled an
@@ -0,0 +1,194 @@
+// RUN: %clang_cc1 -fsyntax-only -fblocks -fcxx-exceptions -verify %s
+// These are in a separate file because errors (e.g. incompatible attributes)
currently prevent
+// the AnalysisBasedWarnings pass from running at all.
+
+// This diagnostic is re-enabled an
@@ -4699,7 +4699,7 @@ class FunctionEffect {
private:
LLVM_PREFERRED_TYPE(Kind)
- unsigned FKind : 3;
+ uint8_t FKind : 3;
jroelofs wrote:
also, github won't let me comment on the correct lines, but:
```
- FunctionEffect() : FKind(unsigned(Kind::None))
@@ -4699,7 +4699,7 @@ class FunctionEffect {
private:
LLVM_PREFERRED_TYPE(Kind)
- unsigned FKind : 3;
+ uint8_t FKind : 3;
jroelofs wrote:
I'd split this into it's own NFC PR, and land it before the rest.
https://github.com/llvm/llvm-project/pull/99656
_
@@ -2085,6 +2099,37 @@ void
AArch64AsmPrinter::emitMachOIFuncStubHelperBody(Module &M,
.addImm(2),
*STI);
+ if (TM.getTargetTriple().isArm64e()) {
+// autibsp
jroelofs wrote:
https://gi
https://github.com/jroelofs updated
https://github.com/llvm/llvm-project/pull/84704
>From b352a41e1788819afbb15a5b6d79cbf2968dc5dc Mon Sep 17 00:00:00 2001
From: Jon Roelofs
Date: Sun, 10 Mar 2024 16:46:50 -0700
Subject: [PATCH 1/3] [CodeGen][AArch64][FMV] PAC the stub_helper's frame on
arm64e
jroelofs wrote:
I think you also need:
```
diff --git a/llvm/lib/Target/AArch64/AArch64FMV.td
b/llvm/lib/Target/AArch64/AArch64FMV.td
index 7a40c83b2bb2..8f42ece439a7 100644
--- a/llvm/lib/Target/AArch64/AArch64FMV.td
+++ b/llvm/lib/Target/AArch64/AArch64FMV.td
@@ -75,7 +75,7 @@ def : FMVExtensi
Author: Jon Roelofs
Date: 2024-07-18T14:48:11-07:00
New Revision: 6c6baf019a25710cdc3c3971b09cd6332d0379c0
URL:
https://github.com/llvm/llvm-project/commit/6c6baf019a25710cdc3c3971b09cd6332d0379c0
DIFF:
https://github.com/llvm/llvm-project/commit/6c6baf019a25710cdc3c3971b09cd6332d0379c0.diff
L
Author: Jon Roelofs
Date: 2024-07-18T13:48:17-07:00
New Revision: 06518cea3905556d8d1eea4088132ebb234bfdab
URL:
https://github.com/llvm/llvm-project/commit/06518cea3905556d8d1eea4088132ebb234bfdab
DIFF:
https://github.com/llvm/llvm-project/commit/06518cea3905556d8d1eea4088132ebb234bfdab.diff
L
Author: Jon Roelofs
Date: 2024-07-18T13:25:23-07:00
New Revision: 05275b05ca58e4d015eea1503f120e6967ef1b91
URL:
https://github.com/llvm/llvm-project/commit/05275b05ca58e4d015eea1503f120e6967ef1b91
DIFF:
https://github.com/llvm/llvm-project/commit/05275b05ca58e4d015eea1503f120e6967ef1b91.diff
L
jroelofs wrote:
This broke the aix-ppc64 bot:
https://lab.llvm.org/buildbot/#/builders/64/builds/335
Must have a different version of `sed` than I do :(
https://github.com/llvm/llvm-project/pull/99365
___
cfe-commits mailing list
cfe-commits@lists.ll
https://github.com/jroelofs approved this pull request.
Strong support for this from me. Thanks!
https://github.com/llvm/llvm-project/pull/99522
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe
https://github.com/jroelofs closed
https://github.com/llvm/llvm-project/pull/99365
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,106 @@
+; RUN: llc -mtriple arm64e-apple-darwin \
jroelofs wrote:
does update_test_checks do horrible things to the CHECK lines in this?
https://github.com/llvm/llvm-project/pull/97647
___
cfe-commits mailin
https://github.com/jroelofs created
https://github.com/llvm/llvm-project/pull/99365
Whenever these tests change, it's difficult to see why they don't match, and
the diff after you've fixed them isn't easy to grok. By splitting them with a
sed pipe, we fix both issues simultaneously.
>From 91
https://github.com/jroelofs approved this pull request.
https://github.com/llvm/llvm-project/pull/98698
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jroelofs wrote:
This one looks fairly benign, and while I agree that it is good to upstream
things wherever possible, my slight objection rests mostly on two principles:
* In general the community doesn't take behavior-changing patches purely on the
basis of eliminating merge conflicts for cod
@@ -161,14 +161,10 @@ struct CpuInfo {
StringRef Name; // Name, as written for -mcpu.
const ArchInfo &Arch;
AArch64::ExtensionBitset
- DefaultExtensions; // Default extensions for this CPU. These will be
- // ORd with the architecture defaults
https://github.com/jroelofs edited
https://github.com/llvm/llvm-project/pull/98267
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jroelofs commented:
lgtm
https://github.com/llvm/llvm-project/pull/98550
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -161,14 +161,10 @@ struct CpuInfo {
StringRef Name; // Name, as written for -mcpu.
const ArchInfo &Arch;
AArch64::ExtensionBitset
- DefaultExtensions; // Default extensions for this CPU. These will be
- // ORd with the architecture defaults
@@ -161,14 +161,10 @@ struct CpuInfo {
StringRef Name; // Name, as written for -mcpu.
const ArchInfo &Arch;
AArch64::ExtensionBitset
- DefaultExtensions; // Default extensions for this CPU. These will be
- // ORd with the architecture defaults
https://github.com/jroelofs closed
https://github.com/llvm/llvm-project/pull/98267
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -161,14 +161,10 @@ struct CpuInfo {
StringRef Name; // Name, as written for -mcpu.
const ArchInfo &Arch;
AArch64::ExtensionBitset
- DefaultExtensions; // Default extensions for this CPU. These will be
- // ORd with the architecture defaults
https://github.com/jroelofs updated
https://github.com/llvm/llvm-project/pull/98267
>From 2ca4c255338f1c41a2cad929604c70febf6c7eaf Mon Sep 17 00:00:00 2001
From: Jon Roelofs
Date: Tue, 9 Jul 2024 19:47:27 -0700
Subject: [PATCH 1/5] [llvm][AArch64] apple-m4 is armv9.2-a
But since SVE and friend
https://github.com/jroelofs updated
https://github.com/llvm/llvm-project/pull/98267
>From 2ca4c255338f1c41a2cad929604c70febf6c7eaf Mon Sep 17 00:00:00 2001
From: Jon Roelofs
Date: Tue, 9 Jul 2024 19:47:27 -0700
Subject: [PATCH 1/5] [llvm][AArch64] apple-m4 is armv9.2-a
But since SVE and friend
https://github.com/jroelofs edited
https://github.com/llvm/llvm-project/pull/98267
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -797,7 +797,7 @@ def HasV8_9aOps : Architecture64<8, 9, "a", "v8.9a",
!listconcat(HasV8_8aOps.DefaultExts, [FeatureSPECRES2, FeatureCSSC,
FeatureRASv2])>;
def HasV9_0aOps : Architecture64<9, 0, "a", "v9a",
- [HasV8_5aOps, FeatureMEC],
jroelofs wrote:
https://github.com/jroelofs updated
https://github.com/llvm/llvm-project/pull/98267
>From 2ca4c255338f1c41a2cad929604c70febf6c7eaf Mon Sep 17 00:00:00 2001
From: Jon Roelofs
Date: Tue, 9 Jul 2024 19:47:27 -0700
Subject: [PATCH 1/4] [llvm][AArch64] apple-m4 is armv9.2-a
But since SVE and friend
https://github.com/jroelofs approved this pull request.
https://github.com/llvm/llvm-project/pull/98097
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,23 @@
+// REQUIRES: aarch64-registered-target
jroelofs wrote:
Is this a new folder? There should be a `lit.local.cfg` in there that takes
care of this:
```
if "AArch64" not in config.root.targets:
config.unsupported = True
```
If not, please add
jroelofs wrote:
If the diagnostics are downstream-only, does this really need go upstream?
https://github.com/llvm/llvm-project/pull/98091
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commi
https://github.com/jroelofs approved this pull request.
https://github.com/llvm/llvm-project/pull/98058
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jroelofs approved this pull request.
https://github.com/llvm/llvm-project/pull/97761
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jroelofs approved this pull request.
https://github.com/llvm/llvm-project/pull/97829
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,24 @@
+// REQUIRES: aarch64-registered-target
jroelofs wrote:
these should probably all go in an `aarch64` folder with a `lit.local.cfg`:
```
targets = set(config.root.targets_to_build.split())
if not "AArch64" in targets:
config.unsupported = Tru
jroelofs wrote:
> The new tests are autogenerated, but I am not merging the script that creates
> them because there isn't a good place for it.
Mind sticking it in a gist at least so folks can use it for downstream
subtargets?
https://github.com/llvm/llvm-project/pull/97829
__
@@ -0,0 +1,18 @@
+// RUN: %clang_cc1 %s -Wobjc-prefixes=NS,NSCF,NSURL -fsyntax-only -verify
jroelofs wrote:
is it worth adding a `-Wobjc-forbidden-prefixes=NS`, so you can e.g. forbid
things outside of Foundation from defining things with the `NS` prefix?
https
https://github.com/jroelofs closed
https://github.com/llvm/llvm-project/pull/96249
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jroelofs updated
https://github.com/llvm/llvm-project/pull/96249
>From ee1389a36aac9eecf00513d98cc99787b2cfe17a Mon Sep 17 00:00:00 2001
From: Jon Roelofs
Date: Thu, 20 Jun 2024 16:26:45 -0700
Subject: [PATCH 1/5] [llvm][AArch64][TableGen] Create a ProcessorAlias record.
NFC
https://github.com/jroelofs edited
https://github.com/llvm/llvm-project/pull/96249
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jroelofs edited
https://github.com/llvm/llvm-project/pull/96249
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jroelofs updated
https://github.com/llvm/llvm-project/pull/96249
>From ee1389a36aac9eecf00513d98cc99787b2cfe17a Mon Sep 17 00:00:00 2001
From: Jon Roelofs
Date: Thu, 20 Jun 2024 16:26:45 -0700
Subject: [PATCH 1/4] [llvm][AArch64][TableGen] Create a ProcessorAlias record.
NFC
@@ -5,11 +5,11 @@
// RUN: not %clang_cc1 -triple arm64--- -target-cpu not-a-cpu -fsyntax-only %s
2>&1 | FileCheck %s --check-prefix AARCH64
// AARCH64: error: unknown target CPU 'not-a-cpu'
-// AARCH64-NEXT: note: valid target CPU values are: generic, cortex-a35,
cortex-a34,
@@ -304,8 +304,21 @@ struct Alias {
StringRef Name;
};
-inline constexpr Alias CpuAliases[] = {{"cobalt-100", "neoverse-n2"},
- {"grace", "neoverse-v2"}};
+inline constexpr Alias CpuAliases[] = {
jroelofs wrote:
Since I
@@ -304,8 +304,21 @@ struct Alias {
StringRef Name;
};
-inline constexpr Alias CpuAliases[] = {{"cobalt-100", "neoverse-n2"},
- {"grace", "neoverse-v2"}};
+inline constexpr Alias CpuAliases[] = {
+{"cobalt-100", "neoverse-n2"},
+
@@ -88,10 +88,14 @@ StringRef AArch64::getArchExtFeature(StringRef ArchExt) {
void AArch64::fillValidCPUArchList(SmallVectorImpl &Values) {
for (const auto &C : CpuInfos)
- Values.push_back(C.Name);
+Values.push_back(C.Name);
for (const auto &Alias : CpuAliases
https://github.com/jroelofs approved this pull request.
https://github.com/llvm/llvm-project/pull/96795
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -21,7 +21,7 @@
// RUN: %clang --target=aarch64 -march=armv8a+fp16fml -### -c %s 2>&1 |
FileCheck -check-prefix=GENERICV8A-FP16FML %s
jroelofs wrote:
oh, ok. TIL.
https://github.com/llvm/llvm-project/pull/96795
_
@@ -315,37 +315,37 @@
// RUN: %clang -target aarch64 -mcpu=thunderx2t99 -### -c %s 2>&1 | FileCheck
-check-prefix=CHECK-MCPU-THUNDERX2T99 %s
// RUN: %clang -target aarch64 -mcpu=a64fx -### -c %s 2>&1 | FileCheck
-check-prefix=CHECK-MCPU-A64FX %s
// RUN: %clang -target aarch64
@@ -21,7 +21,7 @@
// RUN: %clang --target=aarch64 -march=armv8a+fp16fml -### -c %s 2>&1 |
FileCheck -check-prefix=GENERICV8A-FP16FML %s
jroelofs wrote:
needs a `// REQUIRES: aarch64-registered-target`. Ditto for all the rest of the
`aarch64-*` files being mo
jroelofs wrote:
Looks like the new tests in `Driver` are missing `// REQUIRES:
aarch64-registered-target`
https://github.com/llvm/llvm-project/pull/95805
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/li
Author: Jon Roelofs
Date: 2024-06-24T15:35:53-07:00
New Revision: b3c668b0055717633503ed26787037d9e3499781
URL:
https://github.com/llvm/llvm-project/commit/b3c668b0055717633503ed26787037d9e3499781
DIFF:
https://github.com/llvm/llvm-project/commit/b3c668b0055717633503ed26787037d9e3499781.diff
L
@@ -1130,7 +1130,8 @@ INSTANTIATE_TEST_SUITE_P(
AArch64::AEK_MTE, AArch64::AEK_SSBS,
AArch64::AEK_FP16,AArch64::AEK_FP16FML,
AArch64::AEK_SB, AArch64::AEK_JSCVT,
- AArch64::AEK_FCMA,
1 - 100 of 400 matches
Mail list logo
