RFR: 8346836: C2: Introduce a way to verify the correctness of ConstraintCastNodes at runtime [v7]

Vladimir Ivanov vlivanov at openjdk.org
Tue Apr 8 00:49:14 UTC 2025


On Tue, 8 Apr 2025 00:22:36 GMT, Vladimir Ivanov <vlivanov at openjdk.org> wrote:

>> Quan Anh Mai has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   make the flag diagnostic
>
> src/hotspot/cpu/x86/c2_MacroAssembler_x86.cpp line 867:
> 
>> 865: 
>> 866: static void abort_checked_cast_long(jlong val, jlong lo, jlong hi) {
>> 867:   fatal("Invalid CastLL, val: %lld, lo: %lld, hi: %lld", (long long)val, (long long)lo, (long long)hi);
> 
> There's `JLONG_FORMAT` to pretty-print jlongs.

FTR it would be nice to include CastII/CastLL node ID to assist diagnosing the bug, but, unfortunately, there's no easy way to capture such information during matching.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22880#discussion_r2032207380


More information about the hotspot-compiler-dev mailing list