Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Uniform intrinsic filename #17

Open
HanKuanChen opened this issue Mar 22, 2021 · 2 comments
Open

Uniform intrinsic filename #17

HanKuanChen opened this issue Mar 22, 2021 · 2 comments

Comments

@HanKuanChen
Copy link

Hi,
I notice that intrinsic filename is in a chaos status.
V extension uses riscv_vector.h and b extension uses rvintrin.h.
Maybe we can separate each intrinsic into an independent file, and users can include what they really need instead of all intrinsics.
(e.g. to use v, b and p extension, they should include riscv_v_intrinsic.h, riscv_b_intrinsic.h and riscv_p_intrinsic.h respectively)
If users want to use all intrinsics, they can include riscv_intrinsic.h.
Any idea?

@jim-wilson
Copy link
Collaborator

The B extension rvintrin.h file can't be upstreamed to FSF GCC due to lack of copyright assignments. It should be ignored.

The B extension plan was to have a single header file for all of the intrinsics, but given the size of the V intrinsic header file, that seems unwise. We should have separate files for each extension, and maybe a file that combines them all. Your suggestion sounds reasonable to me.

@zakk0610
Copy link

Good idea.
IMO, V intrinsic header is too big, so I prefer to have separate header for each extension.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants