[PATCH] D100011: [lld-macho] Make time-trace* options more permissive.
Vy Nguyen via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Apr 7 12:38:18 PDT 2021
oontvoo marked an inline comment as done.
oontvoo added inline comments.
================
Comment at: lld/test/MachO/time-trace.s:17
+## Test trace requested to stdout, also test that `--time-trace` is not needed if the other two are used.
+# RUN: %lld --time-trace-file=- --time-trace-granularity=0 -o %t3.macho %t.o \
# RUN: | %python -c 'import json, sys; json.dump(json.loads(sys.stdin.read()), sys.stdout, sort_keys=True, indent=2)' \
----------------
int3 wrote:
> technically we should test if *either* of the other two are used :)
Done - I've removed the `--time-trace` from line 11 - that should test it :)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D100011/new/
https://reviews.llvm.org/D100011
More information about the llvm-commits
mailing list