From 46123a8891c979ed28a61c35278bf610057b7762 Mon Sep 17 00:00:00 2001 From: Justin Lebar Date: Mon, 22 Feb 2016 18:17:27 +0000 Subject: [PATCH] Revert "[ifcnv] Add comment explaining why it's OK to duplicate convergent MIs in ifcnv." This reverts r261543. Accidental commit (not LGTM'ed). llvm-svn: 261547 --- llvm/lib/CodeGen/IfConversion.cpp | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/llvm/lib/CodeGen/IfConversion.cpp b/llvm/lib/CodeGen/IfConversion.cpp index a8e72f4..41f9f0b 100644 --- a/llvm/lib/CodeGen/IfConversion.cpp +++ b/llvm/lib/CodeGen/IfConversion.cpp @@ -7,8 +7,7 @@ // //===----------------------------------------------------------------------===// // -// This file implements the machine instruction level if-conversion pass, which -// tries to convert conditional branches into predicated instructions. +// This file implements the machine instruction level if-conversion pass. // //===----------------------------------------------------------------------===// @@ -674,9 +673,6 @@ void IfConverter::ScanInstructions(BBInfo &BBI) { if (I->isDebugValue()) continue; - // Don't need to check isConvergent(). It's OK to duplicate convergent - // instructions here, because we'll only push convergent operations up the - // CFG -- this can only *remove* control-flow dependencies. if (I->isNotDuplicable()) BBI.CannotBeCopied = true; -- 2.7.4