Commit ca387dbd authored by mstarzinger's avatar mstarzinger Committed by Commit bot

[compiler] Unify printing of schedule block ids.

We use the pattern "B23" to refer to RPO numbers and "id:42" when
referring to unordered block ids. This unifies printing functions to
follow that pattern throughout the system.

R=danno@chromium.org

Review URL: https://codereview.chromium.org/1824313002

Cr-Commit-Position: refs/heads/master@{#35022}
parent 7e4ce043
...@@ -392,7 +392,7 @@ std::ostream& operator<<(std::ostream& os, const Schedule& s) { ...@@ -392,7 +392,7 @@ std::ostream& operator<<(std::ostream& os, const Schedule& s) {
for (BasicBlock* block : for (BasicBlock* block :
((s.RpoBlockCount() == 0) ? *s.all_blocks() : *s.rpo_order())) { ((s.RpoBlockCount() == 0) ? *s.all_blocks() : *s.rpo_order())) {
if (block->rpo_number() == -1) { if (block->rpo_number() == -1) {
os << "--- BLOCK B" << block->id().ToInt() << " (block id)"; os << "--- BLOCK id:" << block->id().ToInt();
} else { } else {
os << "--- BLOCK B" << block->rpo_number(); os << "--- BLOCK B" << block->rpo_number();
} }
...@@ -403,7 +403,7 @@ std::ostream& operator<<(std::ostream& os, const Schedule& s) { ...@@ -403,7 +403,7 @@ std::ostream& operator<<(std::ostream& os, const Schedule& s) {
if (comma) os << ", "; if (comma) os << ", ";
comma = true; comma = true;
if (predecessor->rpo_number() == -1) { if (predecessor->rpo_number() == -1) {
os << "B" << predecessor->id().ToInt(); os << "id:" << predecessor->id().ToInt();
} else { } else {
os << "B" << predecessor->rpo_number(); os << "B" << predecessor->rpo_number();
} }
...@@ -432,7 +432,7 @@ std::ostream& operator<<(std::ostream& os, const Schedule& s) { ...@@ -432,7 +432,7 @@ std::ostream& operator<<(std::ostream& os, const Schedule& s) {
if (comma) os << ", "; if (comma) os << ", ";
comma = true; comma = true;
if (successor->rpo_number() == -1) { if (successor->rpo_number() == -1) {
os << "B" << successor->id().ToInt(); os << "id:" << successor->id().ToInt();
} else { } else {
os << "B" << successor->rpo_number(); os << "B" << successor->rpo_number();
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment