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

xRaddextexp - replaced yaml files with header table #7410

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

nitheshsrikanth-mcw
Copy link

No description provided.

@nitheshsrikanth-mcw nitheshsrikanth-mcw force-pushed the xnn_raddextexp branch 2 times, most recently from de662aa to e853c77 Compare November 4, 2024 13:37
Copy link
Collaborator

@dsharlet dsharlet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same comments as #7408

@nitheshsrikanth-mcw nitheshsrikanth-mcw force-pushed the xnn_raddextexp branch 7 times, most recently from 2dfbc74 to 1e33a73 Compare November 6, 2024 12:34
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Delete this file

@dsharlet
Copy link
Collaborator

Update kernel declarations?

copybara-service bot pushed a commit that referenced this pull request Nov 12, 2024
--
e853c77 by nithesh <[email protected]>:

Raddextexp - replaced yaml files with header table
Replaced yaml files with header table for Raddextexp op

--
1e33a73 by nithesh <[email protected]>:

Addressed review comments

--
d03ce4f by nithesh <[email protected]>:

Addressed review comments

FUTURE_COPYBARA_INTEGRATE_REVIEW=#7410 from nitheshsrikanth-mcw:xnn_raddextexp 8af7373
PiperOrigin-RevId: 695826022
copybara-service bot pushed a commit that referenced this pull request Nov 12, 2024
--
e853c77 by nithesh <[email protected]>:

Raddextexp - replaced yaml files with header table
Replaced yaml files with header table for Raddextexp op

--
1e33a73 by nithesh <[email protected]>:

Addressed review comments

--
d03ce4f by nithesh <[email protected]>:

Addressed review comments

FUTURE_COPYBARA_INTEGRATE_REVIEW=#7410 from nitheshsrikanth-mcw:xnn_raddextexp 8af7373
PiperOrigin-RevId: 695826022
copybara-service bot pushed a commit that referenced this pull request Nov 12, 2024
--
e853c77 by nithesh <[email protected]>:

Raddextexp - replaced yaml files with header table
Replaced yaml files with header table for Raddextexp op

--
1e33a73 by nithesh <[email protected]>:

Addressed review comments

--
d03ce4f by nithesh <[email protected]>:

Addressed review comments

FUTURE_COPYBARA_INTEGRATE_REVIEW=#7410 from nitheshsrikanth-mcw:xnn_raddextexp 8af7373
PiperOrigin-RevId: 695826022
#define XNN_TEST_RADDEXTEXP_ELEMENT_DIV(ukernel, arch_flags, element_tile, ...) \
TEST(ukernel, element_gt) \
{ \
for (size_t element_size = element_tile * 2; element_size < element_tile * 10; element_size += element_tile) { \
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TEST_REQUIRES_ARCH_FLAGS(arch_flags)?

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for pointing out the missing TEST_REQUIRES_ARCH_FLAGS(arch_flags). The issue with the illegal instruction for AVX-512 has now been resolved and now the tests are passing.

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.

3 participants