RFR: 8267625: AARCH64: typo in LIR_Assembler::emit_profile_type

Nick Gasson ngasson at openjdk.java.net
Tue Jul 6 09:17:08 UTC 2021


This code is generated when C1 is profiling types but it is known statically that there should only be one possible class at this point. It's supposed to then store the klass pointer into the method data when the expected class is actually observed but currently it does a load instead of a store.

I've attached a simple example to the JBS entry that shows the correct method data after changing LDR to STR. Also tested tier1 on AArch64.

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

Commit messages:
 - 8267625: AARCH64: typo in LIR_Assembler::emit_profile_type

Changes: https://git.openjdk.java.net/jdk/pull/4686/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=4686&range=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8267625
  Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
  Patch: https://git.openjdk.java.net/jdk/pull/4686.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/4686/head:pull/4686

PR: https://git.openjdk.java.net/jdk/pull/4686


More information about the hotspot-compiler-dev mailing list