[PATCH] D80928: [BFloat] Add convert/copy instrinsic support

Mikhail Maltsev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jun 22 06:57:45 PDT 2020


miyuki marked 5 inline comments as done.
miyuki added inline comments.


================
Comment at: clang/utils/TableGen/NeonEmitter.cpp:1066
 
   if (Name == "vcvt_f16_f32" || Name == "vcvt_f32_f16" ||
+      Name == "vcvt_f32_f64" || Name == "vcvt_f64_f32" ||
----------------
LukeGeeson wrote:
> I always wondered why we need special treatment for cvt intrinsics here, is there a better place to put this?
If we were to refactor this, it should be done in a separate patch.


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

https://reviews.llvm.org/D80928





More information about the cfe-commits mailing list