[llvm] 678da7b - AMDGPU/GlobalISel: Remove leftover #if 0

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 19 17:07:15 PDT 2020


Author: Matt Arsenault
Date: 2020-03-19T20:07:05-04:00
New Revision: 678da7b109fffe6ed4a5e50974d28535e73b4dae

URL: https://github.com/llvm/llvm-project/commit/678da7b109fffe6ed4a5e50974d28535e73b4dae
DIFF: https://github.com/llvm/llvm-project/commit/678da7b109fffe6ed4a5e50974d28535e73b4dae.diff

LOG: AMDGPU/GlobalISel: Remove leftover #if 0

The subtarget feature used to be missing from subtargets, but that was
fixed.

Added: 
    

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

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp b/llvm/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp
index 47866501776d..01fd8483e5c1 100644
--- a/llvm/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp
+++ b/llvm/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp
@@ -3548,12 +3548,10 @@ bool AMDGPULegalizerInfo::legalizeImageIntrinsic(
 
   // Rewrite the addressing register layout before doing anything else.
   if (IsA16) {
-#if 0
     // FIXME: this feature is missing from gfx10. When that is fixed, this check
     // should be introduced.
-    if (!ST.hasFeature(AMDGPU::FeatureR128A16))
+    if (!ST.hasR128A16() && !ST.hasGFX10A16())
       return false;
-#endif
 
     if (NumVAddrs > 1) {
       SmallVector<Register, 4> PackedRegs;


        


More information about the llvm-commits mailing list