[14] RFR(XS): 8230762: Change MacroAssembler::debug32/64 to use fatal instead of assert

Christian Hagedorn christian.hagedorn at oracle.com
Mon Sep 9 14:14:18 UTC 2019


Please review the following patch:

The webrev is based on JDK-8225653. MacroAssembler::debug32/64 is 
changed such that the last operation is fatal(). I think that some 
additional instructions like ThreadStateTransition::transition or 
tty->print_cr are then not needed anymore. I also ensured that after 
each debug32/64 call a hlt is directly emitted (which was required in 
generate_verify_oop - in MacroAssembler::stop we already have a hlt 
instruction afterwards).

Thank you!

Best regards,

More information about the hotspot-dev mailing list