[PATCH] D85024: [llvm-objdump] Implement --prefix option

Vinicius Tinti via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 5 19:49:44 PDT 2020


tinti marked an inline comment as done.
tinti added a comment.

Hi @jhenderson,

I believe I should create more tests addressing:

- On Windows trailing '\' should be removed.
- On Posix trailing '\' should be kept.

As far as I know I would need to duplicate  `source-interleave-absolute-paths.test` and add REQUIRES: x (or UNSUPPORTED: x).

Is there a better way of doing it?


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

https://reviews.llvm.org/D85024



More information about the llvm-commits mailing list