RFR: 8352869: Verify.checkEQ: extension for NaN, VectorAPI and arbitrary Objects [v3]
Emanuel Peter
epeter at openjdk.org
Fri Mar 28 09:28:14 UTC 2025
On Fri, 28 Mar 2025 05:04:42 GMT, Galder Zamarreño <galder at openjdk.org> wrote:
>> Emanuel Peter has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Update test/hotspot/jtreg/compiler/lib/verify/Verify.java
>>
>> Co-authored-by: Galder Zamarreño <galder at redhat.com>
>
> test/hotspot/jtreg/compiler/lib/verify/Verify.java line 264:
>
>> 262: private void checkEQimpl(float a, float b, String field, Object aParent, Object bParent) {
>> 263: if (isFloatEQ(a, b)) {
>> 264: System.err.println("ERROR: Verify.checkEQ failed: value mismatch. check raw: " + isFloatCheckWithRawBits);
>
> Would using a text block here make it more readable?
@galderz What do you mean by a text block? String Templates would be nice, but we don't have them. Do you mean I should use `String.format`? But there is always so tricky to know what is going to be formatted to where...
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24224#discussion_r2018220193
More information about the hotspot-compiler-dev
mailing list