[llvm] Revert "[PowerPC] Expand global named register support" (PR #113457)
via llvm-commits
llvm-commits at lists.llvm.org
Wed Oct 23 06:35:50 PDT 2024
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-backend-powerpc
Author: Lei Huang (lei137)
<details>
<summary>Changes</summary>
Reverts llvm/llvm-project#<!-- -->112603
---
Full diff: https://github.com/llvm/llvm-project/pull/113457.diff
9 Files Affected:
- (modified) llvm/lib/Target/PowerPC/PPCISelLowering.cpp (+15-26)
- (modified) llvm/test/CodeGen/PowerPC/named-reg-alloc-r0.ll (+5-3)
- (modified) llvm/test/CodeGen/PowerPC/named-reg-alloc-r1-64.ll (+4-2)
- (modified) llvm/test/CodeGen/PowerPC/named-reg-alloc-r1.ll (+6-3)
- (modified) llvm/test/CodeGen/PowerPC/named-reg-alloc-r13-64.ll (+5-2)
- (modified) llvm/test/CodeGen/PowerPC/named-reg-alloc-r13.ll (+5-3)
- (modified) llvm/test/CodeGen/PowerPC/named-reg-alloc-r2-64.ll (+3-1)
- (modified) llvm/test/CodeGen/PowerPC/named-reg-alloc-r2.ll (+6-3)
- (removed) llvm/test/CodeGen/PowerPC/named-reg-alloc.ll (-144)
``````````diff
diff --git a/llvm/lib/Target/PowerPC/PPCISelLowering.cpp b/llvm/lib/Target/PowerPC/PPCISelLowering.cpp
index a4f0444ff2a645..7199fac9b110b6 100644
--- a/llvm/lib/Target/PowerPC/PPCISelLowering.cpp
+++ b/llvm/lib/Target/PowerPC/PPCISelLowering.cpp
@@ -17367,36 +17367,25 @@ SDValue PPCTargetLowering::LowerFRAMEADDR(SDValue Op,
return FrameAddr;
}
-#define GET_REGISTER_MATCHER
-#include "PPCGenAsmMatcher.inc"
-
-Register PPCTargetLowering::getRegisterByName(const char *RegName, LLT VT,
+// FIXME? Maybe this could be a TableGen attribute on some registers and
+// this table could be generated automatically from RegInfo.
+Register PPCTargetLowering::getRegisterByName(const char* RegName, LLT VT,
const MachineFunction &MF) const {
- bool IsPPC64 = Subtarget.isPPC64();
+ bool isPPC64 = Subtarget.isPPC64();
- bool Is64Bit = IsPPC64 && VT == LLT::scalar(64);
- if (!Is64Bit && VT != LLT::scalar(32))
+ bool is64Bit = isPPC64 && VT == LLT::scalar(64);
+ if (!is64Bit && VT != LLT::scalar(32))
report_fatal_error("Invalid register global variable type");
- Register Reg = MatchRegisterName(RegName);
- if (!Reg)
- report_fatal_error(
- Twine("Invalid global name register \"" + StringRef(RegName) + "\"."));
-
- // FIXME: These registers are not flagged as reserved and we can generate
- // code for `-O0` but not for `-O2`. Need followup investigation as to why.
- if ((IsPPC64 && Reg == PPC::R2) || Reg == PPC::R0)
- report_fatal_error(Twine("Trying to reserve an invalid register \"" +
- StringRef(RegName) + "\"."));
-
- // Convert GPR to GP8R register for 64bit.
- if (Is64Bit && StringRef(RegName).starts_with_insensitive("r"))
- Reg = Reg.id() - PPC::R0 + PPC::X0;
-
- if (Subtarget.getRegisterInfo()->getReservedRegs(MF).test(Reg))
- report_fatal_error(Twine("Trying to obtain a reserved register \"" +
- StringRef(RegName) + "\"."));
- return Reg;
+ Register Reg = StringSwitch<Register>(RegName)
+ .Case("r1", is64Bit ? PPC::X1 : PPC::R1)
+ .Case("r2", isPPC64 ? Register() : PPC::R2)
+ .Case("r13", (is64Bit ? PPC::X13 : PPC::R13))
+ .Default(Register());
+
+ if (Reg)
+ return Reg;
+ report_fatal_error("Invalid register name global variable");
}
bool PPCTargetLowering::isAccessedAsGotIndirect(SDValue GA) const {
diff --git a/llvm/test/CodeGen/PowerPC/named-reg-alloc-r0.ll b/llvm/test/CodeGen/PowerPC/named-reg-alloc-r0.ll
index c6b9498a680f4d..11cb72296e2c43 100644
--- a/llvm/test/CodeGen/PowerPC/named-reg-alloc-r0.ll
+++ b/llvm/test/CodeGen/PowerPC/named-reg-alloc-r0.ll
@@ -1,9 +1,11 @@
-; RUN: not --crash llc -O0 < %s -mtriple=powerpc-unknown-linux-gnu 2>&1 | FileCheck %s
-; RUN: not --crash llc -O0 < %s -mtriple=powerpc64-unknown-linux-gnu 2>&1 | FileCheck %s
+; RUN: not --crash llc < %s -mtriple=powerpc-unknown-linux-gnu 2>&1 | FileCheck %s
+; RUN: not --crash llc < %s -mtriple=powerpc-unknown-linux-gnu 2>&1 | FileCheck %s
+; RUN: not --crash llc < %s -mtriple=powerpc64-unknown-linux-gnu 2>&1 | FileCheck %s
define i32 @get_reg() nounwind {
entry:
-; CHECK: Trying to reserve an invalid register "r0".
+; FIXME: Include an allocatable-specific error message
+; CHECK: Invalid register name global variable
%reg = call i32 @llvm.read_register.i32(metadata !0)
ret i32 %reg
}
diff --git a/llvm/test/CodeGen/PowerPC/named-reg-alloc-r1-64.ll b/llvm/test/CodeGen/PowerPC/named-reg-alloc-r1-64.ll
index 96dbddd5bf1578..080b1982c88cbf 100644
--- a/llvm/test/CodeGen/PowerPC/named-reg-alloc-r1-64.ll
+++ b/llvm/test/CodeGen/PowerPC/named-reg-alloc-r1-64.ll
@@ -1,10 +1,12 @@
-; RUN: not --crash llc < %s -mtriple=powerpc64-unknown-linux-gnu 2>&1 | FileCheck %s
+; RUN: llc -verify-machineinstrs < %s -mtriple=powerpc64-unknown-linux-gnu 2>&1 | FileCheck %s
define i64 @get_reg() nounwind {
-; CHECK: Trying to obtain a reserved register "r1".
entry:
%reg = call i64 @llvm.read_register.i64(metadata !0)
ret i64 %reg
+
+; CHECK-LABEL: get_reg
+; CHECK: mr 3, 1
}
declare i64 @llvm.read_register.i64(metadata) nounwind
diff --git a/llvm/test/CodeGen/PowerPC/named-reg-alloc-r1.ll b/llvm/test/CodeGen/PowerPC/named-reg-alloc-r1.ll
index 5980d09bc9c906..6edd787e445e46 100644
--- a/llvm/test/CodeGen/PowerPC/named-reg-alloc-r1.ll
+++ b/llvm/test/CodeGen/PowerPC/named-reg-alloc-r1.ll
@@ -1,11 +1,14 @@
-; RUN: not --crash llc < %s -mtriple=powerpc-unknown-linux-gnu 2>&1 | FileCheck %s
-; RUN: not --crash llc < %s -mtriple=powerpc64-unknown-linux-gnu 2>&1 | FileCheck %s
+; RUN: llc -verify-machineinstrs < %s -mtriple=powerpc-unknown-linux-gnu 2>&1 | FileCheck %s
+; RUN: llc -verify-machineinstrs < %s -mtriple=powerpc64-unknown-linux-gnu 2>&1 | FileCheck %s
define i32 @get_reg() nounwind {
-; CHECK: Trying to obtain a reserved register "r1".
entry:
%reg = call i32 @llvm.read_register.i32(metadata !0)
ret i32 %reg
+
+; CHECK-LABEL: @get_reg
+; CHECK: mr 3, 1
+
}
declare i32 @llvm.read_register.i32(metadata) nounwind
diff --git a/llvm/test/CodeGen/PowerPC/named-reg-alloc-r13-64.ll b/llvm/test/CodeGen/PowerPC/named-reg-alloc-r13-64.ll
index 8bd4d899f72609..4d73fb3e81869c 100644
--- a/llvm/test/CodeGen/PowerPC/named-reg-alloc-r13-64.ll
+++ b/llvm/test/CodeGen/PowerPC/named-reg-alloc-r13-64.ll
@@ -1,10 +1,13 @@
-; RUN: not --crash llc < %s -mtriple=powerpc64-unknown-linux-gnu 2>&1 | FileCheck %s
+; RUN: llc -verify-machineinstrs < %s -mtriple=powerpc64-unknown-linux-gnu 2>&1 | FileCheck %s
define i64 @get_reg() nounwind {
-; CHECK: Trying to obtain a reserved register "r13".
entry:
%reg = call i64 @llvm.read_register.i64(metadata !0)
ret i64 %reg
+
+; CHECK-LABEL: @get_reg
+; CHECK: mr 3, 13
+
}
declare i64 @llvm.read_register.i64(metadata) nounwind
diff --git a/llvm/test/CodeGen/PowerPC/named-reg-alloc-r13.ll b/llvm/test/CodeGen/PowerPC/named-reg-alloc-r13.ll
index 2185a8a6cd5379..a7b778147980ac 100644
--- a/llvm/test/CodeGen/PowerPC/named-reg-alloc-r13.ll
+++ b/llvm/test/CodeGen/PowerPC/named-reg-alloc-r13.ll
@@ -1,11 +1,13 @@
-; RUN: not --crash llc < %s -mtriple=powerpc-unknown-linux-gnu 2>&1 | FileCheck %s
-; RUN: not --crash llc < %s -mtriple=powerpc64-unknown-linux-gnu 2>&1 | FileCheck %s
+; RUN: llc -verify-machineinstrs < %s -mtriple=powerpc-unknown-linux-gnu 2>&1 | FileCheck %s
+; RUN: llc -verify-machineinstrs < %s -mtriple=powerpc64-unknown-linux-gnu 2>&1 | FileCheck %s
define i32 @get_reg() nounwind {
-; CHECK: Trying to obtain a reserved register "r13".
entry:
%reg = call i32 @llvm.read_register.i32(metadata !0)
ret i32 %reg
+
+; CHECK-LABEL: @get_reg
+; CHECK: mr 3, 13
}
declare i32 @llvm.read_register.i32(metadata) nounwind
diff --git a/llvm/test/CodeGen/PowerPC/named-reg-alloc-r2-64.ll b/llvm/test/CodeGen/PowerPC/named-reg-alloc-r2-64.ll
index fdf7ea2711d827..3df778f445c733 100644
--- a/llvm/test/CodeGen/PowerPC/named-reg-alloc-r2-64.ll
+++ b/llvm/test/CodeGen/PowerPC/named-reg-alloc-r2-64.ll
@@ -1,8 +1,10 @@
; RUN: not --crash llc < %s -mtriple=powerpc64-unknown-linux-gnu 2>&1 | FileCheck %s
+; RUN: not --crash llc < %s -mtriple=powerpc64-unknown-linux-gnu 2>&1 | FileCheck %s
define i64 @get_reg() nounwind {
entry:
-; CHECK: Trying to reserve an invalid register "r2".
+; FIXME: Include an allocatable-specific error message
+; CHECK: Invalid register name global variable
%reg = call i64 @llvm.read_register.i64(metadata !0)
ret i64 %reg
}
diff --git a/llvm/test/CodeGen/PowerPC/named-reg-alloc-r2.ll b/llvm/test/CodeGen/PowerPC/named-reg-alloc-r2.ll
index 58c782901f3eb6..ca79f857548ebe 100644
--- a/llvm/test/CodeGen/PowerPC/named-reg-alloc-r2.ll
+++ b/llvm/test/CodeGen/PowerPC/named-reg-alloc-r2.ll
@@ -1,12 +1,15 @@
-; RUN: not --crash llc < %s -mtriple=powerpc-unknown-linux-gnu 2>&1 | FileCheck %s
+; RUN: llc -verify-machineinstrs < %s -mtriple=powerpc-unknown-linux-gnu 2>&1 | FileCheck %s
; RUN: not --crash llc < %s -mtriple=powerpc64-unknown-linux-gnu 2>&1 | FileCheck %s --check-prefix=CHECK-NOTPPC32
define i32 @get_reg() nounwind {
entry:
-; CHECK-NOTPPC32: Trying to reserve an invalid register "r2".
-; CHECK: Trying to obtain a reserved register "r2".
+; FIXME: Include an allocatable-specific error message
+; CHECK-NOTPPC32: Invalid register name global variable
%reg = call i32 @llvm.read_register.i32(metadata !0)
ret i32 %reg
+
+; CHECK-LABEL: @get_reg
+; CHECK: mr 3, 2
}
declare i32 @llvm.read_register.i32(metadata) nounwind
diff --git a/llvm/test/CodeGen/PowerPC/named-reg-alloc.ll b/llvm/test/CodeGen/PowerPC/named-reg-alloc.ll
deleted file mode 100644
index 38d22475ead910..00000000000000
--- a/llvm/test/CodeGen/PowerPC/named-reg-alloc.ll
+++ /dev/null
@@ -1,144 +0,0 @@
-; NOTE: Assertions have been autogenerated by utils/update_llc_test_checks.py UTC_ARGS: --version 5
-; RUN: llc -O0 -verify-machineinstrs < %s -mtriple=powerpc-unknown-linux-gnu 2>&1 | FileCheck %s
-; RUN: llc -O0 -verify-machineinstrs < %s -mtriple=powerpc64-unknown-linux-gnu 2>&1 | FileCheck %s --check-prefix=CHECK64
-
- at mVal = dso_local global i32 15, align 4
- at myGVal = dso_local global i32 0, align 4
-
-define dso_local void @testSetIntReg(i32 noundef signext %xx) {
-; CHECK-LABEL: testSetIntReg:
-; CHECK: # %bb.0: # %entry
-; CHECK-NEXT: mr 5, 3
-; CHECK-NEXT: blr
-;
-; CHECK64-LABEL: testSetIntReg:
-; CHECK64: # %bb.0: # %entry
-; CHECK64-NEXT: mr 5, 3
-; CHECK64-NEXT: blr
-entry:
- tail call void @llvm.write_register.i32(metadata !0, i32 %xx)
- ret void
-}
-
-declare void @llvm.write_register.i32(metadata, i32)
-
-define dso_local signext range(i32 0, 2) i32 @testCmpReg() {
-; CHECK-LABEL: testCmpReg:
-; CHECK: # %bb.0: # %entry
-; CHECK-NEXT: lis 3, mVal at ha
-; CHECK-NEXT: lwz 3, mVal at l(3)
-; CHECK-NEXT: xori 3, 3, 15
-; CHECK-NEXT: cntlzw 3, 3
-; CHECK-NEXT: srwi 3, 3, 5
-; CHECK-NEXT: blr
-;
-; CHECK64-LABEL: testCmpReg:
-; CHECK64: # %bb.0: # %entry
-; CHECK64-NEXT: addis 3, 2, mVal at toc@ha
-; CHECK64-NEXT: addi 3, 3, mVal at toc@l
-; CHECK64-NEXT: lwz 3, 0(3)
-; CHECK64-NEXT: xori 3, 3, 15
-; CHECK64-NEXT: cntlzw 3, 3
-; CHECK64-NEXT: srwi 3, 3, 5
-; CHECK64-NEXT: extsw 3, 3
-; CHECK64-NEXT: blr
-entry:
- tail call void @llvm.write_register.i32(metadata !0, i32 15)
- %0 = load i32, ptr @mVal, align 4
- %1 = tail call i32 @llvm.read_register.i32(metadata !0)
- %cmp = icmp eq i32 %0, %1
- %conv = zext i1 %cmp to i32
- ret i32 %conv
-}
-
-declare i32 @llvm.read_register.i32(metadata)
-
-define dso_local void @testSetIntReg2(i32 noundef signext %xx) {
-; CHECK-LABEL: testSetIntReg2:
-; CHECK: # %bb.0: # %entry
-; CHECK-NEXT: stwu 1, -48(1)
-; CHECK-NEXT: .cfi_def_cfa_offset 48
-; CHECK-NEXT: .cfi_offset r23, -36
-; CHECK-NEXT: stw 23, 12(1) # 4-byte Folded Spill
-; CHECK-NEXT: mr 23, 3
-; CHECK-NEXT: lwz 23, 12(1) # 4-byte Folded Reload
-; CHECK-NEXT: addi 1, 1, 48
-; CHECK-NEXT: blr
-;
-; CHECK64-LABEL: testSetIntReg2:
-; CHECK64: # %bb.0: # %entry
-; CHECK64-NEXT: std 23, -72(1) # 8-byte Folded Spill
-; CHECK64-NEXT: mr 23, 3
-; CHECK64-NEXT: ld 23, -72(1) # 8-byte Folded Reload
-; CHECK64-NEXT: blr
-entry:
- tail call void @llvm.write_register.i32(metadata !1, i32 %xx)
- ret void
-}
-
-define dso_local signext i32 @testReturnReg() {
-; CHECK-LABEL: testReturnReg:
-; CHECK: # %bb.0: # %entry
-; CHECK-NEXT: stwu 1, -48(1)
-; CHECK-NEXT: .cfi_def_cfa_offset 48
-; CHECK-NEXT: .cfi_offset r23, -36
-; CHECK-NEXT: stw 23, 12(1) # 4-byte Folded Spill
-; CHECK-NEXT: li 23, 125
-; CHECK-NEXT: mr 3, 23
-; CHECK-NEXT: lwz 23, 12(1) # 4-byte Folded Reload
-; CHECK-NEXT: addi 1, 1, 48
-; CHECK-NEXT: blr
-;
-; CHECK64-LABEL: testReturnReg:
-; CHECK64: # %bb.0: # %entry
-; CHECK64-NEXT: std 23, -72(1) # 8-byte Folded Spill
-; CHECK64-NEXT: li 23, 125
-; CHECK64-NEXT: extsw 3, 23
-; CHECK64-NEXT: ld 23, -72(1) # 8-byte Folded Reload
-; CHECK64-NEXT: blr
-entry:
- tail call void @llvm.write_register.i32(metadata !1, i32 125)
- %0 = tail call i32 @llvm.read_register.i32(metadata !1)
- ret i32 %0
-}
-
-define dso_local void @testViaASM(i32 noundef signext %xx) {
-; CHECK-LABEL: testViaASM:
-; CHECK: # %bb.0: # %entry
-; CHECK-NEXT: stwu 1, -64(1)
-; CHECK-NEXT: .cfi_def_cfa_offset 64
-; CHECK-NEXT: .cfi_offset r20, -48
-; CHECK-NEXT: stw 20, 16(1) # 4-byte Folded Spill
-; CHECK-NEXT: mr 20, 3
-; CHECK-NEXT: #APP
-; CHECK-NEXT: addi 3, 1, 1
-; CHECK-NEXT: #NO_APP
-; CHECK-NEXT: lis 4, myGVal at ha
-; CHECK-NEXT: stw 3, myGVal at l(4)
-; CHECK-NEXT: lwz 20, 16(1) # 4-byte Folded Reload
-; CHECK-NEXT: addi 1, 1, 64
-; CHECK-NEXT: blr
-;
-; CHECK64-LABEL: testViaASM:
-; CHECK64: # %bb.0: # %entry
-; CHECK64-NEXT: std 20, -96(1) # 8-byte Folded Spill
-; CHECK64-NEXT: mr 20, 3
-; CHECK64-NEXT: #APP
-; CHECK64-NEXT: addi 3, 1, 1
-; CHECK64-NEXT: #NO_APP
-; CHECK64-NEXT: addis 4, 2, myGVal at toc@ha
-; CHECK64-NEXT: addi 4, 4, myGVal at toc@l
-; CHECK64-NEXT: stw 3, 0(4)
-; CHECK64-NEXT: ld 20, -96(1) # 8-byte Folded Reload
-; CHECK64-NEXT: blr
-entry:
- tail call void @llvm.write_register.i32(metadata !2, i32 %xx)
- %0 = tail call i32 @llvm.read_register.i32(metadata !2)
- %1 = tail call i32 asm "addi $0, $2, $2", "=r,{r20},K"(i32 %0, i32 1)
- store i32 %1, ptr @myGVal, align 4
- ret void
-}
-
-!0 = !{!"r5"}
-!1 = !{!"r23"}
-!2 = !{!"r20"}
``````````
</details>
https://github.com/llvm/llvm-project/pull/113457
More information about the llvm-commits
mailing list