RESOLVED FIXED 127155
Eliminate {push,pop}CalleeSaves in favor of individual pushes & pops 
https://bugs.webkit.org/show_bug.cgi?id=127155
Summary Eliminate {push,pop}CalleeSaves in favor of individual pushes & pops
Michael Saboff
Reported 2014-01-16 17:28:33 PST
pushCalleeSaves and popCalleeSaves are pseudo ops in the low level assembler. As such, the registers are not visible where these pseudo ops are used. These pseudo ops should be eliminated and replaced with explicit pushes and pops.
Attachments
Patch (17.70 KB, patch)
2014-08-11 13:56 PDT, Michael Saboff 		ggaren: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Michael Saboff
Comment 1 2014-08-11 13:56:46 PDT
Created attachment 236398 [details] Patch
Geoffrey Garen
Comment 2 2014-08-11 16:11:21 PDT
Comment on attachment 236398 [details] Patch Seems reasonable. We should be careful not to use "emit" in too many places. This is one good place for it.
Michael Saboff
Comment 3 2014-08-11 19:32:50 PDT
(In reply to comment #2) > (From update of attachment 236398 [details]) > Seems reasonable. > > We should be careful not to use "emit" in too many places. This is one good place for it. I agree. Currently I don't have any other uses in mind.
Michael Saboff
Comment 4 2014-08-11 20:20:26 PDT
Committed r172429: <http://trac.webkit.org/changeset/172429>
Note You need to log in before you can comment on or make changes to this bug.