RFR: 8288303: C1: Miscompilation due to broken Class.getModifiers intrinsic [v2]

Aleksey Shipilev shade at openjdk.java.net
Tue Jun 14 09:49:51 UTC 2022


On Tue, 14 Jun 2022 06:54:37 GMT, Dean Long <dlong at openjdk.org> wrote:

> The change looks OK. However, it does seem useful for an intrinsic to be able to use "local" labels that won't confuse the register allocator. That seems better than using a less-efficient cmove or having to write the intrinsic at the LIRAssembler level.

Yeah, this is not the first time this happens. So I submitted [JDK-8288317](https://bugs.openjdk.org/browse/JDK-8288317) yesterday hoping for better diagnostics, at least. Maybe local labels would be nice to have, if they are implementable.

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

PR: https://git.openjdk.org/jdk19/pull/8


More information about the hotspot-compiler-dev mailing list