RFR: 8359435: AArch64: add support for SB instruction to MacroAssembler::spin_wait [v2]

Evgeny Astigeevich eastigeevich at openjdk.org
Wed Jun 25 13:56:10 UTC 2025


On Wed, 25 Jun 2025 13:43:56 GMT, Evgeny Astigeevich <eastigeevich at openjdk.org> wrote:

>> test/hotspot/jtreg/compiler/onSpinWait/TestOnSpinWaitAArch64.java line 80:
>> 
>>> 78:         OutputAnalyzer analyzer = new OutputAnalyzer(pb.start());
>>> 79: 
>>> 80:         if (analyzer.getExitValue() != 0 && "sb".equals(spinWaitInst) && analyzer.contains("CPU does not support SB")) {
>> 
>> The logic here is a bit off. Suppose we _do_ have non-zero exit code for, say, `isb`. This would not fail the test now. Do it something like this instead?
>> 
>> 
>> if ("sb".equals(spinWaitInst) && analyzer.contains("CPU does not support SB")) {
>>     System.out.println("Skipping the test. The current CPU does not support SB instruction.");
>>     return;
>> }
>> 
>> analyzer.shouldHaveExitValue(0);
>
> Thank you, Aleksey for finding this. I accidentally removed `analyzer.shouldHaveExitValue(0)`

Done

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25801#discussion_r2166779245


More information about the hotspot-compiler-dev mailing list