@@ -151,7 +151,7 @@ class LLVM_LIBRARY_VISIBILITY SparcV8TargetInfo : public
SparcTargetInfo {
public:
SparcV8TargetInfo(const llvm::Triple &Triple, const TargetOptions &Opts)
: SparcTargetInfo(Triple, Opts) {
-resetDataLayout("E-m:e-p:32:32-i64:64-f128:64-n32-S64"
@@ -151,7 +151,7 @@ class LLVM_LIBRARY_VISIBILITY SparcV8TargetInfo : public
SparcTargetInfo {
public:
SparcV8TargetInfo(const llvm::Triple &Triple, const TargetOptions &Opts)
: SparcTargetInfo(Triple, Opts) {
-resetDataLayout("E-m:e-p:32:32-i64:64-f128:64-n32-S64"
@@ -151,7 +151,7 @@ class LLVM_LIBRARY_VISIBILITY SparcV8TargetInfo : public
SparcTargetInfo {
public:
SparcV8TargetInfo(const llvm::Triple &Triple, const TargetOptions &Opts)
: SparcTargetInfo(Triple, Opts) {
-resetDataLayout("E-m:e-p:32:32-i64:64-f128:64-n32-S64"
koachan wrote:
My bad, yeah, I forgot to run bugpoint tests.
https://github.com/llvm/llvm-project/pull/110608 should fix it.
https://github.com/llvm/llvm-project/pull/106951
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org
rorth wrote:
FWIW, the same failures exist on Linux/sparc64, too.
https://github.com/llvm/llvm-project/pull/106951
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
rorth wrote:
In a local `sparcv9-sun-solaris2.11` build, I get (with `-silence-passes`
removed)
```
> /var/llvm/dist-sparcv9-release-stage2-A-flang-clang19/tools/clang/stage2-bins/bin/bugpoint
> -load
> /var/llvm/dist-sparcv9-release-stage2-A-flang-clang19/tools/clang/stage2-bins/lib/BugpointP
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `clang-solaris11-sparcv9`
running on `solaris11-sparcv9` while building `clang,llvm` at step 5 "ninja
check 1".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/13/builds/2592
Here is the relevant piec
https://github.com/koachan closed
https://github.com/llvm/llvm-project/pull/106951
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/s-barannikov approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/106951
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/koachan updated
https://github.com/llvm/llvm-project/pull/106951
>From 3eb96009b0b483553c2255893d26e59b8c4f6574 Mon Sep 17 00:00:00 2001
From: Koakuma
Date: Mon, 2 Sep 2024 12:01:04 +0700
Subject: [PATCH 1/5] [SPARC] Align i128 to 16 bytes in SPARC datalayouts
Align i128s to
brad0 wrote:
@s-barannikov
https://github.com/llvm/llvm-project/pull/106951
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -5461,6 +5461,18 @@ std::string llvm::UpgradeDataLayoutString(StringRef DL,
StringRef TT) {
return Res;
}
+ if (T.isSPARC()) {
+// Add "-i128:128"
+std::string I64 = "-i64:64";
+std::string I128 = "-i128:128";
+if (StringRef Ref = Res; !Ref.contains(
https://github.com/nikic approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/106951
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/nikic edited https://github.com/llvm/llvm-project/pull/106951
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/koachan updated
https://github.com/llvm/llvm-project/pull/106951
>From 3eb96009b0b483553c2255893d26e59b8c4f6574 Mon Sep 17 00:00:00 2001
From: Koakuma
Date: Mon, 2 Sep 2024 12:01:04 +0700
Subject: [PATCH 1/4] [SPARC] Align i128 to 16 bytes in SPARC datalayouts
Align i128s to
@@ -5461,6 +5461,18 @@ std::string llvm::UpgradeDataLayoutString(StringRef DL,
StringRef TT) {
return Res;
}
+ if (T.isSPARC()) {
+// Add "-i128:128"
+std::string I128 = "-i128:128";
+if (StringRef Ref = Res; !Ref.contains(I128)) {
+ SmallVector Groups
https://github.com/koachan updated
https://github.com/llvm/llvm-project/pull/106951
>From 3eb96009b0b483553c2255893d26e59b8c4f6574 Mon Sep 17 00:00:00 2001
From: Koakuma
Date: Mon, 2 Sep 2024 12:01:04 +0700
Subject: [PATCH 1/3] [SPARC] Align i128 to 16 bytes in SPARC datalayouts
Align i128s to
@@ -5461,6 +5461,13 @@ std::string llvm::UpgradeDataLayoutString(StringRef DL,
StringRef TT) {
return Res;
}
+ if (T.isSPARC()) {
+// Add "-i128:128"
+if (!DL.empty() && !DL.contains("-i128:128"))
+ Res.append("-i128:128");
+return Res;
+ }
--
koachan wrote:
Ping?
https://github.com/llvm/llvm-project/pull/106951
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
nikic wrote:
> I guess this is a breaking change?
Yes, but as previously discussed, it's acceptable breakage.
> Following the discussion in the linked revision it looks like this also needs
> some AutoUpgrade changes.
Yes, this should be handled in UpgradeDataLayoutString().
https://github.
s-barannikov wrote:
I guess this is a breaking change?
Following the discussion in the linked revision it looks like this also needs
some AutoUpgrade changes.
https://github.com/llvm/llvm-project/pull/106951
___
cfe-commits mailing list
cfe-commits@l
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Koakuma (koachan)
Changes
Align i128s to 16 bytes, following the example at
https://reviews.llvm.org/D86310.
clang already does this implicitly, but do it in backend code too for the
benefit of other frontends (see e.g
https://github.co
koachan wrote:
CC @beetrees
https://github.com/llvm/llvm-project/pull/106951
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/koachan created
https://github.com/llvm/llvm-project/pull/106951
Align i128s to 16 bytes, following the example at
https://reviews.llvm.org/D86310.
clang already does this implicitly, but do it in backend code too for the
benefit of other frontends (see e.g
https://github.
24 matches
Mail list logo
