Skip to content

Port Shuffle to ISLE (AArch64)#4596

Merged
cfallin merged 2 commits intobytecodealliance:mainfrom
dheaton-arm:isle-shuffle
Aug 4, 2022
Merged

Port Shuffle to ISLE (AArch64)#4596
cfallin merged 2 commits intobytecodealliance:mainfrom
dheaton-arm:isle-shuffle

Conversation

@dheaton-arm
Copy link
Copy Markdown
Contributor

Ported the existing implementation of Shuffle for AArch64 to ISLE.

Copyright (c) 2022 Arm Limited

@github-actions github-actions bot added cranelift Issues related to the Cranelift code generator cranelift:area:aarch64 Issues related to AArch64 backend. labels Aug 3, 2022
Copy link
Copy Markdown
Member

@cfallin cfallin left a comment

Choose a reason for hiding this comment

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

Thanks! One potential cleanup nit below but otherwise looks good.

The hardcoded register bit is slightly unfortunate but we'd have to come up with some ugly regalloc hacks to do something better here ("adjacent register" constraint) and it's pre-existing, and this is a direct port of what was there already, so nothing wrong with this PR at all. I'll think a bit on this though...

Ported the existing implementation of `Shuffle` for AArch64 to ISLE.

Copyright (c) 2022 Arm Limited
Copyright (c) 2022 Arm Limited
@cfallin cfallin merged commit 12a9705 into bytecodealliance:main Aug 4, 2022
@dheaton-arm dheaton-arm deleted the isle-shuffle branch August 4, 2022 15:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cranelift:area:aarch64 Issues related to AArch64 backend. cranelift Issues related to the Cranelift code generator

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants