-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Add ARM encoding group IF_SVE_CI_3A #97808
Conversation
Tagging subscribers to this area: @JulieLeeMSFT, @jakobbotsch Issue DetailsAdds support for emitting trn[1/2], zip[1/2] and uzp[1/2] instructions. Matching capstone:
Contribute towards #94549.
|
@dotnet-policy-service agree company="Arm" |
@a74nh @kunalspathak @dotnet/arm64-contrib |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This LGTM. Thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thank you!
@@ -1089,6 +1089,14 @@ void emitter::emitInsSanityCheck(instrDesc* id) | |||
assert(isValidUimm4From1(emitGetInsSC(id))); | |||
break; | |||
|
|||
case IF_SVE_CI_3A: // ........xx..MMMM .......NNNN.DDDD -- SVE permute predicate elements | |||
elemsize = id->idOpSize(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Doesn't have to be done for this PR, but we should come back and clean up all these unused elemsize
locals (and maybe update the tool to not emit them @kunalspathak).
Adds support for emitting trn[1/2], zip[1/2] and uzp[1/2] instructions.
Matching capstone:
Contribute towards #94549.