Hi Gabor, I noticed that one particular test fails intermittently on the AArch64 bot: http://lab.llvm.org:8011/builders/clang-native-aarch64-full FAIL: Profile:: instrprof-set-filename-then-reset-default.c Some times on stage1, others on stage2, others no fail at all. All the commits during these builds are not related to profiling or AArch64, so I believe this has something to do with the setup. Maybe try to reproduce it locally? It could be something to do with the linker, too. cheers, --renato
I added this test yesterday, so will take a look. The test is confirming that the invoking __llvm_profile_set_filename(0) properly resets the profile output to "default.profraw" (the default). When it fails I see the following warning: warning: no profile data available for file "instrprof-set-filename-then-reset-default.c" [-Wprofile-instr-unprofiled] 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. Teresa On Wed, Apr 29, 2015 at 4:06 AM, Renato Golin <renato.golin at linaro.org> wrote:> Hi Gabor, > > I noticed that one particular test fails intermittently on the AArch64 bot: > > http://lab.llvm.org:8011/builders/clang-native-aarch64-full > > FAIL: Profile:: instrprof-set-filename-then-reset-default.c > > Some times on stage1, others on stage2, others no fail at all. > > All the commits during these builds are not related to profiling or > AArch64, so I believe this has something to do with the setup. > > Maybe try to reproduce it locally? It could be something to do with > the linker, too. > > 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
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? cheers, --renato
Hi Renato, Thanks for the warning but I am a little busy this week. If this is still an issue next week I will try to reproduce it locally. Thanks, Gabor On 04/29/2015 01:06 PM, Renato Golin wrote:> Hi Gabor, > > I noticed that one particular test fails intermittently on the AArch64 bot: > > http://lab.llvm.org:8011/builders/clang-native-aarch64-full > > FAIL: Profile:: instrprof-set-filename-then-reset-default.c > > Some times on stage1, others on stage2, others no fail at all. > > All the commits during these builds are not related to profiling or > AArch64, so I believe this has something to do with the setup. > > Maybe try to reproduce it locally? It could be something to do with > the linker, too. > > cheers, > --renato
On 29 April 2015 at 14:31, Gabor Ballabas <gaborb at inf.u-szeged.hu> wrote:> Thanks for the warning but I am a little busy this week. > If this is still an issue next week I will try to reproduce it > locally.Don't worry, Teresa has found the problem, we'll fix on our end, hopefully. :) cheers! --renato