[PATCH] D60777: [llvm-objdump] Test tabs in disassemble-align.s with a more visible character

Jordan Rupprecht via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 16 08:45:19 PDT 2019


rupprecht accepted this revision.
rupprecht added a comment.
This revision is now accepted and ready to land.

Ok, this helps me see what's going on. Thanks!



================
Comment at: test/tools/llvm-objdump/X86/disassemble-align.s:2
 # RUN: llvm-mc -filetype=obj -triple=x86_64 %s -o %t
-# RUN: llvm-objdump -d -print-imm-hex %t | tr '\t' ' ' | FileCheck -strict-whitespace %s
+# RUN: llvm-objdump -d -print-imm-hex %t | tr '\t' '|' | FileCheck -strict-whitespace %s
 
----------------
Can you add a comment (here, or maybe below near the actual CHECK lines) that `|` is to show where the tabs line up?


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D60777/new/

https://reviews.llvm.org/D60777





More information about the llvm-commits mailing list