[PATCH] D64246: [test/Object/nm-trivial-object.test] - Remove 5 precompiled binaries.

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 9 04:37:05 PDT 2019


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

LGTM, with one comment.



================
Comment at: test/Object/nm-trivial-object.test:80
 
 # RUN: yaml2obj -docnum=2 %s > %t.elf-x86-64
 # RUN: llvm-nm %t.elf-x86-64 | FileCheck %s -check-prefix ELF64
----------------
A comment describing this test case might be nice. Perhaps "Test different ELF symbols for 64-bit platform."


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

https://reviews.llvm.org/D64246





More information about the llvm-commits mailing list