<div dir="ltr">Hi Matthias,<div><br></div><div>Thinking about this, perhaps this functionality should be added at a higher level. LNT currently understands and can convert two formats: plist and JSON. Perhaps we should add the LIT JSON as a third type, that can be upgraded to the LNT JSON (with lnt convert, or lnt submit could do it transparently?)</div><div><br></div><div>The caveat would be that it would need to introspect the system as is done in builtintest.py so must be done on the system in question, but this doesn't sound unreasonable. This approach is similar to the one I've taken in the profiling infrastructure for importing/upgrading Linux Perf reports to the LNT profile format.</div><div><br></div><div>What do you think?</div><div><br></div><div>Cheers,</div><div><br></div><div>James</div></div><br><div class="gmail_quote"><div dir="ltr">On Thu, 5 May 2016 at 02:24 Matthias Braun via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">MatzeB added a comment.<br>
<br>
In <a href="http://reviews.llvm.org/D19949#421849" rel="noreferrer" target="_blank">http://reviews.llvm.org/D19949#421849</a>, @cmatthews wrote:<br>
<br>
> I agree with the updated test parsing logic, that is much better.<br>
><br>
> This flag seems to be redundant to the "lnt submit" feature which is supposed to do exactly the same thing.  I would guess that does not work with the lit json directly though?  It seems to me that would be a better place for this functionality, otherwise we have lnt runtest test-suite --just-submit <json> and lnt submit <json> which do the same thing to different kinds of json files.<br>
<br>
<br>
Yes that sounds like the better approach in terms of usability. I just have to find a way to refactor this because it currently relies on some code from <a href="http://test-suite.py/builtintest.py" rel="noreferrer" target="_blank">test-suite.py/builtintest.py</a> to fill in some information about the compiler and run orders which isn't available in the lit output.<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="http://reviews.llvm.org/D19949" rel="noreferrer" target="_blank">http://reviews.llvm.org/D19949</a><br>
<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div>