From cbf52463599c860243d29877021fcdfcd9d46553 Mon Sep 17 00:00:00 2001 From: Jessica Paquette Date: Tue, 19 Jan 2021 10:38:04 -0800 Subject: [PATCH] Fix buildbot after cfc60730179042a93cb9cb338982e71d20707a24 Windows buildbots were not happy with using find_if + instructionsWithoutDebug. In cfc60730179042a9, instructionsWithoutDebug is not technically necessary. So, just iterate over the block directly. http://lab.llvm.org:8011/#/builders/127/builds/4732/steps/7/logs/stdio --- llvm/lib/CodeGen/GlobalISel/CombinerHelper.cpp | 11 ++++------- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/llvm/lib/CodeGen/GlobalISel/CombinerHelper.cpp b/llvm/lib/CodeGen/GlobalISel/CombinerHelper.cpp index c142c7a..df0219f 100644 --- a/llvm/lib/CodeGen/GlobalISel/CombinerHelper.cpp +++ b/llvm/lib/CodeGen/GlobalISel/CombinerHelper.cpp @@ -625,13 +625,10 @@ bool CombinerHelper::isPredecessor(const MachineInstr &DefMI, if (&DefMI == &UseMI) return false; const MachineBasicBlock &MBB = *DefMI.getParent(); - auto NonDbgInsts = - instructionsWithoutDebug(MBB.instr_begin(), MBB.instr_end()); - auto DefOrUse = - find_if(NonDbgInsts, [&DefMI, &UseMI](const MachineInstr &MI) { - return &MI == &DefMI || &MI == &UseMI; - }); - if (DefOrUse == NonDbgInsts.end()) + auto DefOrUse = find_if(MBB, [&DefMI, &UseMI](const MachineInstr &MI) { + return &MI == &DefMI || &MI == &UseMI; + }); + if (DefOrUse == MBB.end()) llvm_unreachable("Block must contain both DefMI and UseMI!"); return &*DefOrUse == &DefMI; } -- 2.7.4