From 842be38162ec91d2a045ee360c46f94d1c09b7c4 Mon Sep 17 00:00:00 2001 From: Tim Renouf Date: Mon, 15 Apr 2019 10:36:24 +0000 Subject: [PATCH] [AMDGPU] Fixed incorrect test in vcnd/vcmp optimization This fixes a test I introduced in change D59191 (that added src0 and src1 modifiers to the v_cndmask instruction for disassembly purposes). Spotted by David Binderman in bug 41488. Differential Revision: https://reviews.llvm.org/D60652 Change-Id: I6ac95e66cd84e812ed3359ad57bcd0e13198ba0c llvm-svn: 358392 --- llvm/lib/Target/AMDGPU/SIOptimizeExecMaskingPreRA.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/llvm/lib/Target/AMDGPU/SIOptimizeExecMaskingPreRA.cpp b/llvm/lib/Target/AMDGPU/SIOptimizeExecMaskingPreRA.cpp index 9db1e5af015b..7e843e83780b 100644 --- a/llvm/lib/Target/AMDGPU/SIOptimizeExecMaskingPreRA.cpp +++ b/llvm/lib/Target/AMDGPU/SIOptimizeExecMaskingPreRA.cpp @@ -223,7 +223,7 @@ static unsigned optimizeVcndVcmpPair(MachineBasicBlock &MBB, return AMDGPU::NoRegister; if (TII->hasModifiersSet(*Sel, AMDGPU::OpName::src0_modifiers) || - TII->hasModifiersSet(*Sel, AMDGPU::OpName::src0_modifiers)) + TII->hasModifiersSet(*Sel, AMDGPU::OpName::src1_modifiers)) return AMDGPU::NoRegister; Op1 = TII->getNamedOperand(*Sel, AMDGPU::OpName::src0);