commit a81198d82de20e14efce44826e302785bfda093a
author Matt Arsenault <Matthew.Arsenault@amd.com> Thu Jul 06 20:56:59 2017 +0000
committer Matt Arsenault <Matthew.Arsenault@amd.com> Thu Jul 06 20:56:59 2017 +0000
tree e81c154788fd7aa06f0c6510336a97fc1cb6ae01
parent 60b91e0ba29e93c91715e367c767f9e223c28aca

AMDGPU: Minor cleanup of shrinking logic

llvm-svn: 307312

llvm/lib/Target/AMDGPU/SIShrinkInstructions.cpp

diff --git a/llvm/lib/Target/AMDGPU/SIShrinkInstructions.cpp b/llvm/lib/Target/AMDGPU/SIShrinkInstructions.cpp
index 96a1854..6bff2c5 100644
--- a/llvm/lib/Target/AMDGPU/SIShrinkInstructions.cpp
+++ b/llvm/lib/Target/AMDGPU/SIShrinkInstructions.cpp
@@ -110,10 +110,8 @@
   }
 
   const MachineOperand *Src1 = TII->getNamedOperand(MI, AMDGPU::OpName::src1);
-  const MachineOperand *Src1Mod =
-      TII->getNamedOperand(MI, AMDGPU::OpName::src1_modifiers);
-
-  if (Src1 && (!isVGPR(Src1, TRI, MRI) || (Src1Mod && Src1Mod->getImm() != 0)))
+  if (Src1 && (!isVGPR(Src1, TRI, MRI) ||
+               TII->hasModifiersSet(MI, AMDGPU::OpName::src1_modifiers)))
     return false;
 
   // We don't need to check src0, all input types are legal, so just make sure
@@ -122,10 +120,8 @@
     return false;
 
   // Check output modifiers
-  if (TII->hasModifiersSet(MI, AMDGPU::OpName::omod))
-    return false;
-
-  return !TII->hasModifiersSet(MI, AMDGPU::OpName::clamp);
+  return !TII->hasModifiersSet(MI, AMDGPU::OpName::omod) &&
+         !TII->hasModifiersSet(MI, AMDGPU::OpName::clamp);
 }
 
 /// \brief This function checks \p MI for operands defined by a move immediate