RFR: 8343964: RISC-V: Improve PrintOptoAssembly output for loadNKlassCompactHeaders node

Fei Yang fyang at openjdk.org
Wed Nov 13 02:59:27 UTC 2024


On Tue, 12 Nov 2024 09:27:44 GMT, Hamlin Li <mli at openjdk.org> wrote:

>> Hi, please review this small change.
>> 
>> Currently, we print a simple `lwu` for this node, which is not accurate becasue we do a `ld` and logic shift right the loaded 64-bit value for this node. This simply changed it into `load_narrow_klass_compact` like other CPU platforms. After this change, we have:
>> 
>> 
>> 070     B2: #   out( B8 B3 ) <- in( B1 )  Freq: 0.9
>> 070 +   load_narrow_klass_compact R28, [R12, #4]        # compressed class ptr, #@loadNKlassCompactHeaders
>> 
>> 
>> (Tagging: @Hamlin-Li)
>
> Looks good to me. Thanks!

@Hamlin-Li @feilongjiang : Thanks for the review!

-------------

PR Comment: https://git.openjdk.org/jdk/pull/22025#issuecomment-2472265275


More information about the hotspot-compiler-dev mailing list