[webkit-reviews] review granted: [Bug 204442] [JSC] Extend MacroAssemblerARM64::load/store for datasize = 16 : [Attachment 384029] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Nov 20 21:57:13 PST 2019


Mark Lam <mark.lam at apple.com> has granted Yusuke Suzuki <ysuzuki at apple.com>'s
request for review:
Bug 204442: [JSC] Extend MacroAssemblerARM64::load/store for datasize = 16
https://bugs.webkit.org/show_bug.cgi?id=204442

Attachment 384029: Patch

https://bugs.webkit.org/attachment.cgi?id=384029&action=review




--- Comment #7 from Mark Lam <mark.lam at apple.com> ---
Comment on attachment 384029
  --> https://bugs.webkit.org/attachment.cgi?id=384029
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=384029&action=review

r=me with fixes.

>>> Source/JavaScriptCore/assembler/testmasm.cpp:1137
>>> +	 });
>> 
>> How is this test case different than the one immediately above it?
> 
> nullptr is always invalid logical imm in ARM64, so this takes the path this
patch is fixing.

Yusuke told me offline that his intent is to take the !logicalImm.isValid()
case in or16.  For that, we need to pass TrustedImm(0), not
AbsoluteAddress(nullptr).


More information about the webkit-reviews mailing list