On Fri, Oct 19, 2012 at 10:24 AM, Daniel Dunbar <span dir="ltr"><<a href="mailto:daniel@zuster.org" target="_blank" class="cremed">daniel@zuster.org</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Fri, Oct 19, 2012 at 1:11 AM, Chandler Carruth <<a href="mailto:chandlerc@google.com" class="cremed">chandlerc@google.com</a>> wrote:<br>

> On Thu, Oct 18, 2012 at 1:43 PM, Daniel Dunbar <<a href="mailto:daniel@zuster.org" class="cremed">daniel@zuster.org</a>> wrote:<br>
>><br>
>> Author: ddunbar<br>
>> Date: Thu Oct 18 15:43:04 2012<br>
>> New Revision: 166224<br>
>><br>
>> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=166224&view=rev" target="_blank" class="cremed">http://llvm.org/viewvc/llvm-project?rev=166224&view=rev</a><br>
>> Log:<br>
>> lit: Allow XFAIL: lines to also refer to "features".<br>
><br>
><br>
> What's the motivation for this patch?<br>
<br>
</div>The specific use case was wanting to XFAIL something that is broken<br>
when the compiler is built with LTO, but the reason I went this route<br>
was it seemed like a reasonably sensical and orthogonal feature.<br>
<div class="im"><br>
> I would much rather continue to use directory structures and/or REQUIRES to<br>
> represent feature differences.<br>
<br>
</div>That is a different use case, that indicates the test needs a<br>
particular feature, this indicates a test fails with a particular<br>
feature.<br>
<br>
>From a design standpoint, it actually seems cleaner to have all of the<br>
REQUIRES, XFAIL, XTARGET lines draw from the same namespace, and maybe<br>
move to treating triples as just another "feature". This is just a<br>
step closer to that.</blockquote><div><br></div><div>Ok, this makes more sense then. Thanks for the explanation.</div></div></div>