RESOLVED FIXED 117719
FTL: Add another temp register regT4 to JSInterfaceJIT 
https://bugs.webkit.org/show_bug.cgi?id=117719
Summary FTL: Add another temp register regT4 to JSInterfaceJIT
Michael Saboff
Reported 2013-06-17 16:14:10 PDT
The arity fixup thunk as currently envisioned needs 5 temp registers. Other code may need another temp register as well. We can use bucketCounterRegister since its current use only really needs some entropy and that it isn't overwritten during value profiling.
Attachments
Patch (5.64 KB, patch)
2013-06-17 16:18 PDT, Michael Saboff 		ggaren: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Michael Saboff
Comment 1 2013-06-17 16:18:23 PDT
Created attachment 204865 [details] Patch
Geoffrey Garen
Comment 2 2013-06-17 16:24:55 PDT
Comment on attachment 204865 [details] Patch You should also make the emitValueProfilingSite() callers pass the scratch register (regT4) to emitValueProfilingSite(). That way, it will be clear if an opcode allocates regT4 for two uses.
Michael Saboff
Comment 3 2013-06-17 16:25:39 PDT
(In reply to comment #2) > (From update of attachment 204865 [details]) > You should also make the emitValueProfilingSite() callers pass the scratch register (regT4) to emitValueProfilingSite(). That way, it will be clear if an opcode allocates regT4 for two uses. Will do.
Michael Saboff
Comment 4 2013-06-17 16:50:43 PDT
Committed r151658: <http://trac.webkit.org/changeset/151658>
Note You need to log in before you can comment on or make changes to this bug.