[PATCH] D25318: [DAG] Don't increase SDNodeOrder for dbg.value/declare.

Mikael Holmén via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 6 05:55:41 PDT 2016


uabelho created this revision.
uabelho added a reviewer: bogner.
uabelho added a subscriber: llvm-commits.
Herald added a subscriber: MatzeB.

The SDNodeOrder is saved in the IROrder field in the SDNode, and this
field may affects scheduling. Thus, letting dbg.value/declare increase
the order numbers may in turn affect scheduling.

      

Because of this change we also need to update the code deciding when
dbg values should be output, in ScheduleDAGSDNodes.cpp/ProcessSDDbgValues.

      

Dbg values now have the same order as the SDNode they are connected to,
not the following orders.


https://reviews.llvm.org/D25318

Files:
  lib/CodeGen/SelectionDAG/ScheduleDAGSDNodes.cpp
  lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp
  test/DebugInfo/Generic/selectiondag-order.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D25318.73776.patch
Type: text/x-patch
Size: 3859 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20161006/afe41a52/attachment.bin>


More information about the llvm-commits mailing list