[llvm] 80ccc72 - [AMDGPU] Remove GFX12 encoding hack (#78702)

via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 19 04:19:33 PST 2024


Author: Jay Foad
Date: 2024-01-19T12:19:29Z
New Revision: 80ccc72ec729623f6898d50f75447c3057557a8f

URL: https://github.com/llvm/llvm-project/commit/80ccc72ec729623f6898d50f75447c3057557a8f
DIFF: https://github.com/llvm/llvm-project/commit/80ccc72ec729623f6898d50f75447c3057557a8f.diff

LOG: [AMDGPU] Remove GFX12 encoding hack (#78702)

This is no longer needed now that we have implemented GFX12 encoding for
all instructions.

Added: 
    

Modified: 
    llvm/lib/Target/AMDGPU/SIInstrInfo.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Target/AMDGPU/SIInstrInfo.cpp b/llvm/lib/Target/AMDGPU/SIInstrInfo.cpp
index 48fdb803e9d7346..5a9401103da8228 100644
--- a/llvm/lib/Target/AMDGPU/SIInstrInfo.cpp
+++ b/llvm/lib/Target/AMDGPU/SIInstrInfo.cpp
@@ -9136,12 +9136,6 @@ int SIInstrInfo::pseudoToMCOpcode(int Opcode) const {
 
   int MCOp = AMDGPU::getMCOpcode(Opcode, Gen);
 
-  // TODO-GFX12: Remove this.
-  // Hack to allow some GFX12 codegen tests to run before all the encodings are
-  // implemented.
-  if (MCOp == (uint16_t)-1 && Gen == SIEncodingFamily::GFX12)
-    MCOp = AMDGPU::getMCOpcode(Opcode, SIEncodingFamily::GFX11);
-
   // -1 means that Opcode is already a native instruction.
   if (MCOp == -1)
     return Opcode;


        


More information about the llvm-commits mailing list