<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
<meta name="Generator" content="Microsoft Word 14 (filtered medium)">
<style><!--
/* Font Definitions */
@font-face
{font-family:Calibri;
panose-1:2 15 5 2 2 2 4 3 2 4;}
@font-face
{font-family:Tahoma;
panose-1:2 11 6 4 3 5 4 4 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
{margin:0in;
margin-bottom:.0001pt;
font-size:12.0pt;
font-family:"Times New Roman","serif";}
a:link, span.MsoHyperlink
{mso-style-priority:99;
color:blue;
text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
{mso-style-priority:99;
color:purple;
text-decoration:underline;}
span.EmailStyle17
{mso-style-type:personal-reply;
font-family:"Calibri","sans-serif";
color:#1F497D;}
.MsoChpDefault
{mso-style-type:export-only;
font-family:"Calibri","sans-serif";}
@page WordSection1
{size:8.5in 11.0in;
margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
{page:WordSection1;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]-->
</head>
<body lang="EN-US" link="blue" vlink="purple">
<div class="WordSection1">
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D">A simple 'mkdir' certainly works on Windows, but probably if you add any extra options that would be a problem.<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D">--paulr<o:p></o:p></span></p>
<p class="MsoNormal"><a name="_MailEndCompose"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D"><o:p> </o:p></span></a></p>
<div style="border:none;border-left:solid blue 1.5pt;padding:0in 0in 0in 4.0pt">
<div>
<div style="border:none;border-top:solid #B5C4DF 1.0pt;padding:3.0pt 0in 0in 0in">
<p class="MsoNormal"><b><span style="font-size:10.0pt;font-family:"Tahoma","sans-serif"">From:</span></b><span style="font-size:10.0pt;font-family:"Tahoma","sans-serif""> llvmdev-bounces@cs.uiuc.edu [mailto:llvmdev-bounces@cs.uiuc.edu]
<b>On Behalf Of </b>David Blaikie<br>
<b>Sent:</b> Wednesday, April 29, 2015 11:43 AM<br>
<b>To:</b> Teresa Johnson<br>
<b>Cc:</b> LLVM Dev<br>
<b>Subject:</b> Re: [LLVMdev] AArch64 bot unstable<o:p></o:p></span></p>
</div>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
<div>
<p class="MsoNormal">On Wed, Apr 29, 2015 at 11:31 AM, Teresa Johnson <<a href="mailto:tejohnson@google.com" target="_blank">tejohnson@google.com</a>> wrote:<o:p></o:p></p>
<p class="MsoNormal">On Wed, Apr 29, 2015 at 11:19 AM, David Blaikie <<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>> wrote:<br>
><br>
><br>
> On Wed, Apr 29, 2015 at 10:55 AM, Teresa Johnson <<a href="mailto:tejohnson@google.com">tejohnson@google.com</a>><br>
> wrote:<br>
>><br>
>> On Wed, Apr 29, 2015 at 10:05 AM, David Blaikie <<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>><br>
>> wrote:<br>
>> ><br>
>> ><br>
>> > On Wed, Apr 29, 2015 at 9:48 AM, Teresa Johnson <<a href="mailto:tejohnson@google.com">tejohnson@google.com</a>><br>
>> > wrote:<br>
>> >><br>
>> >> Ok, thanks for the suggestion. I will rework the tests to do that.<br>
>> ><br>
>> ><br>
>> > In case you haven't found it already, %T in the lit syntax gives you a<br>
>> > uniquely named directory for the test<br>
>><br>
>> Actually %T is just the base directory where all the<br>
>> compiler-rt/profile tests are run, not unique per .c test.<br>
><br>
><br>
> Ah, sorry for the misleading advice - thanks for the correction.<br>
><br>
>><br>
>> I fixed the<br>
>> tests by creating a "%t.d" directory and cd'ing into it and generating<br>
>> all output locally in that directory.<br>
><br>
><br>
> A quick "grep -r "RUN.*mkdir" certainly shows a few tests doing quite<br>
> similar things (most use -p, which I see you've done too).<br>
><br>
> You'll probably also need REQUIRES: shell in this test to ensure it doesn't<br>
> run in environments that don't have a full bash-like shell available (such<br>
> as Windows). Check the other tests that use mkdir to see that sort of thing.<br>
<br>
Some have the "REQUIRES: shell" (e.g.<br>
tools/clang/test/VFS/include-virtual-from-real.c) and some don't (e.g.<br>
./tools/clang/test/Analysis/html-diags.c). I looked around for<br>
documentation on when "REQUIRES: shell" is needed but couldn't find<br>
anything specific. The latter test (html-diags.c), which doesn't have<br>
the REQUIRES also does mkdir -p, cd, rm -rf", which are the same<br>
shell-like operations my tests are doing. The former<br>
(include-virtual-from-real.c) is doing some additional shell-like<br>
operations such as sed and echo, which I am not doing. <o:p></o:p></p>
<div>
<p class="MsoNormal"><br>
Sounds plausible then.<br>
<o:p></o:p></p>
</div>
<blockquote style="border:none;border-left:solid #CCCCCC 1.0pt;padding:0in 0in 0in 6.0pt;margin-left:4.8pt;margin-right:0in">
<p class="MsoNormal">Is there a list<br>
of operations safe to do without adding the "REQUIRES: shell"?<o:p></o:p></p>
</blockquote>
<div>
<p class="MsoNormal"><br>
Not that I know of. Usually a bit of "oh, this fails on a windows buildbot -> slap REQUIRES: shell on it" (maybe other people know more). But as you've seen tests that don't seem to require it and do the same things your test does - just go with it & if buildbots
fail we'll figure it out then.<br>
<br>
- David<br>
<o:p></o:p></p>
</div>
<blockquote style="border:none;border-left:solid #CCCCCC 1.0pt;padding:0in 0in 0in 6.0pt;margin-left:4.8pt;margin-right:0in">
<p class="MsoNormal"><br>
Thanks,<br>
Teresa<o:p></o:p></p>
<div>
<div>
<p class="MsoNormal"><br>
><br>
>><br>
>><br>
>> Patch uploaded here, PTAL:<br>
>> <a href="http://reviews.llvm.org/D9349" target="_blank">http://reviews.llvm.org/D9349</a><br>
>><br>
>> Thanks,<br>
>> Teresa<br>
>><br>
>> ><br>
>> >><br>
>> >> Teresa<br>
>> >><br>
>> >> On Wed, Apr 29, 2015 at 9:47 AM, Eric Christopher <<a href="mailto:echristo@gmail.com">echristo@gmail.com</a>><br>
>> >> wrote:<br>
>> >> ><br>
>> >> ><br>
>> >> > On Wed, Apr 29, 2015 at 9:36 AM Justin Bogner <<a href="mailto:mail@justinbogner.com">mail@justinbogner.com</a>><br>
>> >> > wrote:<br>
>> >> >><br>
>> >> >> Teresa Johnson <<a href="mailto:tejohnson@google.com">tejohnson@google.com</a>> writes:<br>
>> >> >> > On Wed, Apr 29, 2015 at 6:27 AM, Renato Golin<br>
>> >> >> > <<a href="mailto:renato.golin@linaro.org">renato.golin@linaro.org</a>><br>
>> >> >> > wrote:<br>
>> >> >> >> On 29 April 2015 at 14:16, Teresa Johnson <<a href="mailto:tejohnson@google.com">tejohnson@google.com</a>><br>
>> >> >> >> wrote:<br>
>> >> >> >>> Two of the compiler-rt/profile tests I added are checking for<br>
>> >> >> >>> similar<br>
>> >> >> >>> behavior with resetting the filename to the default. I wonder if<br>
>> >> >> >>> they<br>
>> >> >> >>> are running in parallel and clobbering each other since the<br>
>> >> >> >>> profile<br>
>> >> >> >>> output names are not unique after the reset.<br>
>> >> >> >><br>
>> >> >> >> Hi Teresa,<br>
>> >> >> >><br>
>> >> >> >> This would explain the intermittent failures. Maybe making the<br>
>> >> >> >> names<br>
>> >> >> >> unique would fix the issues, would that be an easy change?<br>
>> >> >> ><br>
>> >> >> > After thinking about it I believe this is what is most likely<br>
>> >> >> > happening, and that these two new tests I added will need to be<br>
>> >> >> > reverted:<br>
>> >> >> ><br>
>> >> >> ><br>
>> >> >> ><br>
>> >> >> ><br>
>> >> >> > compiler-rt/trunk/test/profile/instrprof-override-filename-then-reset-default.c<br>
>> >> >> ><br>
>> >> >> ><br>
>> >> >> ><br>
>> >> >> > compiler-rt/trunk/test/profile/instrprof-set-filename-then-reset-default.c<br>
>> >> >> ><br>
>> >> >> > since they aren't writing to unique output names. These are 2 of<br>
>> >> >> > the<br>
>> >> >> > 4<br>
>> >> >> > tests added in r236056 (the other two are fine).<br>
>> >> >> ><br>
>> >> >> > They can't be changed to write to a unique name since those tests<br>
>> >> >> > were<br>
>> >> >> > specifically testing that the profile output name gets reset to<br>
>> >> >> > the<br>
>> >> >> > default when null is passed to the profile filename setting<br>
>> >> >> > interfaces. The other profile tests use unique names.<br>
>> >> >><br>
>> >> >> Could we have these tests `cd` into a uniquely named directory or<br>
>> >> >> something? Maybe that's more complicated than its worth.<br>
>> >> >><br>
>> >> ><br>
>> >> > That's what we've done in the past and seems a reasonable idea.<br>
>> >> ><br>
>> >> > -eric<br>
>> >> ><br>
>> >> >><br>
>> >> >> > Since I don't have write access yet, can someone revert those two<br>
>> >> >> > files for me?<br>
>> >> >> ><br>
>> >> >> > Thanks,<br>
>> >> >> > Teresa<br>
>> >> >> >><br>
>> >> >> >> cheers,<br>
>> >> >> >> --renato<br>
>> >> >> _______________________________________________<br>
>> >> >> LLVM Developers mailing list<br>
>> >> >> <a href="mailto:LLVMdev@cs.uiuc.edu">LLVMdev@cs.uiuc.edu</a> <a href="http://llvm.cs.uiuc.edu" target="_blank">http://llvm.cs.uiuc.edu</a><br>
>> >> >> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">
http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev</a><br>
>> >><br>
>> >><br>
>> >><br>
>> >> --<br>
>> >> Teresa Johnson | Software Engineer | <a href="mailto:tejohnson@google.com">
tejohnson@google.com</a> |<br>
>> >> <a href="tel:408-460-2413">408-460-2413</a><br>
>> >> _______________________________________________<br>
>> >> LLVM Developers mailing list<br>
>> >> <a href="mailto:LLVMdev@cs.uiuc.edu">LLVMdev@cs.uiuc.edu</a> <a href="http://llvm.cs.uiuc.edu" target="_blank">http://llvm.cs.uiuc.edu</a><br>
>> >> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">
http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev</a><br>
>> ><br>
>> ><br>
>><br>
>><br>
>><br>
>> --<br>
>> Teresa Johnson | Software Engineer | <a href="mailto:tejohnson@google.com">tejohnson@google.com</a> |
<a href="tel:408-460-2413">408-460-2413</a><br>
><br>
><br>
<br>
<br>
<br>
--<br>
Teresa Johnson | Software Engineer | <a href="mailto:tejohnson@google.com">tejohnson@google.com</a> |
<a href="tel:408-460-2413">408-460-2413</a><o:p></o:p></p>
</div>
</div>
</blockquote>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
</div>
</div>
</div>
</body>
</html>