diff options
| author | Florian Hahn <florian.hahn@arm.com> | 2018-06-20 17:42:01 +0000 |
|---|---|---|
| committer | Florian Hahn <florian.hahn@arm.com> | 2018-06-20 17:42:01 +0000 |
| commit | 5ac2629823d689309ba000ab624ab667b9f123dd (patch) | |
| tree | 458dad826771a040a05268f1e700ba70a8cb16d6 /llvm/utils/git-svn/git-svnup | |
| parent | 0583d7a56cb5f14ce57183d13269c13390acfada (diff) | |
| download | bcm5719-llvm-5ac2629823d689309ba000ab624ab667b9f123dd.tar.gz bcm5719-llvm-5ac2629823d689309ba000ab624ab667b9f123dd.zip | |
[PredicateInfo] Order instructions in different BBs by DFSNumIn.
Using OrderedInstructions::dominates as comparator for instructions in
BBs without dominance relation can cause a non-deterministic order
between such instructions. That in turn can cause us to materialize
copies in a non-deterministic order. While this does not effect
correctness, it causes some minor non-determinism in the final generated
code, because values have slightly different labels.
Without this patch, running -print-predicateinfo on a reasonably large
module produces slightly different output on each run.
This patch uses the dominator trees DFSInNum to order instruction from
different BBs, which should enforce a deterministic ordering and
guarantee that dominated instructions come after the instructions that
dominate them.
Reviewers: dberlin, efriedma, davide
Reviewed By: efriedma
Differential Revision: https://reviews.llvm.org/D48230
llvm-svn: 335150
Diffstat (limited to 'llvm/utils/git-svn/git-svnup')
0 files changed, 0 insertions, 0 deletions

