[PATCH] D82501: [sve][acle] Add reinterpret intrinsics for brain float.

David Sherwood via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 26 00:31:43 PDT 2020


david-arm accepted this revision.
david-arm added a comment.
This revision is now accepted and ready to land.

Can you remove the duplicate tests before submitting? Otherwise LGTM!



================
Comment at: llvm/test/CodeGen/AArch64/sve-bitcast-bfloat.ll:8
+
+define <vscale x 16 x i8> @bitcast_bfloat_to_i8(<vscale x 8 x bfloat> %v) {
+; CHECK-LABEL: bitcast_bfloat_to_i8:
----------------
Aren't these tests all duplicates of ones in llvm/test/CodeGen/AArch64/sve-bitcast.ll? Looks like you can remove this file completely.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82501





More information about the llvm-commits mailing list