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

Copybara import of the project: #7182

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

copybara-service[bot]
Copy link
Contributor

@copybara-service copybara-service bot commented Sep 25, 2024

Copybara import of the project:

--
07db838 by Mihai.Olinovici [email protected]:

Add RVV f32-f16-vcvt kernels and configs.

--
2f40edd by kaustubh-raste [email protected]:

Fix the cmake and bzl file

FUTURE_COPYBARA_INTEGRATE_REVIEW=#7128 from imaginationtech:img_patch30_f32_f16_vcvt 2f40edd

@copybara-service copybara-service bot force-pushed the test_678634358 branch 2 times, most recently from 1c8124e to b979c5e Compare September 25, 2024 17:13
@dsharlet
Copy link
Collaborator

Note test failures:

Generated bzl/cmake files do not match the source files.
Please run the following command in your source directory to update them:

    ./tools/update-microkernels.py

It looks like the u4 variant is missing.

BTW, for such kernels that only need very few registers, does it make sense to have the small unroll factors? i.e. should we only have u4 and u8 variants?

@copybara-service copybara-service bot force-pushed the test_678634358 branch 6 times, most recently from cc5d8eb to 54c89dd Compare September 26, 2024 05:47
--
07db838 by Mihai.Olinovici <[email protected]>:

Add RVV f32-f16-vcvt kernels and configs.

--
2f40edd by kaustubh-raste <[email protected]>:

Fix the cmake and bzl file

FUTURE_COPYBARA_INTEGRATE_REVIEW=#7128 from imaginationtech:img_patch30_f32_f16_vcvt 2f40edd
PiperOrigin-RevId: 678634358
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

Successfully merging this pull request may close these issues.

2 participants