]> granicus.if.org Git - llvm/commitdiff
bpf: proper print imm64 expression in inst printer
authorYonghong Song <yhs@fb.com>
Fri, 8 Sep 2017 23:32:38 +0000 (23:32 +0000)
committerYonghong Song <yhs@fb.com>
Fri, 8 Sep 2017 23:32:38 +0000 (23:32 +0000)
Fixed an issue in printImm64Operand where if the value is
an expression, print out the expression properly. Currently,
it will print
  r1 = <MCOperand Expr:(tx_port)>ll
With the patch, the printout will be
  r1 = tx_port

Suggested-by: Jiong Wang <jiong.wang@netronome.com>
Signed-off-by: Yonghong Song <yhs@fb.com>
Acked-by: Alexei Starovoitov <ast@kernel.org>
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@312833 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Target/BPF/BPFInstrInfo.td
lib/Target/BPF/InstPrinter/BPFInstPrinter.cpp

index 3b239e7463d260da6e4376347c244c893e004825..d4c50e768f745ae0de19249484719f62d7793359 100644 (file)
@@ -249,7 +249,7 @@ class MOV_RI<string OpcodeStr>
 
 class LD_IMM64<bits<4> Pseudo, string OpcodeStr>
     : InstBPF<(outs GPR:$dst), (ins u64imm:$imm),
-              "$dst "#OpcodeStr#" ${imm}ll",
+              "$dst "#OpcodeStr#" ${imm}",
               [(set GPR:$dst, (i64 imm:$imm))]> {
 
   bits<3> mode;
index 64e986fe0f04693205c026aff9f66fbcdc2c93a8..bb5546ea40ed5c261246350966730e02c4e7dff5 100644 (file)
@@ -88,7 +88,9 @@ void BPFInstPrinter::printImm64Operand(const MCInst *MI, unsigned OpNo,
                                        raw_ostream &O) {
   const MCOperand &Op = MI->getOperand(OpNo);
   if (Op.isImm())
-    O << (uint64_t)Op.getImm();
+    O << (uint64_t)Op.getImm() << "ll";
+  else if (Op.isExpr())
+    printExpr(Op.getExpr(), O);
   else
     O << Op;
 }