[PATCH] D88683: [SVE] Lower fixed length vector fneg and fsqrt operations.

Cameron McInally via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 1 15:09:43 PDT 2020


cameron.mcinally accepted this revision.
cameron.mcinally added a subscriber: arsenm.
cameron.mcinally added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: llvm/test/CodeGen/AArch64/sve-fp.ll:333
+  %res = fneg <vscale x 8 x half> %a
+  ret <vscale x 8 x half> %res
 }
----------------
Nit: These changes should probably go in with a standalone patch, but seem harmless enough to be included with this one.

@arsenm Also, there's an ongoing project to stop FMUL(-1.0, X)->FNEG(X) transformations when denormals are present, so this is a good change.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88683



More information about the llvm-commits mailing list