[webkit-reviews] review denied: [Bug 122617] transition void cti_op_* methods to JIT operations. : [Attachment 214265] With fixes Mark suggested
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Tue Oct 15 09:22:59 PDT 2013
Geoffrey Garen <ggaren at apple.com> has denied Michael Saboff
<msaboff at apple.com>'s request for review:
Bug 122617: transition void cti_op_* methods to JIT operations.
https://bugs.webkit.org/show_bug.cgi?id=122617
Attachment 214265: With fixes Mark suggested
https://bugs.webkit.org/attachment.cgi?id=214265&action=review
------- Additional Comments from Geoffrey Garen <ggaren at apple.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=214265&action=review
> Source/JavaScriptCore/jit/CCallHelpers.h:1096
> + poke(arg4, POKE_ARGUMENT_OFFSET);
Will this poke be valid when we're running on the C stack? Will each stack
frame reserve space for these pokes?
> Source/JavaScriptCore/jit/JITOpcodes32_64.cpp:1185
> + callOperation(operationProfileWillCall, regT1, regT0);
Wrong function = broken profiler. Did the profiler tests pass for this patch?
More information about the webkit-reviews
mailing list