From: Evandro Menezes Date: Wed, 26 Jun 2019 15:11:31 +0000 (+0000) Subject: [CodeGen] Improve formatting of jump tables (NFC) X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=6016bbfa462e7a98d0a58697ad33f9f7fbccb273;p=llvm [CodeGen] Improve formatting of jump tables (NFC) Split jump tables into individual lines and fix spacing. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@364436 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/lib/CodeGen/MachineFunction.cpp b/lib/CodeGen/MachineFunction.cpp index c2f1c3c6f62..4394ea29df6 100644 --- a/lib/CodeGen/MachineFunction.cpp +++ b/lib/CodeGen/MachineFunction.cpp @@ -912,9 +912,11 @@ void MachineJumpTableInfo::print(raw_ostream &OS) const { OS << "Jump Tables:\n"; for (unsigned i = 0, e = JumpTables.size(); i != e; ++i) { - OS << printJumpTableEntryReference(i) << ": "; + OS << printJumpTableEntryReference(i) << ':'; for (unsigned j = 0, f = JumpTables[i].MBBs.size(); j != f; ++j) OS << ' ' << printMBBReference(*JumpTables[i].MBBs[j]); + if (i != e) + OS << '\n'; } OS << '\n'; diff --git a/test/CodeGen/AArch64/max-jump-table.ll b/test/CodeGen/AArch64/max-jump-table.ll index e6bdeb46149..431db274325 100644 --- a/test/CodeGen/AArch64/max-jump-table.ll +++ b/test/CodeGen/AArch64/max-jump-table.ll @@ -33,17 +33,17 @@ entry: ; CHECK0-NEXT: %jump-table.0: %bb.1 %bb.2 %bb.3 %bb.4 %bb.5 %bb.6 %bb.7 %bb.8 %bb.9 %bb.10 %bb.11 %bb.12 %bb.13 %bb.14 %bb.15 %bb.16 %bb.17 ; CHECK0-NOT: %jump-table.1: ; CHECK4-NEXT: %jump-table.0: %bb.2 %bb.3 %bb.4 %bb.5 -; CHECK4-SAME: %jump-table.1: %bb.6 %bb.7 %bb.8 %bb.9 -; CHECK4-SAME: %jump-table.2: %bb.10 %bb.11 %bb.12 %bb.13 -; CHECK4-SAME: %jump-table.3: %bb.14 %bb.15 %bb.16 %bb.17 +; CHECK4-NEXT: %jump-table.1: %bb.6 %bb.7 %bb.8 %bb.9 +; CHECK4-NEXT: %jump-table.2: %bb.10 %bb.11 %bb.12 %bb.13 +; CHECK4-NEXT: %jump-table.3: %bb.14 %bb.15 %bb.16 %bb.17 ; CHECK4-NOT: %jump-table.4: ; CHECK8-NEXT: %jump-table.0: %bb.2 %bb.3 %bb.4 %bb.5 %bb.6 %bb.7 %bb.8 %bb.9 -; CHECK8-SAME: %jump-table.1: %bb.10 %bb.11 %bb.12 %bb.13 %bb.14 %bb.15 %bb.16 %bb.17 +; CHECK8-NEXT: %jump-table.1: %bb.10 %bb.11 %bb.12 %bb.13 %bb.14 %bb.15 %bb.16 %bb.17 ; CHECK8-NOT: %jump-table.2: ; CHECK16-NEXT: %jump-table.0: %bb.2 %bb.3 %bb.4 %bb.5 %bb.6 %bb.7 %bb.8 %bb.9 %bb.10 %bb.11 %bb.12 %bb.13 %bb.14 %bb.15 %bb.16 %bb.17 ; CHECK16-NOT: %jump-table.1: ; CHECKM1-NEXT: %jump-table.0: %bb.2 %bb.3 %bb.4 %bb.5 %bb.6 %bb.7 %bb.8 %bb.9 -; CHECKM1-SAME: %jump-table.1: %bb.10 %bb.11 %bb.12 %bb.13 %bb.14 %bb.15 %bb.16 %bb.17 +; CHECKM1-NEXT: %jump-table.1: %bb.10 %bb.11 %bb.12 %bb.13 %bb.14 %bb.15 %bb.16 %bb.17 ; CHECKM1-NOT: %jump-table.2: ; CHECKM3-NEXT: %jump-table.0: %bb.1 %bb.2 %bb.3 %bb.4 %bb.5 %bb.6 %bb.7 %bb.8 %bb.9 %bb.10 %bb.11 %bb.12 %bb.13 %bb.14 %bb.15 %bb.16 %bb.17 ; CHECKM3-NOT: %jump-table.1: @@ -83,17 +83,17 @@ entry: ; CHECK-LABEL: function jt2: ; CHECK-NEXT: Jump Tables: ; CHECK0-NEXT: %jump-table.0: %bb.1 %bb.2 %bb.3 %bb.4 %bb.7 %bb.7 %bb.7 %bb.7 %bb.7 %bb.7 %bb.7 %bb.7 %bb.7 %bb.5 %bb.6{{$}} -; CHECK0-NOT: %jump-table.1 +; CHECK0-NOT: %jump-table.1: ; CHECK4-NEXT: %jump-table.0: %bb.1 %bb.2 %bb.3 %bb.4{{$}} -; CHECK4-NOT: %jump-table.1 +; CHECK4-NOT: %jump-table.1: ; CHECK8-NEXT: %jump-table.0: %bb.1 %bb.2 %bb.3 %bb.4{{$}} -; CHECK8-NOT: %jump-table.1 +; CHECK8-NOT: %jump-table.1: ; CHECK16-NEXT: %jump-table.0: %bb.1 %bb.2 %bb.3 %bb.4 %bb.7 %bb.7 %bb.7 %bb.7 %bb.7 %bb.7 %bb.7 %bb.7 %bb.7 %bb.5 %bb.6{{$}} ; CHECK16-NOT: %jump-table.1: ; CHECKM1-NEXT: %jump-table.0: %bb.1 %bb.2 %bb.3 %bb.4{{$}} -; CHECKM1-NOT: %jump-table.1 +; CHECKM1-NOT: %jump-table.1: ; CHECKM3-NEXT: %jump-table.0: %bb.1 %bb.2 %bb.3 %bb.4 %bb.7 %bb.7 %bb.7 %bb.7 %bb.7 %bb.7 %bb.7 %bb.7 %bb.7 %bb.5 %bb.6{{$}} -; CHECKM3-NOT: %jump-table.1 +; CHECKM3-NOT: %jump-table.1: ; CHECK-DAG: End machine code for function jt2. bb1: tail call void @ext(i32 6, i32 1) br label %return @@ -127,21 +127,21 @@ entry: ; CHECK-LABEL: function jt3: ; CHECK-NEXT: Jump Tables: ; CHECK0-NEXT: %jump-table.0: %bb.1 %bb.2 %bb.3 %bb.4 %bb.13 %bb.13 %bb.13 %bb.13 %bb.13 %bb.13 %bb.13 %bb.13 %bb.13 %bb.5 %bb.6 %bb.7 %bb.8 %bb.13 %bb.9 %bb.10 %bb.13 %bb.11 %bb.12 -; CHECK0-NOT: %jump-table.1 +; CHECK0-NOT: %jump-table.1: ; CHECK4-NEXT: %jump-table.0: %bb.1 %bb.2 %bb.3 %bb.4 -; CHECK4-SAME: %jump-table.1: %bb.5 %bb.6 %bb.7 %bb.8 -; CHECK4-NOT: %jump-table.2 +; CHECK4-NEXT: %jump-table.1: %bb.5 %bb.6 %bb.7 %bb.8 +; CHECK4-NOT: %jump-table.2: ; CHECK8-NEXT: %jump-table.0: %bb.1 %bb.2 %bb.3 %bb.4 -; CHECK8-SAME: %jump-table.1: %bb.5 %bb.6 %bb.7 %bb.8 %bb.13 %bb.9 %bb.10 -; CHECK8-NOT: %jump-table.2 +; CHECK8-NEXT: %jump-table.1: %bb.5 %bb.6 %bb.7 %bb.8 %bb.13 %bb.9 %bb.10 +; CHECK8-NOT: %jump-table.2: ; CHECK16-NEXT: %jump-table.0: %bb.1 %bb.2 %bb.3 %bb.4 %bb.13 %bb.13 %bb.13 %bb.13 %bb.13 %bb.13 %bb.13 %bb.13 %bb.13 %bb.5 %bb.6 %bb.7 -; CHECK16-SAME: %jump-table.1: %bb.8 %bb.13 %bb.9 %bb.10 %bb.13 %bb.11 %bb.12 -; CHECK16-NOT: %jump-table.2 +; CHECK16-NEXT: %jump-table.1: %bb.8 %bb.13 %bb.9 %bb.10 %bb.13 %bb.11 %bb.12 +; CHECK16-NOT: %jump-table.2: ; CHECKM1-NEXT: %jump-table.0: %bb.1 %bb.2 %bb.3 %bb.4 -; CHECKM1-SAME: %jump-table.1: %bb.5 %bb.6 %bb.7 %bb.8 %bb.13 %bb.9 %bb.10 -; CHECKM1-NOT: %jump-table.2 +; CHECKM1-NEXT: %jump-table.1: %bb.5 %bb.6 %bb.7 %bb.8 %bb.13 %bb.9 %bb.10 +; CHECKM1-NOT: %jump-table.2: ; CHECKM3-NEXT: %jump-table.0: %bb.1 %bb.2 %bb.3 %bb.4 %bb.13 %bb.13 %bb.13 %bb.13 %bb.13 %bb.13 %bb.13 %bb.13 %bb.13 %bb.5 %bb.6 %bb.7 %bb.8 %bb.13 %bb.9 %bb.10 -; CHECKM3-NOT: %jump-table.1 +; CHECKM3-NOT: %jump-table.1: ; CHECK-DAG: End machine code for function jt3. bb1: tail call void @ext(i32 1, i32 12) br label %return @@ -182,21 +182,21 @@ entry: ; CHECK-LABEL: function jt4: ; CHECK-NEXT: Jump Tables: ; CHECK0-NEXT: %jump-table.0: %bb.1 %bb.2 %bb.3 %bb.4 %bb.13 %bb.13 %bb.13 %bb.13 %bb.13 %bb.13 %bb.13 %bb.13 %bb.13 %bb.5 %bb.6 %bb.7 %bb.8 %bb.13 %bb.9 %bb.10 %bb.13 %bb.11 %bb.12 -; CHECK0-NOT: %jump-table.1 +; CHECK0-NOT: %jump-table.1: ; CHECK4-NEXT: %jump-table.0: %bb.1 %bb.2 %bb.3 %bb.4 -; CHECK4-SAME: %jump-table.1: %bb.5 %bb.6 %bb.7 %bb.8 -; CHECK4-NOT: %jump-table.2 +; CHECK4-NEXT: %jump-table.1: %bb.5 %bb.6 %bb.7 %bb.8 +; CHECK4-NOT: %jump-table.2: ; CHECK8-NEXT: %jump-table.0: %bb.1 %bb.2 %bb.3 %bb.4 -; CHECK8-SAME: %jump-table.1: %bb.5 %bb.6 %bb.7 %bb.8 %bb.13 %bb.9 %bb.10 -; CHECK8-NOT: %jump-table.2 +; CHECK8-NEXT: %jump-table.1: %bb.5 %bb.6 %bb.7 %bb.8 %bb.13 %bb.9 %bb.10 +; CHECK8-NOT: %jump-table.2: ; CHECK16-NEXT: %jump-table.0: %bb.1 %bb.2 %bb.3 %bb.4 %bb.13 %bb.13 %bb.13 %bb.13 %bb.13 %bb.13 %bb.13 %bb.13 %bb.13 %bb.5 %bb.6 %bb.7 -; CHECK16-SAME: %jump-table.1: %bb.8 %bb.13 %bb.9 %bb.10 %bb.13 %bb.11 %bb.12 -; CHECK16-NOT: %jump-table.2 +; CHECK16-NEXT: %jump-table.1: %bb.8 %bb.13 %bb.9 %bb.10 %bb.13 %bb.11 %bb.12 +; CHECK16-NOT: %jump-table.2: ; CHECKM1-NEXT: %jump-table.0: %bb.1 %bb.2 %bb.3 %bb.4 -; CHECKM1-SAME: %jump-table.1: %bb.5 %bb.6 %bb.7 %bb.8 %bb.13 %bb.9 %bb.10 -; CHECKM1-NOT: %jump-table.2 +; CHECKM1-NEXT: %jump-table.1: %bb.5 %bb.6 %bb.7 %bb.8 %bb.13 %bb.9 %bb.10 +; CHECKM1-NOT: %jump-table.2: ; CHECKM3-NEXT: %jump-table.0: %bb.1 %bb.2 %bb.3 %bb.4 %bb.13 %bb.13 %bb.13 %bb.13 %bb.13 %bb.13 %bb.13 %bb.13 %bb.13 %bb.5 %bb.6 %bb.7 %bb.8 %bb.13 %bb.9 %bb.10 -; CHECKM3-NOT: %jump-table.1 +; CHECKM3-NOT: %jump-table.1: ; CHECK-DAG: End machine code for function jt4. bb1: tail call void @ext(i32 1, i32 12) br label %return