[PATCH] D133433: [AArch64]: Force generating code compatible to streaming mode

Paul Walker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 14 08:32:07 PDT 2022


paulwalker-arm accepted this revision.
paulwalker-arm added a comment.

As discussed offline there's more we can do here to improve code quality but this'll come as you increase ISD node coverage.  I've one minor issue but otherwise this looks good to me.



================
Comment at: llvm/lib/Target/AArch64/AArch64Subtarget.cpp:441
+  if (ForceStreamingCompatibleSVE) {
+    assert(hasSVE() && "Expected SVE to be available");
+    return hasSVE() || hasSME();
----------------
I guess this should match the code below, although I'm not quite sure why the assert it needed.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D133433



More information about the llvm-commits mailing list