[webkit-reviews] review granted: [Bug 216082] AbstractMacroAssembler::Jump class has uninitialized instance variables : [Attachment 407778] Patch v1

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Sep 2 11:40:15 PDT 2020

Michael Saboff <msaboff at apple.com> has granted David Kilzer (:ddkilzer)
<ddkilzer at webkit.org>'s request for review:
Bug 216082: AbstractMacroAssembler::Jump class has uninitialized instance

Attachment 407778: Patch v1


--- Comment #3 from Michael Saboff <msaboff at apple.com> ---
Comment on attachment 407778
  --> https://bugs.webkit.org/attachment.cgi?id=407778
Patch v1

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

r=me.  This is good defensive programming, but I don't think that this fixes
any latent bugs as these fields are set for the instruction variants that use

>> Source/JavaScriptCore/assembler/AbstractMacroAssembler.h:685
>> +	    bool m_is64Bit { false };
> Should this be { true } instead since this is only compiled on CPU(ARM64)? 
Or do we have 32-bit jumps in arm64[e]?

All jumps for ARM64[e] are 64 bit in terms of changes to the PC.  The m_is64Bit
is for the compare and branch jumps (CBZ and CBNZ) and relate to the size of
the comparison.  The m_is64Bit flag is not used for the other branch types.

More information about the webkit-reviews mailing list