[PATCH] D60376: [llvm-objdump] Align instructions to a tab stop

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 10 07:02:08 PDT 2019


jhenderson added inline comments.


================
Comment at: test/tools/llvm-objdump/X86/disassemble-align.s:4
+
+# -w/--wide is ignored. llvm-objdump always produces wide output.
+# RUN: llvm-objdump -wd -print-imm-hex %t | sed 'y/\t/ /' | FileCheck -strict-whitespace %s
----------------
Since you did -w/--wide in a different change, you should probably remove the references to it from here.


================
Comment at: tools/llvm-objdump/llvm-objdump.cpp:609
+
+    {
+      formatted_raw_ostream FOS(OS);
----------------
Why do you need to scope this block?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D60376





More information about the llvm-commits mailing list