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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to