This is an archive of the discontinued LLVM Phabricator instance.

[SDAG] Remove FixedArgs parameter from CallLoweringInfo::setCallee
ClosedPublic

Authored by kparzysz on May 18 2016, 10:31 AM.

Details

Summary

The setCallee function will set the number of fixed arguments based on the size of the argument list. The FixedArgs parameter was often explicitly set to 0, leading to a lack of consistent value for non-vararg functions.

Diff Detail

Repository
rL LLVM

Event Timeline

kparzysz updated this revision to Diff 57647.May 18 2016, 10:31 AM
kparzysz retitled this revision from to Properly set CLI.NumFixedArgs for mem{set,move,cpy} builtins in SelectionDAG.
kparzysz updated this object.
kparzysz added a reviewer: mehdi_amini.
kparzysz set the repository for this revision to rL LLVM.
kparzysz added a subscriber: llvm-commits.
kparzysz updated this revision to Diff 60888.Jun 15 2016, 1:29 PM
kparzysz retitled this revision from Properly set CLI.NumFixedArgs for mem{set,move,cpy} builtins in SelectionDAG to [SDAG] Remove FixedArgs parameter from CallLoweringInfo::setCallee.
kparzysz updated this object.
kparzysz removed a reviewer: mehdi_amini.

As per earlier comments, updated the diff to remove FixedArgs parameter from setCallee and fixed all calls to setCallee to reflect the change.

This revision was automatically updated to reflect the committed changes.