[PATCH] D25398: [lit] Use path as test suite name

Galina via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 6 11:35:39 PST 2016


gkistanova added a comment.

LitTestCommand parses LIT printout, looking for the ::, then uses the built "path" to make temp files with log chunks.

I have done a change to skip some intermediate folder names, as it was running out of the MAX_PATH on bots. If you want to have the absolute path there, we should have a reliable way to tell the root then, which could be stripped for these purposes. Ideas?

If you want to change the printout, we would need to change the LitTestCommand in advance, otherwise you would get a lot of bot failures.

Is there a spec of the proposed format somewhere? I'll see if I can change the LitTestCommand to support both formats during the transition. Unless this is something you plan to do yourself.


https://reviews.llvm.org/D25398





More information about the llvm-commits mailing list