================
@@ -6597,7 +6659,7 @@ static SDValue convertFromF16(SDValue Op, SDLoc DL, 
SelectionDAG &DAG) {
   SDValue BCast = DAG.getNode(ISD::BITCAST, DL, MVT::i64, In64);
   SDValue Shft = DAG.getNode(ISD::SRL, DL, MVT::i64, BCast,
                              DAG.getConstant(48, DL, MVT::i32));
-  return Shft;
+  return DAG.getZExtOrTrunc(Shft, DL, MVT::i32);
----------------
uweigand wrote:

I think it would be preferable to do this in the caller, to keep the interfaces 
of `convertToFp16` and `convertFromFp16` symmetrical.

https://github.com/llvm/llvm-project/pull/171066
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to