[llvm] 2f82662 - [PowerPC] Let base implementation decide if MI is rematerizable by default (#75772)

via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 18 01:39:27 PST 2023


Author: Kai Luo
Date: 2023-12-18T17:39:22+08:00
New Revision: 2f82662ce901c6666fceb9c6c5e0de216a1c9667

URL: https://github.com/llvm/llvm-project/commit/2f82662ce901c6666fceb9c6c5e0de216a1c9667
DIFF: https://github.com/llvm/llvm-project/commit/2f82662ce901c6666fceb9c6c5e0de216a1c9667.diff

LOG: [PowerPC] Let base implementation decide if MI is rematerizable by default (#75772)

If MI is not PPC specific instructions, let base implementation decide
if MI is rematerizable.
This can fix failure in #75570 after #75271 .

Added: 
    

Modified: 
    llvm/lib/Target/PowerPC/PPCInstrInfo.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Target/PowerPC/PPCInstrInfo.cpp b/llvm/lib/Target/PowerPC/PPCInstrInfo.cpp
index d0a6cced1b1974..4dfd0358a16fd4 100644
--- a/llvm/lib/Target/PowerPC/PPCInstrInfo.cpp
+++ b/llvm/lib/Target/PowerPC/PPCInstrInfo.cpp
@@ -1067,9 +1067,7 @@ bool PPCInstrInfo::isReallyTriviallyReMaterializable(
     const MachineInstr &MI) const {
   switch (MI.getOpcode()) {
   default:
-    // This function should only be called for opcodes with the ReMaterializable
-    // flag set.
-    llvm_unreachable("Unknown rematerializable operation!");
+    // Let base implementaion decide.
     break;
   case PPC::LI:
   case PPC::LI8:


        


More information about the llvm-commits mailing list