replica
/
hanchenye-llvm-project
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Make this a bit more explicit about which cases need the 

check.  No functionality change.

llvm-svn: 62474
This commit is contained in:
Chris Lattner 2009-01-18 23:22:07 +00:00
parent 6e2c6844c1
commit e1c01e4e2b
1 changed files with 6 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
llvm/lib/Transforms/Utils/SimplifyCFG.cpp
View File

@ -969,14 +969,18 @@ static bool SpeculativelyExecuteBB(BranchInst *BI, BasicBlock *BB1) {
default: return false; // Not safe / profitable to hoist.
case Instruction::Add:
case Instruction::Sub:
// FP arithmetic might trap. Not worth doing for vector ops.
if (I->getType()->isFloatingPoint() || isa<VectorType>(I->getType()))
return false;
break;
case Instruction::And:
case Instruction::Or:
case Instruction::Xor:
case Instruction::Shl:
case Instruction::LShr:
case Instruction::AShr:
if (!I->getOperand(0)->getType()->isInteger())
// FP arithmetic might trap. Not worth doing for vector ops.
// Don't mess with vector operations.
if (isa<VectorType>(I->getType()))
return false;
break; // These are all cheap and non-trapping instructions.
}