From 75f55d559a87d669e5748a3a858e281e426013d6 Mon Sep 17 00:00:00 2001
From: Dan Gohman <dan433584@gmail.com>
Date: Wed, 18 Nov 2015 16:25:38 +0000
Subject: [PATCH] [WebAssembly] Add some spaces to the assembly output to
 vertically align operands.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@253468 91177308-0d34-0410-b5e6-96231b3b80d8
---
 .../WebAssembly/WebAssemblyInstrFloat.td      | 28 +++++++++----------
 .../WebAssembly/WebAssemblyInstrInteger.td    | 24 ++++++++--------
 2 files changed, 27 insertions(+), 25 deletions(-)

diff --git a/lib/Target/WebAssembly/WebAssemblyInstrFloat.td b/lib/Target/WebAssembly/WebAssemblyInstrFloat.td
index 59d2e6b474e..33efbb350cb 100644
--- a/lib/Target/WebAssembly/WebAssemblyInstrFloat.td
+++ b/lib/Target/WebAssembly/WebAssemblyInstrFloat.td
@@ -12,18 +12,18 @@
 ///
 //===----------------------------------------------------------------------===//
 
-defm ADD : BinaryFP<fadd, "add">;
-defm SUB : BinaryFP<fsub, "sub">;
-defm MUL : BinaryFP<fmul, "mul">;
-defm DIV : BinaryFP<fdiv, "div">;
+defm ADD : BinaryFP<fadd, "add ">;
+defm SUB : BinaryFP<fsub, "sub ">;
+defm MUL : BinaryFP<fmul, "mul ">;
+defm DIV : BinaryFP<fdiv, "div ">;
 defm SQRT : UnaryFP<fsqrt, "sqrt">;
 
-defm ABS : UnaryFP<fabs, "abs">;
-defm NEG : UnaryFP<fneg, "neg">;
+defm ABS : UnaryFP<fabs, "abs ">;
+defm NEG : UnaryFP<fneg, "neg ">;
 defm COPYSIGN : BinaryFP<fcopysign, "copysign">;
 
-defm MIN : BinaryFP<fminnan, "min">;
-defm MAX : BinaryFP<fmaxnan, "max">;
+defm MIN : BinaryFP<fminnan, "min ">;
+defm MAX : BinaryFP<fmaxnan, "max ">;
 
 defm CEIL : UnaryFP<fceil, "ceil">;
 defm FLOOR : UnaryFP<ffloor, "floor">;
@@ -34,12 +34,12 @@ defm NEAREST : UnaryFP<fnearbyint, "nearest">;
 def : Pat<(frint f32:$src), (NEAREST_F32 f32:$src)>;
 def : Pat<(frint f64:$src), (NEAREST_F64 f64:$src)>;
 
-defm EQ : ComparisonFP<SETOEQ, "eq">;
-defm NE : ComparisonFP<SETUNE, "ne">;
-defm LT : ComparisonFP<SETOLT, "lt">;
-defm LE : ComparisonFP<SETOLE, "le">;
-defm GT : ComparisonFP<SETOGT, "gt">;
-defm GE : ComparisonFP<SETOGE, "ge">;
+defm EQ : ComparisonFP<SETOEQ, "eq  ">;
+defm NE : ComparisonFP<SETUNE, "ne  ">;
+defm LT : ComparisonFP<SETOLT, "lt  ">;
+defm LE : ComparisonFP<SETOLE, "le  ">;
+defm GT : ComparisonFP<SETOGT, "gt  ">;
+defm GE : ComparisonFP<SETOGE, "ge  ">;
 
 // Don't care floating-point comparisons, supported via other comparisons.
 def : Pat<(seteq f32:$lhs, f32:$rhs), (EQ_F32 f32:$lhs, f32:$rhs)>;
diff --git a/lib/Target/WebAssembly/WebAssemblyInstrInteger.td b/lib/Target/WebAssembly/WebAssemblyInstrInteger.td
index 65a9f0c19b4..1d634dfae08 100644
--- a/lib/Target/WebAssembly/WebAssemblyInstrInteger.td
+++ b/lib/Target/WebAssembly/WebAssemblyInstrInteger.td
@@ -12,22 +12,24 @@
 ///
 //===----------------------------------------------------------------------===//
 
-defm ADD : BinaryInt<add, "add">;
-defm SUB : BinaryInt<sub, "sub">;
-defm MUL : BinaryInt<mul, "mul">;
+// The spaces after the names are for aesthetic purposes only, to make
+// operands line up vertically after tab expansion.
+defm ADD : BinaryInt<add, "add ">;
+defm SUB : BinaryInt<sub, "sub ">;
+defm MUL : BinaryInt<mul, "mul ">;
 defm DIV_S : BinaryInt<sdiv, "div_s">;
 defm DIV_U : BinaryInt<udiv, "div_u">;
 defm REM_S : BinaryInt<srem, "rem_s">;
 defm REM_U : BinaryInt<urem, "rem_u">;
-defm AND : BinaryInt<and, "and">;
-defm OR : BinaryInt<or, "or">;
-defm XOR : BinaryInt<xor, "xor">;
-defm SHL : BinaryInt<shl, "shl">;
+defm AND : BinaryInt<and, "and ">;
+defm OR : BinaryInt<or, "or  ">;
+defm XOR : BinaryInt<xor, "xor ">;
+defm SHL : BinaryInt<shl, "shl ">;
 defm SHR_U : BinaryInt<srl, "shr_u">;
 defm SHR_S : BinaryInt<sra, "shr_s">;
 
-defm EQ : ComparisonInt<SETEQ, "eq">;
-defm NE : ComparisonInt<SETNE, "ne">;
+defm EQ : ComparisonInt<SETEQ, "eq  ">;
+defm NE : ComparisonInt<SETNE, "ne  ">;
 defm LT_S : ComparisonInt<SETLT, "lt_s">;
 defm LE_S : ComparisonInt<SETLE, "le_s">;
 defm LT_U : ComparisonInt<SETULT, "lt_u">;
@@ -37,8 +39,8 @@ defm GE_S : ComparisonInt<SETGE, "ge_s">;
 defm GT_U : ComparisonInt<SETUGT, "gt_u">;
 defm GE_U : ComparisonInt<SETUGE, "ge_u">;
 
-defm CLZ : UnaryInt<ctlz, "clz">;
-defm CTZ : UnaryInt<cttz, "ctz">;
+defm CLZ : UnaryInt<ctlz, "clz ">;
+defm CTZ : UnaryInt<cttz, "ctz ">;
 defm POPCNT : UnaryInt<ctpop, "popcnt">;
 
 // Expand the "don't care" operations to supported operations.
-- 
2.34.1