[PATCH] D16871: lit: Add StringMetricValue for TestResult.addMetric()

Matthias Braun via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 29 12:48:39 PST 2016


MatzeB abandoned this revision.
MatzeB added a comment.

Turns out you don't need a special StringMetricValue as the default fallback JSONMetricValue works as well here. Discovered because James
independently created a pretty similar patch for the testsuite (http://reviews.llvm.org/rL262220) in the meantime...


Repository:
  rL LLVM

http://reviews.llvm.org/D16871





More information about the llvm-commits mailing list