On Thu, 5 Nov 2026 00:01:17 GMT, Yasumasa Suenaga <[email protected]> wrote:
>> Gui Cao has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Code format
>
> test/hotspot/jtreg/serviceability/sa/TestContinuationEntry.java line 64:
>
>> 62: // See ContinuationEntry::entry_fp() in
>> continuationEntry_riscv.inline.hpp and
>> 63: // RISCV64ContinuationEntry.getEntryFP() in SA.
>> 64: long expectedOffset = alignedSize + (cpu.equals("riscv64") ?
>> 2 * wordSize : 0);
>
> We can use `jdk.test.lib.Platform::isRISCV64()` here.
Thanks for the review.
I see there is already code that follows the same style, let's leave it as is.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/33695#discussion_r4475577123
