Skip to content

Conversation

@stevesuzuki-arm
Copy link
Contributor

Use llvm.vector.(de)interleaveN intrinsic to lower vector shuffle operation if it matches deinterleave or interleave pattern. Note that the supported factor N is limited by llvm-ir spec and also target backend.

Use llvm.vector.(de)interleaveN intrinsic to lower vector shuffle
operation if it matches deinterleave or interleave pattern.
Note that the supported factor N is limited by llvm-ir spec
and also target backend.
Take vscale into accout in slicing small vectors.
Copy link
Member

@alexreinking alexreinking left a comment

Choose a reason for hiding this comment

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

nit: please use the concat_strings helper since this gives us an easy thing to search for when eventually modernizing to C++20 and have std::format available.

Copy link
Member

@alexreinking alexreinking left a comment

Choose a reason for hiding this comment

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

Looks great, thanks!

@stevesuzuki-arm
Copy link
Contributor Author

Just want to follow up, I suppose the bot job failures are not related to this commit. Let me know if you’d like me to update anything before it can be merged. I have some other items depending on this one. Thanks!

@alexreinking alexreinking merged commit 2b019da into halide:main Dec 11, 2025
15 of 19 checks passed
@alexreinking
Copy link
Member

@stevesuzuki-arm -- thanks for the nudge. Sorry for the delay.

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