Author: David Spickett
Date: 2020-07-15T09:37:09+01:00
New Revision: fe5912249efa1ec5e6aa6e565f722dd4d33d1e54
URL:
https://github.com/llvm/llvm-project/commit/fe5912249efa1ec5e6aa6e565f722dd4d33d1e54
DIFF:
https://github.com/llvm/llvm-project/commit/fe5912249efa1ec5e6aa6e565f722dd4d33d1e54.diff
Author: David Spickett
Date: 2020-06-22T09:41:13+01:00
New Revision: 028571d60843cb87e2637ef69ee09090d4526c62
URL:
https://github.com/llvm/llvm-project/commit/028571d60843cb87e2637ef69ee09090d4526c62
DIFF:
https://github.com/llvm/llvm-project/commit/028571d60843cb87e2637ef69ee09090d4526c62.diff
Author: David Spickett
Date: 2020-06-22T14:18:54+01:00
New Revision: f570d5810485fa6fb2e1009f795a899d79bd429f
URL:
https://github.com/llvm/llvm-project/commit/f570d5810485fa6fb2e1009f795a899d79bd429f
DIFF:
https://github.com/llvm/llvm-project/commit/f570d5810485fa6fb2e1009f795a899d79bd429f.diff
//45.33.8.238/mac/15950/step_7.txt Please take a look and revert if it
> takes a while to investigate.
>
> Nico
>
> On Mon, Jun 22, 2020 at 4:41 AM David Spickett via cfe-commits
> wrote:
>>
>>
>> Author: David Spickett
>> Date: 2020-06-22T09:41:13+01:
Author: David Spickett
Date: 2020-06-25T09:33:43+01:00
New Revision: d6efc9811646edbfe13f06c2676fb469f1c155b1
URL:
https://github.com/llvm/llvm-project/commit/d6efc9811646edbfe13f06c2676fb469f1c155b1
DIFF:
https://github.com/llvm/llvm-project/commit/d6efc9811646edbfe13f06c2676fb469f1c155b1.diff
Author: David Spickett
Date: 2022-09-08T09:49:48Z
New Revision: e428baf0019e5292d943a8e37bf08f1192a1870c
URL:
https://github.com/llvm/llvm-project/commit/e428baf0019e5292d943a8e37bf08f1192a1870c
DIFF:
https://github.com/llvm/llvm-project/commit/e428baf0019e5292d943a8e37bf08f1192a1870c.diff
LOG
DavidSpickett wrote:
I think the commit title would make more sense at a glance if it was saying
what is being done instead of what's being fixed. How about:
```
[llvm][ARM] Emit MVE .arch_extension after .fpu directive if it does not
include MVE features
```
Then the commit message is fine as
@@ -0,0 +1,56 @@
+; REQUIRES: arm-registered-target
+; RUN: %clang --target=arm-none-eabi -mcpu=cortex-m85 -mfloat-abi=hard
-save-temps=obj -S -o - %s | FileCheck %s
+; RUN: %clang --target=arm-none-eabi -mcpu=cortex-m55 -mfloat-abi=hard
-save-temps=obj -S -o - %s | FileCheck %s
@@ -0,0 +1,56 @@
+; REQUIRES: arm-registered-target
+; RUN: %clang --target=arm-none-eabi -mcpu=cortex-m85 -mfloat-abi=hard
-save-temps=obj -S -o - %s | FileCheck %s
+; RUN: %clang --target=arm-none-eabi -mcpu=cortex-m55 -mfloat-abi=hard
-save-temps=obj -S -o - %s | FileCheck %s
@@ -0,0 +1,56 @@
+; REQUIRES: arm-registered-target
+; RUN: %clang --target=arm-none-eabi -mcpu=cortex-m85 -mfloat-abi=hard
-save-temps=obj -S -o - %s | FileCheck %s
+; RUN: %clang --target=arm-none-eabi -mcpu=cortex-m55 -mfloat-abi=hard
-save-temps=obj -S -o - %s | FileCheck %s
@@ -0,0 +1,56 @@
+; REQUIRES: arm-registered-target
+; RUN: %clang --target=arm-none-eabi -mcpu=cortex-m85 -mfloat-abi=hard
-save-temps=obj -S -o - %s | FileCheck %s
+; RUN: %clang --target=arm-none-eabi -mcpu=cortex-m55 -mfloat-abi=hard
-save-temps=obj -S -o - %s | FileCheck %s
https://github.com/DavidSpickett edited
https://github.com/llvm/llvm-project/pull/71545
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
DavidSpickett wrote:
(maybe this is for a later patch but...)
Does this patch include the clang side options for the ABI or does
`-march=...+no-fp-armv8` already work for that?
https://github.com/llvm/llvm-project/pull/74460
___
cfe-commits mailing l
DavidSpickett wrote:
That makes sense. Didn't realise we had an `fp` flag, I don't see it in
`llvm/include/llvm/TargetParser/AArch64TargetParser.h`.
https://github.com/llvm/llvm-project/pull/74460
___
cfe-commits mailing list
cfe-commits@lists.llvm.o
DavidSpickett wrote:
I also made a small change in this area the other day and got a mountain of
clang-format-diff changes.
So in case it's not clear, you can ignore the formatter and it won't block the
approval/merge. Makes sense not to fill the change with unrelated formatting
changes.
htt
@@ -102,7 +102,7 @@ Changes to the AMDGPU Backend
* Implemented :ref:`llvm.get.rounding `
-* Added support for Cortex-A520, Cortex-A720 and Cortex-X4 CPUs.
+* Added support for Cortex-A520, Cortex-A720, Cortex-X4 and Cortex-M52 CPUs.
DavidSpickett wrote:
Thi
DavidSpickett wrote:
Going by the page (didn't see a link to a manual, maybe I missed it), MVE and
FPU are optional.
"Optional Helium technology (M-profile Vector Extension) supporting up to:"
"Optional FPU with support for half precision (fp16), single precision (fp32)
and double precision (f
DavidSpickett wrote:
> Yes, that's correct. We enable all mandatory and optional architecture
> extensions, with the exception of crypto.
Cool.
That said then, should CDE be added?
```
Accelerator support
Optional coprocessor interface (64-bit) supporting up to 8 coprocessor units
for c
@@ -896,9 +896,13 @@ Arm and AArch64 Support
Support has been added for the following processors (-mcpu identifiers in
parenthesis):
- * Arm Cortex-A520 (cortex-a520).
- * Arm Cortex-A720 (cortex-a720).
- * Arm Cortex-X4 (cortex-x4).
+ --target=arm
+ * Arm Cortex-M
@@ -896,9 +896,13 @@ Arm and AArch64 Support
Support has been added for the following processors (-mcpu identifiers in
parenthesis):
- * Arm Cortex-A520 (cortex-a520).
- * Arm Cortex-A720 (cortex-a720).
- * Arm Cortex-X4 (cortex-x4).
+ --target=arm
+ * Arm Cortex-M
@@ -896,9 +896,13 @@ Arm and AArch64 Support
Support has been added for the following processors (-mcpu identifiers in
parenthesis):
- * Arm Cortex-A520 (cortex-a520).
- * Arm Cortex-A720 (cortex-a720).
- * Arm Cortex-X4 (cortex-x4).
+ --target=arm
+ * Arm Cortex-M
https://github.com/DavidSpickett approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/74822
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -94,6 +94,11 @@ Changes to the AArch64 Backend
* Added support for Cortex-A520, Cortex-A720 and Cortex-X4 CPUs.
+* Neoverse-N2 was incorrectly marked as an Armv8.5a core. This has been
+ changed to an Armv9.0a core. However, crypto options are not enabled
+ by default fo
@@ -94,6 +94,11 @@ Changes to the AArch64 Backend
* Added support for Cortex-A520, Cortex-A720 and Cortex-X4 CPUs.
+* Neoverse-N2 was incorrectly marked as an Armv8.5a core. This has been
+ changed to an Armv9.0a core. However, crypto options are not enabled
+ by default fo
@@ -94,6 +94,11 @@ Changes to the AArch64 Backend
* Added support for Cortex-A520, Cortex-A720 and Cortex-X4 CPUs.
+* Neoverse-N2 was incorrectly marked as an Armv8.5a core. This has been
+ changed to an Armv9.0a core. However, crypto options are not enabled
+ by default fo
@@ -94,6 +94,11 @@ Changes to the AArch64 Backend
* Added support for Cortex-A520, Cortex-A720 and Cortex-X4 CPUs.
+* Neoverse-N2 was incorrectly marked as an Armv8.5a core. This has been
+ changed to an Armv9.0a core. However, crypto options are not enabled
+ by default fo
https://github.com/DavidSpickett edited
https://github.com/llvm/llvm-project/pull/75055
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -94,6 +94,11 @@ Changes to the AArch64 Backend
* Added support for Cortex-A520, Cortex-A720 and Cortex-X4 CPUs.
+* Neoverse-N2 was incorrectly marked as an Armv8.5a core. This has been
+ changed to an Armv9.0a core. However, crypto options are not enabled
+ by default fo
https://github.com/DavidSpickett approved this pull request.
LGTM
Once again clang-format rightly complaining about a lot of the surrounding
code, but it would obscure the change here so you can ignore it.
https://github.com/llvm/llvm-project/pull/75055
@@ -0,0 +1,35 @@
+// RUN: %clang --target=arm-none-eabi -mcpu=cortex-m85 -mfloat-abi=hard -O2
-save-temps=obj -S -o - %s | FileCheck %s
+// RUN: %clang --target=arm-none-eabi -mcpu=cortex-m85 -mfloat-abi=hard -O2 -c
-mthumb -save-temps=obj %s
+// RUN: %clang --target=arm-none-ea
@@ -0,0 +1,35 @@
+// RUN: %clang --target=arm-none-eabi -mcpu=cortex-m85 -mfloat-abi=hard -O2
-save-temps=obj -S -o - %s | FileCheck %s
DavidSpickett wrote:
Should there be an m55 test for this as well, as you've got 55 and 85 for
save-temps.
https://github.co
@@ -0,0 +1,35 @@
+// RUN: %clang --target=arm-none-eabi -mcpu=cortex-m85 -mfloat-abi=hard -O2
-save-temps=obj -S -o - %s | FileCheck %s
+// RUN: %clang --target=arm-none-eabi -mcpu=cortex-m85 -mfloat-abi=hard -O2 -c
-mthumb -save-temps=obj %s
+// RUN: %clang --target=arm-none-ea
DavidSpickett wrote:
Just small things on the test case from me. Nominate @davemgreen to give it a
final check and approval, since I have not been around this area in a while.
https://github.com/llvm/llvm-project/pull/71545
___
cfe-commits mailing lis
DavidSpickett wrote:
Tests now LGTM, thanks for the updates.
https://github.com/llvm/llvm-project/pull/71545
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/DavidSpickett approved this pull request.
LGTM, thank you for the fix!
You'll know within ~15 minutes of landing whether there is still an issue, but
I doubt there will be.
https://github.com/llvm/llvm-project/pull/73110
___
cfe-co
DavidSpickett wrote:
I think this caused a test suite program to fail to build:
```
cd
/home/tcwg-buildbot/worker/clang-aarch64-global-isel/test/sandbox/build/SingleSource/UnitTests
&&
/home/tcwg-buildbot/worker/clang-aarch64-global-isel/stage1.install/bin/llvm-size
--format=sysv
/home/tcwg-
DavidSpickett wrote:
Actually this may be https://github.com/llvm/llvm-test-suite/pull/59, which
landed in the test suite just now as well, but is not listed as a change there
(which is a known issue with buildbot).
https://github.com/llvm/llvm-project/pull/73258
__
DavidSpickett wrote:
Not so sure about that:
```
LINK: command "C:\BuildTools\VC\Tools\MSVC\14.29.30133\bin\Hostx64\x64\link.exe
/nologo
tools\clang\tools\clang-format\CMakeFiles\clang-format.dir\ClangFormat.cpp.obj
tools\clang\tools\clang-format\CMakeFiles\clang-format.dir\C_\ws\src\llvm\reso
DavidSpickett wrote:
Also it seems that `clangAST` links to `clangBasic` but it doesn't `DEPENDS` on
them? Not sure what that is trying to achieve.
https://github.com/llvm/llvm-project/pull/74812
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
DavidSpickett wrote:
I was mistaken about the `target_link_libraries`, that's what `clangBasic`
links to not `clangAST`.
It's possible that `clangBasic` now needs to depend on `clangAST`, assuming
cmake and the linker are ok with that.
https://github.com/llvm/llvm-project/pull/74812
_
DavidSpickett wrote:
Also seen on Linaro's Windows on Arm 2 stage bot:
https://lab.llvm.org/buildbot/#/builders/120/builds/5990
https://github.com/llvm/llvm-project/pull/73685
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.
Author: David Spickett
Date: 2023-11-27T13:31:15Z
New Revision: d7c03a196edb1e5f5a45121bb51899e8f5e72ca6
URL:
https://github.com/llvm/llvm-project/commit/d7c03a196edb1e5f5a45121bb51899e8f5e72ca6
DIFF:
https://github.com/llvm/llvm-project/commit/d7c03a196edb1e5f5a45121bb51899e8f5e72ca6.diff
LOG
https://github.com/DavidSpickett closed
https://github.com/llvm/llvm-project/pull/76453
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -50,6 +50,7 @@ class LLVM_LIBRARY_VISIBILITY AArch64TargetInfo : public
TargetInfo {
bool HasMatMul = false;
bool HasBFloat16 = false;
bool HasSVE2 = false;
+ bool HasSVE2p1 = false;
DavidSpickett wrote:
FYI
```
/home/david.spickett/llvm-project/cla
Author: David Spickett
Date: 2022-07-26T09:16:45Z
New Revision: f3fbbe1cf33bf4da8e2620c770997d9ff68a5384
URL:
https://github.com/llvm/llvm-project/commit/f3fbbe1cf33bf4da8e2620c770997d9ff68a5384
DIFF:
https://github.com/llvm/llvm-project/commit/f3fbbe1cf33bf4da8e2620c770997d9ff68a5384.diff
LOG
Author: David Spickett
Date: 2022-03-23T09:51:02Z
New Revision: eb5ecbbcbb6ce38e29237ab5d17156fcb2e96e74
URL:
https://github.com/llvm/llvm-project/commit/eb5ecbbcbb6ce38e29237ab5d17156fcb2e96e74
DIFF:
https://github.com/llvm/llvm-project/commit/eb5ecbbcbb6ce38e29237ab5d17156fcb2e96e74.diff
LOG
Author: David Spickett
Date: 2022-03-23T10:43:20Z
New Revision: edb7ba714acba1d18a20d9f4986d2e38aee1d109
URL:
https://github.com/llvm/llvm-project/commit/edb7ba714acba1d18a20d9f4986d2e38aee1d109
DIFF:
https://github.com/llvm/llvm-project/commit/edb7ba714acba1d18a20d9f4986d2e38aee1d109.diff
LOG
Author: David Spickett
Date: 2022-03-23T11:43:43Z
New Revision: c3b98194df5572bc9b33024b48457538a7213b4c
URL:
https://github.com/llvm/llvm-project/commit/c3b98194df5572bc9b33024b48457538a7213b4c
DIFF:
https://github.com/llvm/llvm-project/commit/c3b98194df5572bc9b33024b48457538a7213b4c.diff
LOG
Author: David Spickett
Date: 2022-04-11T13:42:24Z
New Revision: 55b6a3186cfa8b85a1defe05861d73f447e4c990
URL:
https://github.com/llvm/llvm-project/commit/55b6a3186cfa8b85a1defe05861d73f447e4c990
DIFF:
https://github.com/llvm/llvm-project/commit/55b6a3186cfa8b85a1defe05861d73f447e4c990.diff
LOG
Author: David Spickett
Date: 2022-04-19T13:49:55Z
New Revision: 218b5c83940d469424564ba6f1ec488c4970a5e5
URL:
https://github.com/llvm/llvm-project/commit/218b5c83940d469424564ba6f1ec488c4970a5e5
DIFF:
https://github.com/llvm/llvm-project/commit/218b5c83940d469424564ba6f1ec488c4970a5e5.diff
LOG
Author: David Spickett
Date: 2022-02-03T10:08:20Z
New Revision: d15e7dd1238df20e9c09cc91930f716e0d3d5b05
URL:
https://github.com/llvm/llvm-project/commit/d15e7dd1238df20e9c09cc91930f716e0d3d5b05
DIFF:
https://github.com/llvm/llvm-project/commit/d15e7dd1238df20e9c09cc91930f716e0d3d5b05.diff
LOG
@@ -191,6 +191,60 @@ static MarshallingInfo createMarshallingInfo(const Record
&R) {
return Ret;
}
+static void EmitHelpTextsForVariants(
+raw_ostream &OS, std::vector,
StringRef>>
+ HelpTextsForVariants) {
+ // OptTable must be constexpr so it
https://github.com/DavidSpickett created
https://github.com/llvm/llvm-project/pull/81490
The motivation here was a suggestion over in Compiler Explorer. You can use
`-mllvm` already to do this but since gfortran supports `-masm`, I figured I'd
try to add it.
This is done by flang expanding `-
https://github.com/DavidSpickett edited
https://github.com/llvm/llvm-project/pull/81490
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/DavidSpickett edited
https://github.com/llvm/llvm-project/pull/81490
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/DavidSpickett updated
https://github.com/llvm/llvm-project/pull/81490
>From 6cc1f2073033c13fa45f888553ea3b3c384dd508 Mon Sep 17 00:00:00 2001
From: David Spickett
Date: Mon, 12 Feb 2024 14:56:33 +
Subject: [PATCH 1/2] [flang][Driver] Add -masm option to flang
The motivat
@@ -0,0 +1,13 @@
+// RUN: %flang -target x86_64-unknown-linux -masm=intel -S %s -### 2>&1 |
FileCheck --check-prefix=CHECK-INTEL %s
DavidSpickett wrote:
1. Done, I forgot this wasn't actually parsing the file. So I've removed the
function as well, it's not need
https://github.com/DavidSpickett closed
https://github.com/llvm/llvm-project/pull/81490
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: David Spickett
Date: 2024-02-13T16:05:03Z
New Revision: 381a00de4fdcccd904dac6a0856fb44f12ba0abb
URL:
https://github.com/llvm/llvm-project/commit/381a00de4fdcccd904dac6a0856fb44f12ba0abb
DIFF:
https://github.com/llvm/llvm-project/commit/381a00de4fdcccd904dac6a0856fb44f12ba0abb.diff
LOG
Author: David Spickett
Date: 2024-02-13T16:14:03Z
New Revision: 1d8479225a8c1efc8c90511e6c7fe608ff38163c
URL:
https://github.com/llvm/llvm-project/commit/1d8479225a8c1efc8c90511e6c7fe608ff38163c
DIFF:
https://github.com/llvm/llvm-project/commit/1d8479225a8c1efc8c90511e6c7fe608ff38163c.diff
LOG
Author: David Spickett
Date: 2024-02-13T16:39:59Z
New Revision: 7a5c1a4abc750fef335c2ee5191d59ebe9e4bf18
URL:
https://github.com/llvm/llvm-project/commit/7a5c1a4abc750fef335c2ee5191d59ebe9e4bf18
DIFF:
https://github.com/llvm/llvm-project/commit/7a5c1a4abc750fef335c2ee5191d59ebe9e4bf18.diff
LOG
DavidSpickett wrote:
Should this be added to the release notes? (so it doesn't get forgotten in a
mad scramble in a few months time)
https://github.com/llvm/llvm-project/pull/84485
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.
DavidSpickett wrote:
Also does this address the -mcpu=native part of
https://github.com/llvm/llvm-project/issues/84450 as well?
https://github.com/llvm/llvm-project/pull/84485
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.
DavidSpickett wrote:
> I've added a host id (0xd42) to llvm/lib/TargetParser/Host.cpp, so I think so
> (unless there's something else required).
Right, of course, somehow I skipped right over that.
https://github.com/llvm/llvm-project/pull/84485
___
https://github.com/DavidSpickett approved this pull request.
Looks good to me.
https://github.com/llvm/llvm-project/pull/84485
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -3382,10 +3382,19 @@ def fopenmp : Flag<["-"], "fopenmp">, Group,
HelpText<"Parse OpenMP pragmas and generate parallel code.">;
def fno_openmp : Flag<["-"], "fno-openmp">, Group,
Flags<[NoArgumentUnused]>;
+class OpenMPVersionHelp {
+ string str = !strconcat(
+"Set
https://github.com/DavidSpickett updated
https://github.com/llvm/llvm-project/pull/81869
>From 72bbd9d38cb6e292d92391fcf04154cfbc336192 Mon Sep 17 00:00:00 2001
From: David Spickett
Date: Thu, 15 Feb 2024 09:52:22 +
Subject: [PATCH 1/5] [flang][clang] Add Visibility specific help text for
https://github.com/DavidSpickett edited
https://github.com/llvm/llvm-project/pull/81869
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -3382,10 +3382,19 @@ def fopenmp : Flag<["-"], "fopenmp">, Group,
HelpText<"Parse OpenMP pragmas and generate parallel code.">;
def fno_openmp : Flag<["-"], "fno-openmp">, Group,
Flags<[NoArgumentUnused]>;
+class OpenMPVersionHelp {
+ string str = !strconcat(
+"Set
https://github.com/DavidSpickett updated
https://github.com/llvm/llvm-project/pull/81869
>From 72bbd9d38cb6e292d92391fcf04154cfbc336192 Mon Sep 17 00:00:00 2001
From: David Spickett
Date: Thu, 15 Feb 2024 09:52:22 +
Subject: [PATCH 1/6] [flang][clang] Add Visibility specific help text for
https://github.com/DavidSpickett updated
https://github.com/llvm/llvm-project/pull/81869
>From 72bbd9d38cb6e292d92391fcf04154cfbc336192 Mon Sep 17 00:00:00 2001
From: David Spickett
Date: Thu, 15 Feb 2024 09:52:22 +
Subject: [PATCH 1/7] [flang][clang] Add Visibility specific help text for
https://github.com/DavidSpickett updated
https://github.com/llvm/llvm-project/pull/81869
>From 72bbd9d38cb6e292d92391fcf04154cfbc336192 Mon Sep 17 00:00:00 2001
From: David Spickett
Date: Thu, 15 Feb 2024 09:52:22 +
Subject: [PATCH 1/8] [flang][clang] Add Visibility specific help text for
https://github.com/DavidSpickett updated
https://github.com/llvm/llvm-project/pull/81869
>From 72bbd9d38cb6e292d92391fcf04154cfbc336192 Mon Sep 17 00:00:00 2001
From: David Spickett
Date: Thu, 15 Feb 2024 09:52:22 +
Subject: [PATCH 1/9] [flang][clang] Add Visibility specific help text for
https://github.com/DavidSpickett edited
https://github.com/llvm/llvm-project/pull/81869
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -93,6 +93,11 @@ class OptionGroup {
// Define the option class.
+class HelpTextForVisibility {
DavidSpickett wrote:
I've changed it to:
HelpTextForVariants (for a single entry in the list, which is many variants to
one string)
Help**Texts**ForVariants (f
https://github.com/DavidSpickett updated
https://github.com/llvm/llvm-project/pull/81869
>From 72bbd9d38cb6e292d92391fcf04154cfbc336192 Mon Sep 17 00:00:00 2001
From: David Spickett
Date: Thu, 15 Feb 2024 09:52:22 +
Subject: [PATCH 01/12] [flang][clang] Add Visibility specific help text for
DavidSpickett wrote:
I've also tested this with increased array sizes. You can check the .inc files
to see that they are intialised properly, `/lib/ToolDrivers/llvm-lib/Options.inc` is one.
https://github.com/llvm/llvm-project/pull/81869
___
cfe-commi
https://github.com/DavidSpickett updated
https://github.com/llvm/llvm-project/pull/81869
>From 72bbd9d38cb6e292d92391fcf04154cfbc336192 Mon Sep 17 00:00:00 2001
From: David Spickett
Date: Thu, 15 Feb 2024 09:52:22 +
Subject: [PATCH 1/4] [flang][clang] Add Visibility specific help text for
https://github.com/DavidSpickett edited
https://github.com/llvm/llvm-project/pull/81869
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
DavidSpickett wrote:
@banach-space Your thoughts as the Flang driver owner?
https://github.com/llvm/llvm-project/pull/81869
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/DavidSpickett updated
https://github.com/llvm/llvm-project/pull/78292
>From 20822b4a2f8e228365c8fa912af18afc9956749e Mon Sep 17 00:00:00 2001
From: David Spickett
Date: Tue, 16 Jan 2024 13:36:15 +
Subject: [PATCH 01/10] [GitHub][workflows] Add buildbot information comment
https://github.com/DavidSpickett closed
https://github.com/llvm/llvm-project/pull/78292
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
DavidSpickett wrote:
> I tried running clang-format but the diff was huge and beyond just my changes.
https://clang.llvm.org/docs/ClangFormat.html#git-integration tells you how to
narrow it down to just your changes. Once that's setup you can use the command
from the comment the bot left.
Som
Author: David Spickett
Date: 2024-01-31T10:29:48Z
New Revision: 0217d2e089afba8ca0713787521ba52a1056
URL:
https://github.com/llvm/llvm-project/commit/0217d2e089afba8ca0713787521ba52a1056
DIFF:
https://github.com/llvm/llvm-project/commit/0217d2e089afba8ca0713787521ba52a1056.diff
LOG
https://github.com/DavidSpickett edited
https://github.com/llvm/llvm-project/pull/80103
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
DavidSpickett wrote:
Just for citations here, the A profile manual says:
FEAT_PAuth is mandatory in Armv8.3 implementations.
The R Profile manual says:
The Armv8-R AArch64 architecture supports FEAT_PAuth2 feature defined in
Armv8-A architecture with a
modified definition of PAC field as descri
https://github.com/DavidSpickett edited
https://github.com/llvm/llvm-project/pull/78027
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/DavidSpickett approved this pull request.
LGTM.
I think the reason we didn't do this is because we have so far:
* Used nop space encodings
* Used a CPU that would enable it
* Been adding -mbranch-protection and friends anyway, so adding +pauth didn't
seem like a big deal.
Bu
@@ -1280,7 +1282,7 @@ INSTANTIATE_TEST_SUITE_P(
AArch64::AEK_DOTPROD, AArch64::AEK_FP, AArch64::AEK_SIMD,
AArch64::AEK_FP16, AArch64::AEK_FP16FML, AArch64::AEK_RAS,
AArch64::AEK_RCPC, AArch64::AEK_LSE, AArch64::AEK_SB,
-
DavidSpickett wrote:
And the code formatter is going crazy over some of this code :) One of these
days we'll fix it.
You can ignore it for now, save yourself and my downstream colleagues a massive
headache with the merge.
https://github.com/llvm/llvm-project/pull/78027
___
https://github.com/DavidSpickett edited
https://github.com/llvm/llvm-project/pull/80691
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
DavidSpickett wrote:
If you're referring to the CI that runs here, we've been having capacity issues
lately. Another way to "resubmit" is to rebase the PR.
https://github.com/llvm/llvm-project/pull/79655
___
cfe-commits mailing list
cfe-commits@lists.
https://github.com/DavidSpickett approved this pull request.
```
/var/lib/buildkite-agent/builds/linux-56-59b8f5d88-r6v66-1/llvm-project/github-pull-requests/llvm/unittests/TargetParser/TargetParserTest.cpp:1662:
Failure
Expected equality of these values:
List.size()
Which is: 68
NumAArc
Author: David Spickett
Date: 2024-01-12T16:08:52Z
New Revision: 6f55c134d4aaa9eab9ef53886c2532d6da72ca47
URL:
https://github.com/llvm/llvm-project/commit/6f55c134d4aaa9eab9ef53886c2532d6da72ca47
DIFF:
https://github.com/llvm/llvm-project/commit/6f55c134d4aaa9eab9ef53886c2532d6da72ca47.diff
LOG
https://github.com/DavidSpickett approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/78305
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/DavidSpickett edited
https://github.com/llvm/llvm-project/pull/78270
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/DavidSpickett commented:
Happy to see this change, will be a lot easier to reason about this once it's
all in one place.
https://github.com/llvm/llvm-project/pull/78270
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https:/
@@ -308,6 +312,94 @@ inline constexpr ExtensionInfo Extensions[] = {
};
// clang-format on
+struct ExtensionSet {
+ // Set of extensions which are currently enabled.
+ ExtensionBitset Enabled;
+ // Set of extensions which have been enabled or disabled at any point. Used
+
@@ -1,8 +1,9 @@
// RUN: %clang -### --target=aarch64-none-elf -march=armv8a+predres %s
2>&1 | FileCheck %s
+// RUN: %clang -### --target=aarch64-none-elf -mcpu=cortex-a520 %s
2>&1 | FileCheck %s
// CHECK: "-target-feature" "+predres"
// CHECK-NOT: "-target-feature
@@ -308,6 +312,94 @@ inline constexpr ExtensionInfo Extensions[] = {
};
// clang-format on
+struct ExtensionSet {
+ // Set of extensions which are currently enabled.
+ ExtensionBitset Enabled;
+ // Set of extensions which have been enabled or disabled at any point. Used
+
1 - 100 of 633 matches
Mail list logo
