Reviewers: Benedikt Meurer,
Description:
[turbofan] Skip printing of empty gap moves.
[email protected]
BUG=
Please review this at https://codereview.chromium.org/812983002/
Base URL: https://chromium.googlesource.com/v8/v8.git@master
Affected files (+11, -9 lines):
M src/compiler/instruction.cc
Index: src/compiler/instruction.cc
diff --git a/src/compiler/instruction.cc b/src/compiler/instruction.cc
index
ba72eda2ec73ff58018c2b3a83ab45f301e106ed..962548a48e60ae8deb49bab63909a5de7baa7503
100644
--- a/src/compiler/instruction.cc
+++ b/src/compiler/instruction.cc
@@ -287,16 +287,18 @@ std::ostream& operator<<(std::ostream& os,
if (instr.IsGapMoves()) {
const GapInstruction* gap = GapInstruction::cast(&instr);
- os << "gap ";
- for (int i = GapInstruction::FIRST_INNER_POSITION;
- i <= GapInstruction::LAST_INNER_POSITION; i++) {
- os << "(";
- if (gap->parallel_moves_[i] != NULL) {
- PrintableParallelMove ppm = {printable.register_configuration_,
- gap->parallel_moves_[i]};
- os << ppm;
+ if (!gap->IsRedundant()) {
+ os << "gap ";
+ for (int i = GapInstruction::FIRST_INNER_POSITION;
+ i <= GapInstruction::LAST_INNER_POSITION; i++) {
+ os << "(";
+ if (gap->parallel_moves_[i] != NULL) {
+ PrintableParallelMove ppm = {printable.register_configuration_,
+ gap->parallel_moves_[i]};
+ os << ppm;
+ }
+ os << ") ";
}
- os << ") ";
}
} else if (instr.IsSourcePosition()) {
const SourcePositionInstruction* pos =
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
