[PATCH] D133894: [AArch64][SME] Add intrinsics for enabling/disabling ZA.

[Amara Emerson via Phabricator via llvm-commits]

aemerson added inline comments.


================
Comment at: llvm/test/CodeGen/AArch64/sme-toggle-pstateza.ll:10
+; CHECK-NEXT:    ret
+  call void @llvm.aarch64.sme.start.pstateza()
+  call void @llvm.aarch64.sme.stop.pstateza()
----------------
paulwalker-arm wrote:
> Sorry for bikeshedding and feel free to ignore but what about `llvm.aarch64.sme.za.enable`/`llvm.aarch64.sme.za.disable`?
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.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D133894/new/

https://reviews.llvm.org/D133894