[PATCH] D64206: [llvm\test\Object] - An initial step to cleanup the test cases.

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 5 04:17:04 PDT 2019


MaskRay added inline comments.


================
Comment at: test/Object/X86/objdump-trivial-object.test:61
+# ELF-x86-64:        0:       48 83 ec 08             subq    $8, %rsp
+# ELF-x86-64:        4:       c7 44 24 04 00 00 00 00 movl    $0, 4(%rsp)
+# ELF-x86-64:        c:       bf 00 00 00 00          movl    $0, %edi
----------------
MaskRay wrote:
> `       4: c7 44 24 04 00 00 00 00       movl    $0, 4(%rsp)`
> `00       movl`

Since you are updating this test, it may be good to make the number of spaces consistent with the actual output.




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

https://reviews.llvm.org/D64206





More information about the llvm-commits mailing list