Reviewers: fschneider,
Message:
These operands were unused and only complicated things. I've left the
position
of the PushArguments instructions in the instruction stream unchanged.
I'm aware that using inheritance for number of operands is turning out to
be a
bit of a wart. I still did it that way to save code duplication but I'd
really
like to change it.
Description:
Remove the uses of the arguments from all calls.
Before, Hydrogen call instructions had uses of the PushArgument instructions
for their arguments. These operands were unneeded, bloated the IR, and
caused calls to be the only Hydrogen instructions with an unpredictable
number of operands.
Now, PushArgument is a pure side-effecting instruction that has no uses.
Please review this at http://codereview.chromium.org/6480030/
SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge
Affected files:
M src/hydrogen-instructions.h
M src/hydrogen-instructions.cc
M src/hydrogen.h
M src/hydrogen.cc
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
