[PATCH] D19133: Fix typing on generated LXV2DX/STXV2DX instructions

Nirav Dave via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 15 07:04:57 PDT 2016


niravd added inline comments.

================
Comment at: lib/Target/PowerPC/PPCISelLowering.cpp:10368
@@ +10367,3 @@
+    DCI.AddToWorklist(N.getNode());
+    return DAG.getNode(ISD::MERGE_VALUES, dl, DAG.getVTList(VecTy, MVT::Other),
+                       N, Swap.getValue(1));
----------------
nemanjai wrote:
> Pardon my ignorance here, I just don't follow why we're adding this ISD::MERGE_VALUES node here.
Merge Value is an identity node that returns its inputs and is used to work around the fact that node replacements do not always result in a final node with constructing all the values in the correct way. Here replacing the load with a different load and a swap node was fine because they returned the same shape output (value, chain), but with the bitcast we need to make a new ephemeral node to hold the bitcast result and the swap node's chain value. 


http://reviews.llvm.org/D19133





More information about the llvm-commits mailing list