[llvm] r253468 - [WebAssembly] Add some spaces to the assembly output to vertically align operands.

Dan Gohman via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 18 08:25:38 PST 2015


Author: djg
Date: Wed Nov 18 10:25:38 2015
New Revision: 253468

URL: http://llvm.org/viewvc/llvm-project?rev=253468&view=rev
Log:
[WebAssembly] Add some spaces to the assembly output to vertically align operands.

Modified:
    llvm/trunk/lib/Target/WebAssembly/WebAssemblyInstrFloat.td
    llvm/trunk/lib/Target/WebAssembly/WebAssemblyInstrInteger.td

Modified: llvm/trunk/lib/Target/WebAssembly/WebAssemblyInstrFloat.td
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/WebAssembly/WebAssemblyInstrFloat.td?rev=253468&r1=253467&r2=253468&view=diff
==============================================================================
--- llvm/trunk/lib/Target/WebAssembly/WebAssemblyInstrFloat.td (original)
+++ llvm/trunk/lib/Target/WebAssembly/WebAssemblyInstrFloat.td Wed Nov 18 10:25:38 2015
@@ -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, "near
 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)>;

Modified: llvm/trunk/lib/Target/WebAssembly/WebAssemblyInstrInteger.td
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/WebAssembly/WebAssemblyInstrInteger.td?rev=253468&r1=253467&r2=253468&view=diff
==============================================================================
--- llvm/trunk/lib/Target/WebAssembly/WebAssemblyInstrInteger.td (original)
+++ llvm/trunk/lib/Target/WebAssembly/WebAssemblyInstrInteger.td Wed Nov 18 10:25:38 2015
@@ -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.




More information about the llvm-commits mailing list