[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