[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