From: Nirav Dave Date: Thu, 24 Jan 2019 15:04:17 +0000 (+0000) Subject: [X86] Add missing isReg() guards in FixupSetCCs pass. X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=845e81db063f8f9ac37f3f910900717a055979eb;p=llvm [X86] Add missing isReg() guards in FixupSetCCs pass. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@352051 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/lib/Target/X86/X86FixupSetCC.cpp b/lib/Target/X86/X86FixupSetCC.cpp index 421e6e0ba8a..5bfad71ec05 100644 --- a/lib/Target/X86/X86FixupSetCC.cpp +++ b/lib/Target/X86/X86FixupSetCC.cpp @@ -102,7 +102,7 @@ X86FixupSetCCPass::findFlagsImpDef(MachineBasicBlock *MBB, auto MBBStart = MBB->rend(); for (int i = 0; (i < SearchBound) && (MI != MBBStart); ++i, ++MI) for (auto &Op : MI->implicit_operands()) - if ((Op.getReg() == X86::EFLAGS) && (Op.isDef())) + if (Op.isReg() && (Op.getReg() == X86::EFLAGS) && Op.isDef()) return &*MI; return nullptr; @@ -110,7 +110,7 @@ X86FixupSetCCPass::findFlagsImpDef(MachineBasicBlock *MBB, bool X86FixupSetCCPass::impUsesFlags(MachineInstr *MI) { for (auto &Op : MI->implicit_operands()) - if ((Op.getReg() == X86::EFLAGS) && (Op.isUse())) + if (Op.isReg() && (Op.getReg() == X86::EFLAGS) && Op.isUse()) return true; return false;