<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Mar 30, 2015 at 2:46 PM, 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:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><span class="">In <a href="http://reviews.llvm.org/D8703#149021" target="_blank">http://reviews.llvm.org/D8703#149021</a>, @ed wrote:<br>
<br>
> In <a href="http://reviews.llvm.org/D8703#148896" target="_blank">http://reviews.llvm.org/D8703#148896</a>, @EricWF wrote:<br>
><br>
> > The CMake and LIT stuff LGTM. However I would rather keep the directory structure in test/std the same as the structure of the C++ standard clauses. Could you rename the fail tests, put them in the correct directories and use `// REQUIRES: libcpp-has-no-thread-unsafe-c-functions`?<br>
><br>
><br>
> Sure! Just to make sure I understand it correctly: in this specific case, the tests should be called:<br>
><br>
>   test/std/utilities/date.time/asctime.fail.cpp<br>
>   test/std/utilities/date.time/ctime.fail.cpp<br>
>   test/std/utilities/date.time/gmtime.fail.cpp<br>
>   test/std/utilities/date.time/localtime.fail.cpp<br>
><br>
><br>
> As in, the same as they are right now, but without the no.thread.unsafe.c.functions component in between?<br>
<br>
<br>
</span>Pretty much, but I would make the test names more descriptive so they infer that these are failure tests for when we have no unsafe C functions. The test names don't *have* to follow the standard names.<br><br></blockquote><div> </div>The rule is that each (leaf) folder corresponds to a section in the standard.<div>Since there is no section named "[no.thread.unsafe.c.functions]", we shouldn't have a folder with that name. </div><div> </div></div>-- Marshall</div><div class="gmail_extra"><br></div></div>