[Lldb-commits] [PATCH] D74023: [RISCV] ELF attribute section for RISC-V

Hsiangkai Wang via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Mon Mar 30 00:30:07 PDT 2020


HsiangKai added a comment.

In D74023#1948388 <https://reviews.llvm.org/D74023#1948388>, @MaskRay wrote:

> The code generally looks good. For unittests, I think we can either make llvm-readobj -A canonical or the unittests canonical. If we decide to place tests on one place, we should delete most tests on the other side.
>
> My current preference is that we use more of unittests and leave the minimum to `test/llvm-readobj/ELF/{ARM,RISCV}/`


Agree. I will remove redundant tests in test/tools/llvm-readobj/ELF/{ARM,RISCV}/.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74023





More information about the lldb-commits mailing list