On Mon, 7 Apr 2025 06:44:16 GMT, Per Minborg <[email protected]> wrote:
>> Vladimir Ivanov has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> features_string -> cpu_info_string
>
> src/jdk.incubator.vector/share/classes/jdk/incubator/vector/VectorMathLibrary.java
> line 258:
>
>> 256: if (LIBRARY.isSupported(op, vspecies)) {
>> 257: String symbol = LIBRARY.symbolName(op, vspecies);
>> 258: MemorySegment addr = LOOKUP.find(symbol)
>
> It is better to use `LOOKUP.findOrThrow()` because it does not require lambda
> creation.
Thanks, changed as you suggested. I introduced a try-catch block instead.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24462#discussion_r2032138430
