Il-Capitano wrote:
> There is a `Arm and AArch64 Support` section too, is it worth adding a note
> there either instead or as well? It could maybe just be a note that points
> back to the Modified Compiler Flags section.
Thanks, I didn't notice that section.
I simplified the note in "Modified
https://github.com/Il-Capitano updated
https://github.com/llvm/llvm-project/pull/134802
From c48168df95781b8fe5d6f72c9bdcbe1ad6a94fe5 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Csan=C3=A1d=20Hajd=C3=BA?=
Date: Tue, 8 Apr 2025 09:47:46 +0200
Subject: [PATCH 1/2] [Clang][Docs] Add release note for
https://github.com/Il-Capitano created
https://github.com/llvm/llvm-project/pull/134802
None
From c48168df95781b8fe5d6f72c9bdcbe1ad6a94fe5 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Csan=C3=A1d=20Hajd=C3=BA?=
Date: Tue, 8 Apr 2025 09:47:46 +0200
Subject: [PATCH] [Clang][Docs] Add release note fo
Il-Capitano wrote:
Ping.
https://github.com/llvm/llvm-project/pull/125688
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Il-Capitano wrote:
@jansvoboda11 You're tagged as the compiler options maintainer for Clang, so
this may be of interest to you as well.
https://github.com/llvm/llvm-project/pull/125688
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://li
Il-Capitano wrote:
@smithp35 @davemgreen @paulwalker-arm @jh7370
Pinging to ask for more reviews on this, please.
https://github.com/llvm/llvm-project/pull/125688
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/m
https://github.com/Il-Capitano updated
https://github.com/llvm/llvm-project/pull/125688
From a25845dcb484514c83022a21ee924e31a39882f4 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Csan=C3=A1d=20Hajd=C3=BA?=
Date: Tue, 4 Feb 2025 14:51:55 +0100
Subject: [PATCH 1/2] [Clang][AArch64] Add support for SH
@@ -0,0 +1,6 @@
+// RUN: %clang --sysroot=%S/Inputs -c -fdriver-only -Werror
--target=aarch64-unknown-linux-gnu \
Il-Capitano wrote:
There must have been a good reason I removed the `!ForMultilib` check, but I
can't remember now...
You're right, the check is n
Il-Capitano wrote:
I've rebased this patch onto main, since
https://github.com/llvm/llvm-project/pull/125687 has been merged, so it's now
ready for review.
https://github.com/llvm/llvm-project/pull/125688
___
cfe-commits mailing list
cfe-commits@list
https://github.com/Il-Capitano ready_for_review
https://github.com/llvm/llvm-project/pull/125688
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Il-Capitano edited
https://github.com/llvm/llvm-project/pull/125688
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Il-Capitano updated
https://github.com/llvm/llvm-project/pull/125688
From a25845dcb484514c83022a21ee924e31a39882f4 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Csan=C3=A1d=20Hajd=C3=BA?=
Date: Tue, 4 Feb 2025 14:51:55 +0100
Subject: [PATCH] [Clang][AArch64] Add support for SHF_AA
https://github.com/Il-Capitano converted_to_draft
https://github.com/llvm/llvm-project/pull/125688
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Il-Capitano created
https://github.com/llvm/llvm-project/pull/125688
Add support for the new SHF_AARCH64_PURECODE ELF section flag:
https://github.com/ARM-software/abi-aa/pull/304
The general implementation follows the existing one for ARM targets. Simlarly
to ARM targets,
Il-Capitano wrote:
Just to clarify: `@llvm.vector.reduce.fadd` does sequential reduction by
default, so I don't see a point in doing that manually in the frontend. Your
previous implementation had the same behaviour.
The inconsistency with the sequential approach is that Clang defines the
`__
Il-Capitano wrote:
`__builtin_reduce_add/mul` is defined to do _recursive even-odd pairwise
reduction_ in Clang, and the `@llvm.vector.reduce.fadd` intrinsic doesn't do
that.
I'm not sure if there's any benefit to this definition over a sequential
reduction. The only thing I can think that co
https://github.com/Il-Capitano edited
https://github.com/llvm/llvm-project/pull/120367
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -4274,12 +4274,37 @@ RValue CodeGenFunction::EmitBuiltinExpr(const
GlobalDecl GD, unsigned BuiltinID,
*this, E, GetIntrinsicID(E->getArg(0)->getType()), "rdx.min"));
}
- case Builtin::BI__builtin_reduce_add:
+ case Builtin::BI__builtin_reduce_add: {
+// Not
@@ -4274,12 +4274,37 @@ RValue CodeGenFunction::EmitBuiltinExpr(const
GlobalDecl GD, unsigned BuiltinID,
*this, E, GetIntrinsicID(E->getArg(0)->getType()), "rdx.min"));
}
- case Builtin::BI__builtin_reduce_add:
+ case Builtin::BI__builtin_reduce_add: {
+// Not
Il-Capitano wrote:
Thanks for reviewing!
I updated the code to use `err_drv_unsupported_opt_for_target`, and added test
cases for the `-mpure-code` alias of `-mexecute-only` also.
https://github.com/llvm/llvm-project/pull/118816
___
cfe-commits maili
https://github.com/Il-Capitano updated
https://github.com/llvm/llvm-project/pull/118816
From 94c933e0e7b7497e76c541f3ea7ed210349d2044 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Csan=C3=A1d=20Hajd=C3=BA?=
Date: Thu, 5 Dec 2024 14:46:08 +0100
Subject: [PATCH 1/2] [Clang] Improve error for `-fsaniti
Il-Capitano wrote:
@MaskRay Can I ask you to review/add appropriate reviewers for this patch?
https://github.com/llvm/llvm-project/pull/118816
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-c
@@ -418,8 +418,14 @@ SanitizerArgs::SanitizerArgs(const ToolChain &TC,
Add & NotAllowedWithExecuteOnly & ~DiagnosedKinds) {
if (DiagnoseErrors) {
std::string Desc = describeSanitizeArg(Arg, KindsToDiagnose);
-D.Diag(diag::err_dr
https://github.com/Il-Capitano edited
https://github.com/llvm/llvm-project/pull/118816
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Il-Capitano updated
https://github.com/llvm/llvm-project/pull/118816
From 94c933e0e7b7497e76c541f3ea7ed210349d2044 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Csan=C3=A1d=20Hajd=C3=BA?=
Date: Thu, 5 Dec 2024 14:46:08 +0100
Subject: [PATCH] [Clang] Improve error for `-fsanitize=f
https://github.com/Il-Capitano created
https://github.com/llvm/llvm-project/pull/118816
The current error message when using the `-fsanitize=function -mexecute-only`
flags together points to the target triple as the reason that
`-fsanitize=function` is not allowed to be used, even when the fun
26 matches
Mail list logo
