<div dir="ltr">I see.<div><br></div><div>Generally speaking, I think it is great to improve the implementation of this test format, but I don't think that it meets the bar to be included in lit proper. I want to keep lit pretty general purpose, and this test format is just so specific to the libc.* use case that its hard to imagine many other clients.</div><div><br></div><div>Part of my hesitation comes from feeling that this isn't such a great test format to start with. As a format, it isn't all that flexible (mangling part of the metadata into the filename really doesn't leave much room for extension, and then if we start putting metadata in the file too we have it in two places).</div><div><br></div><div>Is there another way you can manage sharing the test format implementation with those repositories?</div><div><br></div><div>Alternately, it seems like the main thing going for this test format is it makes sense when you have a large number of tests which all run the same action. We actually end up having that problem in other places too (there are subsets of clang tests which all follow the same pattern), so I wonder if there might some value in exploring an extension of the ShTest format that solved this problem explicitly, and then migrating the libc.* tests to that.</div><div><br></div><div>For example, just to brainstorm, if the ShTest format had a way to use a "prelude" script for all files in a directory, could that be the basis for using the ShTest format for libc++?</div><div><br></div><div> - Daniel</div><div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Dec 6, 2014 at 11:20 AM, Eric Fiselier <span dir="ltr"><<a href="mailto:eric@efcs.ca" target="_blank">eric@efcs.ca</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">>>! In D6206#29, @ddunbar wrote:<br>
> For my own context, is part of the motivation here that you want to share<br>
> this test format between multiple repositories?<br>
><br>
>  - Daniel<br>
<br>
</span>Gaa Sorry @ddunbar, I entirely missed this.<br>
<br>
Yes. both libc++ and libc++abi will use SuffixDispatchTest. They both currently have different versions of basically the same code.<br>
<br>
<a href="http://reviews.llvm.org/D6206" target="_blank">http://reviews.llvm.org/D6206</a><br>
<br>
<br>
</blockquote></div><br></div></div></div>