[PATCH] D88033: [SVE][CodeGen] Legalisation of integer -> floating point conversions

[Sander de Smalen via Phabricator via llvm-commits]

sdesmalen added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64ISelLowering.cpp:9087-9089
+  if (!VT.getVectorElementType().isFloatingPoint() ||
+      VT.getVectorElementCount() != (OpVT.getVectorElementCount()*2))
+    return SDValue();
----------------
paulwalker-arm wrote:
> I imagine that CONCAT_VECTORS operand counts have been normalise to two by this point but given its restriction that all operands must be the same type I think you can protect against this and your `VT == OpVT*2` requirement using `if (getNumOperands() != 2)`.
> 
> Can the isFloatingPoint check be made part of the assert?
> Can the isFloatingPoint check be made part of the assert?
Could this also work for predicates (e.g. nxv8i1 : nxv8i1)?


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

https://reviews.llvm.org/D88033