RFR: 8378312: [VectorAPI] libraryUnaryOp/libraryBinaryOp failed to intrinsify [v5]
Paul Sandoz
psandoz at openjdk.org
Sat Feb 28 00:10:29 UTC 2026
On Sat, 28 Feb 2026 00:07:15 GMT, Yasumasa Suenaga <ysuenaga at openjdk.org> wrote:
>> I ran following code with `-Djdk.incubator.vector.DEBUG=true`, however it was not intrinsified.
>>
>>
>> public static void cos(){
>> var rand = new Random();
>> for(int i = 0; i < 1_000_000; i++){
>> var vec = DoubleVector.fromArray(DoubleVector.SPECIES_256, rand.doubles().limit(4).toArray(), 0);
>>
>> vec.lanewise(VectorOperators.COS);
>> }
>> }
>>
>>
>> I knew `__jsvml_cos4_ha_l9()` in libjsvml.so should be called in this case (via vector debug log), but it was not hitted even though I set breakpoint in it.
>>
>> `LaneType` has been introduced in [JDK-8376187](https://bugs.openjdk.org/browse/JDK-8376187), and `LibraryCallKit::inline_vector_call()` was updated to handle lane type rather than class of element type. However `VectorMathLibrary` wasn't modified.
>>
>> I confirmed `__jsvml_cos4_ha_l9()` is called with this fix, and also all jdk/jdk/incubator/vector jtreg tests passed on Linux AMD64.
>
> Yasumasa Suenaga has updated the pull request incrementally with one additional commit since the last revision:
>
> Rename testcase
Marked as reviewed by psandoz (Reviewer).
-------------
PR Review: https://git.openjdk.org/jdk/pull/29835#pullrequestreview-3869646309
More information about the core-libs-dev
mailing list