[PATCH] D78239: [SveEmitter] Add builtins for FP conversions

[Sander de Smalen via Phabricator via cfe-commits]

sdesmalen marked an inline comment as done.
sdesmalen added a comment.

In D78239#1987859 <https://reviews.llvm.org/D78239#1987859>, @efriedma wrote:

> I think the predicate type is something we should fix.   Even if it's not causing problems now, it seems like the sort of thing that will bite us later.


Yes I agree, I've implemented this in D78402 <https://reviews.llvm.org/D78402>



================
Comment at: clang/utils/TableGen/SveEmitter.cpp:1035
+  OS << "#define svcvtxnt_f32_x     svcvtxnt_f32_m\n";
+  OS << "#define svcvtxnt_f32_f64_x svcvtxnt_f32_f64_m\n\n";
+
----------------
efriedma wrote:
> Not worth implementing these separately?  I guess I can see why it wouldn't be profitable.
We currently don't really optimise the _x (MergeAny) case yet for other intrinsics, so addressing this seems more trouble than it's worth, specifically because the merging with `undef` for only the top/odd lanes would not be trivial to represent. 


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

https://reviews.llvm.org/D78239