math routine implementation with .s files
Vladimir Ivanov
vladimir.x.ivanov at oracle.com
Thu May 31 06:27:04 UTC 2018
Looks good!
As a side note, number of files added per operation looks like an
overkill. What do you think about having a single .s file per operation?
src/hotspot/os_cpu/linux_x86/svml_d_acos2_core_exha.s
src/hotspot/os_cpu/linux_x86/svml_d_acos2_core_l9ha.s
src/hotspot/os_cpu/linux_x86/svml_d_acos4_core_l9ha.s
src/hotspot/os_cpu/linux_x86/svml_d_acos8_core_z0ha.s
src/hotspot/os_cpu/linux_x86/svml_d_acos_cout_exha.s
src/hotspot/os_cpu/linux_x86/svml_d_acos_data_exha.s
Also, some comments about naming. Current convention is to have os/cpu
info duplicated in file name as well (e.g.,
src/hotspot/os_cpu/linux_x86/linux_x86_64.s or assembler_linux_x86.cpp).
And I'd prefer the names clearly state their relation to Vector API
(either in the file name or placed in a subdir).
(But that's not something requiring immediate attention.)
Best regards,
Vladimir Ivanov
On 31/05/2018 04:51, Kandu, Rahul wrote:
> Hi All,
>
> Please find the patch which implements several transcendental and trigonometric vector math routines in from of assembly .s files.
> http://cr.openjdk.java.net/~rlupusoru/panama/webrev_linuxsvml_00/
>
> This is an extension of the patch submitted by Razvan earlier which implemented the Infrastructure for calling math routines in .s files.
>
> regards,
> Rahul
>
More information about the panama-dev
mailing list