[PATCH] D108238: [lit] Add option to output test result as resultdb json format.

[Julian Lettner via Phabricator via llvm-commits]

yln added a reviewer: jdenny.
yln added a comment.

Hi Haowei,

Can you change this to be a new type of report instead of changing an existing one?
Look at `TimeTraceReport` for the recipe.  Maybe `--resultdb-output` and `ResultDbReport` are appropriate names for parameter/class name?

This way the change would be purely additive and I then would be okay landing this as-is, i.e., without tests and documentation.  It would then be "your problem" if we (upstream) break this functionality.  This is only holds if my fellow reviewers think this okay as well, of course.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108238