BlockGenerator: Drop unnecessary return value

llvm-svn: 261473
This commit is contained in:
Tobias Grosser 2016-02-21 15:44:34 +00:00
parent b1cc4d6d69
commit 2b809d1390
2 changed files with 8 additions and 15 deletions

View File

@ -502,12 +502,8 @@ protected:
///
/// The implementation in the BlockGenerator is trivial, however it allows
/// subclasses to handle PHIs different.
///
/// @returns The nullptr as the BlockGenerator does not copy PHIs.
virtual Value *copyPHIInstruction(ScopStmt &, PHINode *, ValueMapT &,
LoopToScevMapT &) {
return nullptr;
}
virtual void copyPHIInstruction(ScopStmt &, PHINode *, ValueMapT &,
LoopToScevMapT &) {}
/// @brief Copy a single Instruction.
///
@ -824,11 +820,9 @@ private:
/// @param BBMap A mapping from old values to their new values
/// (for values recalculated within this basic block).
/// @param LTS A map from old loops to new induction variables as SCEVs.
///
/// @returns The copied instruction or nullptr if no copy was made.
virtual Value *copyPHIInstruction(ScopStmt &Stmt, PHINode *Inst,
ValueMapT &BBMap,
LoopToScevMapT &LTS) override;
virtual void copyPHIInstruction(ScopStmt &Stmt, PHINode *Inst,
ValueMapT &BBMap,
LoopToScevMapT &LTS) override;
};
}
#endif

View File

@ -1313,9 +1313,9 @@ void RegionGenerator::addOperandToPHI(ScopStmt &Stmt, const PHINode *PHI,
PHICopy->addIncoming(OpCopy, BBCopy);
}
Value *RegionGenerator::copyPHIInstruction(ScopStmt &Stmt, PHINode *PHI,
ValueMapT &BBMap,
LoopToScevMapT &LTS) {
void RegionGenerator::copyPHIInstruction(ScopStmt &Stmt, PHINode *PHI,
ValueMapT &BBMap,
LoopToScevMapT &LTS) {
unsigned NumIncoming = PHI->getNumIncomingValues();
PHINode *PHICopy =
Builder.CreatePHI(PHI->getType(), NumIncoming, "polly." + PHI->getName());
@ -1324,5 +1324,4 @@ Value *RegionGenerator::copyPHIInstruction(ScopStmt &Stmt, PHINode *PHI,
for (unsigned u = 0; u < NumIncoming; u++)
addOperandToPHI(Stmt, PHI, PHICopy, PHI->getIncomingBlock(u), LTS);
return PHICopy;
}