[PATCH] D24435: Delete more dead code in SelectionDAG (NFC)

[Simon Pilgrim via llvm-commits]

RKSimon added a comment.

The SelectNodeTo / getMachineNode helpers can probably go as they're very easy to re-add if we ever get a use case.


================
Comment at: include/llvm/CodeGen/SelectionDAG.h:561
@@ -560,3 +560,1 @@
-  SDValue getTargetIndex(int Index, EVT VT, int64_t Offset = 0,
-                         unsigned char TargetFlags = 0);
   // When generating a branch to a BB, we don't in general know enough
----------------
This should stay while TargetIndexSDNode exists.

However, TargetIndexSDNode doesn't appear to be used so maybe start a conversion on llvm-dev about removing it and if successful then propose a separate patch?

================
Comment at: lib/CodeGen/SelectionDAG/SelectionDAGBuilder.h:656
@@ -655,3 +655,1 @@
 
-  unsigned getSDNodeOrder() const { return SDNodeOrder; }
-
----------------
Do you know the history behind this and DanglingDebugInfo::getSDNodeOrder()?


https://reviews.llvm.org/D24435