[PATCH] D76213: [llvm-nm] Add test for `--debug-syms --dynamic`

Xing via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 19 05:54:51 PDT 2020


Higuoxing added a comment.

In D76213#1930891 <https://reviews.llvm.org/D76213#1930891>, @jhenderson wrote:

> LGTM. @Higuoxing, would you mind writing a separate dedicated test for --dynamic? It would allow for testing the corner cases, which are outside the scope of this test file. It can be a separate patch, no problem.


Of course, I'd like to. Thanks for taking time review!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76213





More information about the llvm-commits mailing list