[LLVMdev] AArch64 bot unstable
David Blaikie
dblaikie at gmail.com
Wed Apr 29 10:05:50 PDT 2015
On Wed, Apr 29, 2015 at 9:48 AM, Teresa Johnson <tejohnson at google.com>
wrote:
> Ok, thanks for the suggestion. I will rework the tests to do that.
>
In case you haven't found it already, %T in the lit syntax gives you a
uniquely named directory for the test
> Teresa
>
> On Wed, Apr 29, 2015 at 9:47 AM, Eric Christopher <echristo at gmail.com>
> wrote:
> >
> >
> > On Wed, Apr 29, 2015 at 9:36 AM Justin Bogner <mail at justinbogner.com>
> wrote:
> >>
> >> Teresa Johnson <tejohnson at google.com> writes:
> >> > On Wed, Apr 29, 2015 at 6:27 AM, Renato Golin <
> renato.golin at linaro.org>
> >> > wrote:
> >> >> On 29 April 2015 at 14:16, Teresa Johnson <tejohnson at google.com>
> wrote:
> >> >>> Two of the compiler-rt/profile tests I added are checking for
> similar
> >> >>> behavior with resetting the filename to the default. I wonder if
> they
> >> >>> are running in parallel and clobbering each other since the profile
> >> >>> output names are not unique after the reset.
> >> >>
> >> >> Hi Teresa,
> >> >>
> >> >> This would explain the intermittent failures. Maybe making the names
> >> >> unique would fix the issues, would that be an easy change?
> >> >
> >> > After thinking about it I believe this is what is most likely
> >> > happening, and that these two new tests I added will need to be
> >> > reverted:
> >> >
> >> >
> >> >
> compiler-rt/trunk/test/profile/instrprof-override-filename-then-reset-default.c
> >> >
> >> >
> compiler-rt/trunk/test/profile/instrprof-set-filename-then-reset-default.c
> >> >
> >> > since they aren't writing to unique output names. These are 2 of the 4
> >> > tests added in r236056 (the other two are fine).
> >> >
> >> > They can't be changed to write to a unique name since those tests were
> >> > specifically testing that the profile output name gets reset to the
> >> > default when null is passed to the profile filename setting
> >> > interfaces. The other profile tests use unique names.
> >>
> >> Could we have these tests `cd` into a uniquely named directory or
> >> something? Maybe that's more complicated than its worth.
> >>
> >
> > That's what we've done in the past and seems a reasonable idea.
> >
> > -eric
> >
> >>
> >> > Since I don't have write access yet, can someone revert those two
> >> > files for me?
> >> >
> >> > Thanks,
> >> > Teresa
> >> >>
> >> >> cheers,
> >> >> --renato
> >> _______________________________________________
> >> LLVM Developers mailing list
> >> LLVMdev at cs.uiuc.edu http://llvm.cs.uiuc.edu
> >> http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev
>
>
>
> --
> Teresa Johnson | Software Engineer | tejohnson at google.com | 408-460-2413
> _______________________________________________
> LLVM Developers mailing list
> LLVMdev at cs.uiuc.edu http://llvm.cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20150429/c00e13e5/attachment.html>
More information about the llvm-dev
mailing list