On Sat, 28 Feb 2026 00:07:15 GMT, Yasumasa Suenaga <[email protected]> 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

Thanks both for your help!

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

PR Comment: https://git.openjdk.org/jdk/pull/29835#issuecomment-3976240516

Reply via email to