Do not unswitch loop on same value again and again.

llvm-svn: 34638
This commit is contained in:
Devang Patel 2007-02-26 19:31:58 +00:00
parent 656996aab8
commit b0b949f93e

View File

@ -62,6 +62,8 @@ namespace {
// LoopProcessWorklist - List of loops we need to process.
std::vector<Loop*> LoopProcessWorklist;
std::set<Value *> UnswitchedVals;
public:
virtual bool runOnFunction(Function &F);
bool visitLoop(Loop *L);
@ -186,6 +188,11 @@ bool LoopUnswitch::visitLoop(Loop *L) {
// Find a value to unswitch on:
// FIXME: this should chose the most expensive case!
Constant *UnswitchVal = SI->getCaseValue(1);
// Do not process same value again and again.
if (UnswitchedVals.count(UnswitchVal) != 0)
continue;
UnswitchedVals.insert(UnswitchVal);
if (UnswitchIfProfitable(LoopCond, UnswitchVal, L)) {
++NumSwitches;
return true;