Page MenuHomePhabricator

[AArch64][SME] Add intrinsics for enabling/disabling ZA.
ClosedPublic

Authored by sdesmalen on Sep 14 2022, 2:12 PM.

Details

Summary

This adds the intrinsics:

  • void @llvm.aarch64.sme.start.pstateza() -> smstart za
  • void @llvm.aarch64.sme.stop.pstateza() -> smstop za

Diff Detail

Event Timeline

sdesmalen created this revision.Sep 14 2022, 2:12 PM
Herald added a project: Restricted Project. · View Herald TranscriptSep 14 2022, 2:12 PM
sdesmalen requested review of this revision.Sep 14 2022, 2:12 PM
Herald added a project: Restricted Project. · View Herald TranscriptSep 14 2022, 2:12 PM
This revision is now accepted and ready to land.Sep 15 2022, 12:57 AM
paulwalker-arm added inline comments.Sep 15 2022, 2:41 AM
llvm/test/CodeGen/AArch64/sme-toggle-pstateza.ll
11

Sorry for bikeshedding and feel free to ignore but what about llvm.aarch64.sme.za.enable/llvm.aarch64.sme.za.disable?

aemerson added inline comments.Sep 15 2022, 3:00 AM
llvm/test/CodeGen/AArch64/sme-toggle-pstateza.ll
11

Now that you bring it up, adding PSTATE to the intrinsic names seem a bit too low level, but I don't care enough to object.

Matt added a subscriber: Matt.Sep 16 2022, 8:31 PM
This revision was landed with ongoing or failed builds.Sep 17 2022, 9:42 AM
This revision was automatically updated to reflect the committed changes.
sdesmalen marked 2 inline comments as done.Sep 17 2022, 9:43 AM