[Webkit-unassigned] [Bug 101328] MIPS DFG implementation.
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Mon Jan 7 11:04:42 PST 2013
https://bugs.webkit.org/show_bug.cgi?id=101328
Filip Pizlo <fpizlo at apple.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #181319|review?, commit-queue? |review-, commit-queue-
Flag| |
--- Comment #22 from Filip Pizlo <fpizlo at apple.com> 2013-01-07 11:06:36 PST ---
(From update of attachment 181319)
View in context: https://bugs.webkit.org/attachment.cgi?id=181319&action=review
>>>> Source/JavaScriptCore/dfg/DFGCCallHelpers.h:963
>>>> +
>>>
>>> Why is all of this behind CPU(MIPS)? It looks identical to the normal NUMBER_OF_ARGUMENT_REGISTERS == 4 case. :-/
>>
>> Thank you for the review. On MIPS we have to give space for the register stored arguments on stack also so while other (params in 4 regs) arches can use:
>> poke(arg5, 1);
>> poke(arg4);
>> setupArgumentsWithExecState(arg1, arg2, arg3);
>> on MIPS we have to use:
>> poke(arg5, 5);
>> poke(arg4, 4);
>> setupArgumentsWithExecState(arg1, arg2, arg3);
>> Different stack offset values in poke call.
>
> Then I would suggest abstracting the poke offset, and using #if's only for that. That way, you don't have to duplicate all of that code!
>
> Keep in mind that this code gets churned *a lot*. Every time we add a DFGOperations function with a sufficiently exotic signature, this code gets touched. Therefore, we should reduce code duplication as much as possible; otherwise you guys will have a lot more work to do to keep up with DFG changes.
See above comment. I still think you should abstract the poke offset rather than duplicating all of this code.
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list