Hi Alp, This warning should be fixed by r210301. However, consider investigating why the frame size appears to be that large. I believe we build this code with GCC as well and have seen no complaints from its implementation of -Wframe-larger-than.
And thanks for implementing this flag! On Thu, Jun 5, 2014 at 4:00 PM, Alp Toker <[email protected]> wrote: > Hi Kostya, > > Looks like our new clang diagnostic from r210293 has caught a potential > issue in compiler-rt which is now breaking the build due to -Werror: > > /home/dtoolsbot/build/sanitizer-x86_64-linux/build/ > llvm/projects/compiler-rt/lib/tsan/rtl/tsan_rtl_mutex.cc:460:6: error: > stack frame size of 536 bytes in function '__tsan::ReportDeadlock' > [-Werror,-Wframe-larger-than] > void ReportDeadlock(ThreadState *thr, uptr pc, DDReport *r) { > ^ > [ 93%] [ 93%] Building C object lib/builtins/CMakeFiles/clang_ > rt.builtins-i386.dir/udivti3.c.o > [ 93%] Building CXX object lib/tsan/CMakeFiles/clang_rt. > tsan-x86_64.dir/rtl/tsan_suppressions.cc.o > 1 error generated. > > Can you confirm that the stack frame report is legitimate? > > Assuming it is, you'll probably want to either fix the function or relax > the byte limit passed to clang in your build system. > > Let me know if I can help. > > Alp. > > > > -------- Original Message -------- > Subject: buildbot failure in LLVM on sanitizer-x86_64-linux > Date: Thu, 05 Jun 2014 15:53:19 -0700 > From: [email protected] > To: Alp Toker <[email protected]>, Eric Christopher <[email protected]>, > Jingyue Wu <[email protected]> > CC: [email protected] > > > > The Buildbot has detected a new failure on builder sanitizer-x86_64-linux > while building llvm. > Full details are available at: > http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux/builds/10266 > > Buildbot URL: http://lab.llvm.org:8011/ > > Buildslave for this Build: sanitizer-buildbot1 > > Build Reason: scheduler > Build Source Stamp: [branch trunk] 210295 > Blamelist: alp,echristo,jingyue > > BUILD FAILED: failed annotate failed bootstrap clang failed run asan tests > failed run msan unit tests failed run 64-bit tsan unit tests failed run > 64-bit lsan unit tests failed run sanitizer_common tests > > sincerely, > -The Buildbot > > > > > > _______________________________________________ > cfe-commits mailing list > [email protected] > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits > -- Alexey Samsonov [email protected]
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
