[aarch64-port-dev ] [15] RFR(XS) 8247350: [aarch64] assert(false) failed: wrong size of mach node

[Jiva, Azeem]

Looks good to me.

-- 
Azeem Jiva 

On 6/10/20, 5:42 PM, "hotspot-compiler-dev on behalf of Vladimir Kozlov" <hotspot-compiler-dev-bounces at openjdk.java.net on behalf of vladimir.kozlov at oracle.com> wrote:

    CAUTION: This email originated from outside of the organization. Do not click links or open attachments unless you can confirm the sender and know the content is safe.



    https://bugs.openjdk.java.net/browse/JDK-8247350
    https://cr.openjdk.java.net/~kvn/8247350/webrev.00/

    Code size for decodeHeapOop is different in scratch buffer with -XX:+VerifyOops flag on. The difference comes from
    MacroAssembler::mov_immediate64() which generates different code if distance to message string 'b' in mov(rscratch1,
    (address)b) is different.

    Use movptr() instead of mov() in verify_oop() method to generate consistent code for loading C string address.
    Added debug dump code in case of mismatching code generation.

    Tested with failing test case.

    Thanks,
    Vladimir