RFR: 8299671: speed up compiler/intrinsics/string/TestStringLatin1IndexOfChar.java [v2]

Emanuel Peter epeter at openjdk.org
Fri Jan 6 08:01:20 UTC 2023


On Thu, 5 Jan 2023 18:07:43 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:

>> Emanuel Peter has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   replaced UseSSE=0 with UseAVX=0
>
> test/hotspot/jtreg/compiler/intrinsics/string/TestStringLatin1IndexOfChar.java line 44:
> 
>> 42:  *                   -Xbatch -XX:+UnlockDiagnosticVMOptions -XX:DisableIntrinsic=_indexOfL_char compiler.intrinsics.string.TestStringLatin1IndexOfChar
>> 43:  * @run main/othervm -Xbootclasspath/a:. -XX:+UnlockDiagnosticVMOptions -XX:+WhiteBoxAPI
>> 44:  *                   -Xbatch -XX:+IgnoreUnrecognizedVMOptions -XX:UseSSE=0 compiler.intrinsics.string.TestStringLatin1IndexOfChar
> 
> I think `-XX:UseSSE=0` was typo in original changes. It should be `-XX:UseAVX=0` because we don't support anything less than SSE2 with 64-bit VM.
> `StrIndexOf` is only supported when `UseSSE42Intrinsics == true` for both 32- and 64-bit VMs. See `x86_*.ad` files.

Thanks, I replaced it.

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

PR: https://git.openjdk.org/jdk/pull/11862


More information about the hotspot-compiler-dev mailing list