[PATCH] D158950: [X86][BF16] Lower FP_EXTEND for vector types under AVX512BF16

[Phoebe Wang via Phabricator via llvm-commits]

pengfei added inline comments.


================
Comment at: llvm/lib/Target/X86/X86ISelLowering.cpp:21208
+                     DAG.getConstant(0, DL, IntVT));
+    In = DAG.getVectorShuffle(NVT, DL, In, DAG.getUNDEF(NVT), Mask);
+    return DAG.getBitcast(VT, In);
----------------
RKSimon wrote:
> Would it be better to do a shl(zext(In,vXi32),8) here?
Yes, thanks!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D158950