https://github.com/domin144 closed
https://github.com/llvm/llvm-project/pull/81474
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
domin144 wrote:
@mplatings Can you merge for me? I don't have needed rights.
I rebased hoping the check will pass now, but there are still some unrelated
failures on windows.
https://github.com/llvm/llvm-project/pull/81474
___
cfe-commits mailing
https://github.com/domin144 updated
https://github.com/llvm/llvm-project/pull/81474
From 4eda12594dbc07e3d3509a309d6691dcc983e5cc Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Dominik=20W=C3=B3jt?=
Date: Mon, 12 Feb 2024 13:38:14 +0100
Subject: [PATCH 1/3] [clang][AArch64] multilib: fix deduction
@@ -191,7 +191,11 @@ static void getAArch64MultilibFlags(const Driver ,
for (const auto : AArch64::Extensions)
if (FeatureSet.contains(Ext.NegFeature))
MArch.push_back(("no" + Ext.Name).str());
- MArch.insert(MArch.begin(), ("-march=" +
https://github.com/domin144 updated
https://github.com/llvm/llvm-project/pull/81474
From da3ef3a5705ca4eec5683e0c048dd93da8c58b5d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Dominik=20W=C3=B3jt?=
Date: Mon, 12 Feb 2024 13:38:14 +0100
Subject: [PATCH 1/3] [clang][AArch64] multilib: fix deduction
domin144 wrote:
@MaskRay @petrhosek @smithp35 @mplatings
Looking at previous reviews for multilib related changes you might be
interested.
(ingore the buildkite failure for now. It seems unrelated.)
https://github.com/llvm/llvm-project/pull/81474
https://github.com/domin144 updated
https://github.com/llvm/llvm-project/pull/81474
From e25d580c7297071f067200639d8b8fe3be74eec6 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Dominik=20W=C3=B3jt?=
Date: Mon, 12 Feb 2024 13:38:14 +0100
Subject: [PATCH 1/2] [clang][AArch64] multilib: fix deduction
https://github.com/domin144 updated
https://github.com/llvm/llvm-project/pull/81474
From 9ffef2685c2766c6944e368a739b8c94e9b68092 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Dominik=20W=C3=B3jt?=
Date: Mon, 12 Feb 2024 13:38:14 +0100
Subject: [PATCH 1/2] [clang][AArch64] multilib: fix deduction
https://github.com/domin144 updated
https://github.com/llvm/llvm-project/pull/81474
From 8f8d2b304aa90c0aea00d67e86d285ecd870e443 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Dominik=20W=C3=B3jt?=
Date: Mon, 12 Feb 2024 13:38:14 +0100
Subject: [PATCH 1/2] [clang][AArch64] multilib: fix deduction
https://github.com/domin144 created
https://github.com/llvm/llvm-project/pull/81474
The deduced "-march=" option always started with aarch64, which is not a valid
value. There was also no way to distinguish between armv8-r and armv8-a. After
this commit, the deduced "-march=" option will
https://github.com/domin144 approved this pull request.
https://github.com/llvm/llvm-project/pull/75031
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/domin144 edited
https://github.com/llvm/llvm-project/pull/75031
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -264,6 +273,14 @@ ToolChain::getMultilibFlags(const llvm::opt::ArgList
) const {
break;
}
+ // Include fno-exceptions and fno-rtti
+ // to improve multilib selection
+ if (getRTTIMode() == ToolChain::RTTIMode::RM_Disabled)
+Result.push_back("-fno-rtti");
@@ -264,6 +273,14 @@ ToolChain::getMultilibFlags(const llvm::opt::ArgList
) const {
break;
}
+ // Include fno-exceptions and fno-rtti
+ // to improve multilib selection
+ if (getRTTIMode() == ToolChain::RTTIMode::RM_Disabled)
+Result.push_back("-fno-rtti");
+
+
https://github.com/domin144 updated
https://github.com/llvm/llvm-project/pull/67412
From e20234c6a73a2dded9e5a34e2900e7757ac9460f Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Dominik=20W=C3=B3jt?=
Date: Wed, 20 Sep 2023 16:39:27 +0200
Subject: [PATCH 1/2] [ARM] fix "+fp.dp" in multilib selection
https://github.com/domin144 updated
https://github.com/llvm/llvm-project/pull/67412
From 4cc2328982cc8260263aa803f2b02bff7d7b4a85 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Dominik=20W=C3=B3jt?=
Date: Wed, 20 Sep 2023 16:39:27 +0200
Subject: [PATCH 1/2] [ARM] fix "+fp.dp" in multilib selection
@@ -420,20 +444,35 @@ bool ARM::appendArchExtFeatures(StringRef CPU,
ARM::ArchKind AK,
CPU = "generic";
if (ArchExt == "fp" || ArchExt == "fp.dp") {
+const ARM::FPUKind DefaultFPU = getDefaultFPU(CPU, AK);
ARM::FPUKind FPUKind;
if (ArchExt == "fp.dp") {
+
@@ -420,20 +444,35 @@ bool ARM::appendArchExtFeatures(StringRef CPU,
ARM::ArchKind AK,
CPU = "generic";
if (ArchExt == "fp" || ArchExt == "fp.dp") {
+const ARM::FPUKind DefaultFPU = getDefaultFPU(CPU, AK);
ARM::FPUKind FPUKind;
if (ArchExt == "fp.dp") {
+
@@ -366,26 +366,50 @@ StringRef ARM::getArchExtFeature(StringRef ArchExt) {
}
static ARM::FPUKind findDoublePrecisionFPU(ARM::FPUKind InputFPUKind) {
+ if (InputFPUKind == ARM::FK_INVALID || InputFPUKind == ARM::FK_NONE)
+return ARM::FK_INVALID;
+
+ const ARM::FPUName
https://github.com/domin144 created
https://github.com/llvm/llvm-project/pull/67412
When the FPU was selected with "+(no)fp(.dp)" extensions in "-march" or "-mcpu"
options, the FPU used for multilib selection was still the default one for
given architecture or CPU.
From
20 matches
Mail list logo
