[PATCH] D31946: [legalize-types] Make softening result use a single map for replacements.

[Chih-Hung Hsieh via Phabricator via llvm-commits]

chh added inline comments.


================
Comment at: lib/CodeGen/SelectionDAG/LegalizeFloatTypes.cpp:115
 
-  // If R is null, the sub-method took care of registering the result.
-  if (R.getNode()) {
+  if (R.getNode() && R.getNode() != N) {
     SetSoftenedFloat(SDValue(N, ResNo), R);
----------------
Was there a problem to fix with the new condition (R.getNode() != N)?
I vaguely remember that x86_64 f128 type implementation might depend on the ability to set a note as softened without changing the node.



================
Comment at: lib/CodeGen/SelectionDAG/LegalizeFloatTypes.cpp:813
     case ISD::CopyFromReg:
-    case ISD::CopyToReg:
     case ISD::FABS:
----------------
Why not skipping CopyToReg now?
If it missed some case, could you add a test case?



https://reviews.llvm.org/D31946