[webkit-reviews] review denied: [Bug 117281] Crash in V8 benchmarks set in ARM, softfp, EABI : [Attachment 211279] verification test
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Wed Sep 11 12:22:42 PDT 2013
Geoffrey Garen <ggaren at apple.com> has denied Youngho Yoo
<youngho33.yoo at lge.com>'s request for review:
Bug 117281: Crash in V8 benchmarks set in ARM,softfp,EABI
https://bugs.webkit.org/show_bug.cgi?id=117281
Attachment 211279: verification test
https://bugs.webkit.org/attachment.cgi?id=211279&action=review
------- Additional Comments from Geoffrey Garen <ggaren at apple.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=211279&action=review
> Source/JavaScriptCore/dfg/DFGCCallHelpers.h:571
> + ALWAYS_INLINE void setupArgumentsWithExecState(TrustedImm32 arg1, FPRReg
arg2, GPRReg arg3)
> + {
> + moveDouble(arg2, FPRInfo::argumentFPR0);
> + move(arg3, GPRInfo::argumentGPR1);
> + move(GPRInfo::callFrameRegister, GPRInfo::argumentGPR0);
> + }
> +
> + ALWAYS_INLINE void setupArgumentsWithExecState(TrustedImm32 arg1, GPRReg
arg2, GPRReg arg3, FPRReg arg4)
> + {
> + moveDouble(arg4, FPRInfo::argumentFPR0);
> + setupStubArguments(arg2, arg3);
> + move(GPRInfo::callFrameRegister, GPRInfo::argumentGPR0);
> + }
It looks like arg1 is unused in these functions. Is that a bug? I believe that
shouldn't even build, due to the unused argument warning. Did you test this
code on ARM_HARDFP?
More information about the webkit-reviews
mailing list