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

[Florian Hahn via Phabricator via llvm-commits]

fhahn added a comment.

Ping. Do you think the updated test cases are sufficient or would it help to provide a C example? I have one lying around somewhere, but the test case is quite ugly.

The patch only changes the `Order` value for debug intrinsics, so that the `Order` of debug intrinsics is equal to the previous non-debug instruction. The order of non-debug instructions is not affected by the change (as far as I can tell), because all non-debug instruction still have distinct `Order` values.

The order of debug instructions between two non-debug instructions could change though, because their order now relies on the order returned by DAG->GetDbgValues(N), but this shouldn't be a problem, right?


https://reviews.llvm.org/D25318