[llvm] r371406 - AMDGPU/GlobalISel: Try generated matcher before add/sub code

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 9 08:20:44 PDT 2019


Author: arsenm
Date: Mon Sep  9 08:20:44 2019
New Revision: 371406

URL: http://llvm.org/viewvc/llvm-project?rev=371406&view=rev
Log:
AMDGPU/GlobalISel: Try generated matcher before add/sub code

This will allow optimization patterns which fold adds away to work.

Modified:
    llvm/trunk/lib/Target/AMDGPU/AMDGPUInstructionSelector.cpp

Modified: llvm/trunk/lib/Target/AMDGPU/AMDGPUInstructionSelector.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/AMDGPU/AMDGPUInstructionSelector.cpp?rev=371406&r1=371405&r2=371406&view=diff
==============================================================================
--- llvm/trunk/lib/Target/AMDGPU/AMDGPUInstructionSelector.cpp (original)
+++ llvm/trunk/lib/Target/AMDGPU/AMDGPUInstructionSelector.cpp Mon Sep  9 08:20:44 2019
@@ -1289,11 +1289,9 @@ bool AMDGPUInstructionSelector::select(M
     return selectImpl(I, *CoverageInfo);
   case TargetOpcode::G_ADD:
   case TargetOpcode::G_SUB:
-    if (selectG_ADD_SUB(I))
+    if (selectImpl(I, *CoverageInfo))
       return true;
-    LLVM_FALLTHROUGH;
-  default:
-    return selectImpl(I, *CoverageInfo);
+    return selectG_ADD_SUB(I);
   case TargetOpcode::G_INTTOPTR:
   case TargetOpcode::G_BITCAST:
     return selectCOPY(I);
@@ -1355,6 +1353,8 @@ bool AMDGPUInstructionSelector::select(M
     // is checking for G_CONSTANT
     I.setDesc(TII.get(AMDGPU::ATOMIC_FENCE));
     return true;
+  default:
+    return selectImpl(I, *CoverageInfo);
   }
   return false;
 }




More information about the llvm-commits mailing list