[compiler-rt] r269799 - [sanitizers] disabling LateInitialize call to fix the build.

[Steven Wu via llvm-commits]

It seems that our normal stage 1 builds are not running these test cases so it is not blocking other builds.
I think it should be fine to leave it failing for a while until Kuba can take a look. 
Kuba, do you have time to look at it or should I XFAIL them for now?

Thanks

Steven

> On May 17, 2016, at 1:49 PM, Mike Aizatsky <aizatsky at chromium.org> wrote:
> 
> Yes,
> 
> This is because I was trying to fix a memory leak that Kuba introduced. Rolling back all his stuff seems unrealistic (5 CLs at least). Since these CLs try to make Darwin work, I suggest to keep this build red until Kuba can address the issue. WDYT?
> 
> On Tue, May 17, 2016 at 1:27 PM Steven Wu <stevenwu at apple.com <mailto:stevenwu at apple.com>> wrote:
> Hi Mike
> 
> Your change breaks the Darwin bots: http://lab.llvm.org:8080/green/job/clang-stage1-cmake-RA-globalisel_check/1784/ <http://lab.llvm.org:8080/green/job/clang-stage1-cmake-RA-globalisel_check/1784/>
> It seems you are not in the recipients of the failing email. Can you take a look at it?
> 
> Thanks
> 
> Steven
> 
> > On May 17, 2016, at 11:44 AM, Mike Aizatsky via llvm-commits <llvm-commits at lists.llvm.org <mailto:llvm-commits at lists.llvm.org>> wrote:
> >
> > Author: aizatsky
> > Date: Tue May 17 13:44:21 2016
> > New Revision: 269799
> >
> > URL: http://llvm.org/viewvc/llvm-project?rev=269799&view=rev <http://llvm.org/viewvc/llvm-project?rev=269799&view=rev>
> > Log:
> > [sanitizers] disabling LateInitialize call to fix the build.
> >
> > http://reviews.llvm.org/rL269291 <http://reviews.llvm.org/rL269291> introduced a memory leak.
> > Disabling offending call temprorary rather than rolling back the chain
> > of CLs.
> >
> > Modified:
> >    compiler-rt/trunk/lib/asan/asan_rtl.cc
> >    compiler-rt/trunk/lib/tsan/rtl/tsan_rtl.cc
> >
> > Modified: compiler-rt/trunk/lib/asan/asan_rtl.cc
> > URL: http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/asan/asan_rtl.cc?rev=269799&r1=269798&r2=269799&view=diff <http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/asan/asan_rtl.cc?rev=269799&r1=269798&r2=269799&view=diff>
> > ==============================================================================
> > --- compiler-rt/trunk/lib/asan/asan_rtl.cc (original)
> > +++ compiler-rt/trunk/lib/asan/asan_rtl.cc Tue May 17 13:44:21 2016
> > @@ -553,7 +553,8 @@ static void AsanInitInternal() {
> >
> >   InitializeSuppressions();
> >
> > -  Symbolizer::LateInitialize();
> > +  // TODO(kuba) Fix Me.
> > +  // Symbolizer::LateInitialize();
> >
> >   VReport(1, "AddressSanitizer Init done\n");
> > }
> >
> > Modified: compiler-rt/trunk/lib/tsan/rtl/tsan_rtl.cc
> > URL: http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/tsan/rtl/tsan_rtl.cc?rev=269799&r1=269798&r2=269799&view=diff <http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/tsan/rtl/tsan_rtl.cc?rev=269799&r1=269798&r2=269799&view=diff>
> > ==============================================================================
> > --- compiler-rt/trunk/lib/tsan/rtl/tsan_rtl.cc (original)
> > +++ compiler-rt/trunk/lib/tsan/rtl/tsan_rtl.cc Tue May 17 13:44:21 2016
> > @@ -371,7 +371,8 @@ void Initialize(ThreadState *thr) {
> >   ctx->initialized = true;
> >
> > #ifndef SANITIZER_GO
> > -  Symbolizer::LateInitialize();
> > +  // TODO(kuba) Fix Me.
> > +  // Symbolizer::LateInitialize();
> > #endif
> >
> >   if (flags()->stop_on_start) {
> >
> >
> > _______________________________________________
> > llvm-commits mailing list
> > llvm-commits at lists.llvm.org <mailto:llvm-commits at lists.llvm.org>
> > http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits <http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits>
> 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160517/1b062c75/attachment.html>