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

Replace vscaleexpminusmax yaml with header #7409

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

Conversation

RahulSundarMCW
Copy link

No description provided.

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

@RahulSundarMCW RahulSundarMCW force-pushed the vscaleexpminusmax branch 2 times, most recently from cbdc5f8 to 2543af9 Compare November 5, 2024 09:15
@RahulSundarMCW
Copy link
Author

Address the review comments and pushed the commit.

@RahulSundarMCW RahulSundarMCW force-pushed the vscaleexpminusmax branch 2 times, most recently from 2d6ecc5 to 773892e Compare November 6, 2024 13:47
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

@RahulSundarMCW RahulSundarMCW force-pushed the vscaleexpminusmax branch 4 times, most recently from c520980 to aaf41aa Compare November 12, 2024 12:16
#endif

#if XNN_ARCH_X86 || XNN_ARCH_X86_64
XNN_UKERNEL_WITH_PARAMS(xnn_arch_x86_avx2, xnn_f32_vscaleexpminusmax_ukernel__avx2_p5_u8, 8, float, struct xnn_f32_default_params, xnn_f32_vscaleexpminusmax_ukernel_fn)
Copy link
Collaborator

Choose a reason for hiding this comment

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

I don't understand the last parameter of these, xnn_f32_vscaleexpminusmax_ukernel_fn? Why is this here?

@RahulSundarMCW RahulSundarMCW force-pushed the vscaleexpminusmax branch 3 times, most recently from 768ce6c to 97c65fa Compare November 18, 2024 10:35
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