maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

Remove dead F parameter from Argument constructor 

When Function creates its argument list, it does the ilist push_back
itself. No other caller passes in a parent function, so this is dead,
and it uses the soon-to-be-deleted getArgumentList accessor.

llvm-svn: 298009
This commit is contained in:
Reid Kleckner 2017-03-16 22:58:56 +00:00
parent 720883520d
commit c9a392b9dd
2 changed files with 5 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
llvm/include/llvm/IR/Argument.h
View File

@ -38,10 +38,8 @@ class Argument : public Value, public ilist_node<Argument> {
void setParent(Function *parent);
public:
/// If \p F is specified, the argument is inserted at the end of the argument
/// list for \p F.
explicit Argument(Type *Ty, const Twine &Name = "", Function *F = nullptr,
unsigned ArgNo = 0);
/// Argument constructor.
explicit Argument(Type *Ty, const Twine &Name = "", unsigned ArgNo = 0);
inline const Function *getParent() const { return Parent; }
inline Function *getParent() { return Parent; }

10
llvm/lib/IR/Function.cpp
View File

@ -39,12 +39,8 @@ template class llvm::SymbolTableListTraits<BasicBlock>;
void Argument::anchor() { }
Argument::Argument(Type *Ty, const Twine &Name, Function *Par, unsigned ArgNo)
: Value(Ty, Value::ArgumentVal), ArgNo(ArgNo) {
Parent = nullptr;
if (Par)
Par->getArgumentList().push_back(this);
Argument::Argument(Type *Ty, const Twine &Name, unsigned ArgNo)
: Value(Ty, Value::ArgumentVal), Parent(nullptr), ArgNo(ArgNo) {
setName(Name);
}
@ -233,7 +229,7 @@ void Function::BuildLazyArguments() const {
assert(!FT->getParamType(i)->isVoidTy() &&
"Cannot have void typed arguments!");
ArgumentList.push_back(
new Argument(FT->getParamType(i), "", nullptr, i));
new Argument(FT->getParamType(i), "", i));
}
// Clear the lazy arguments bit.