<div dir="ltr">+thread-sanitizer (which was left off for some reason)<br><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Aug 28, 2015 at 10:06 AM, Jason Kim <span dir="ltr"><<a href="mailto:jasonk@codeaurora.org" target="_blank">jasonk@codeaurora.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div link="blue" vlink="purple" lang="EN-US"><div><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1f497d"><u></u> <u></u></span></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1f497d"><u></u> <u></u></span></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 #e1e1e1 1.0pt;padding:3.0pt 0in 0in 0in"><p class="MsoNormal"><b><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">From:</span></b><span style="font-size:11.0pt;font-family:"Calibri",sans-serif"> Jason Kim [mailto:<a href="mailto:jasonk@codeaurora.org" target="_blank">jasonk@codeaurora.org</a>] <br><b>Sent:</b> Friday, August 28, 2015 10:03 AM<br><b>To:</b> 'Dan Albert'<br><b>Cc:</b> 'Renato Golin'; 'Dmitry Vyukov'; 'Kostya Serebryany'; 'Tamas Berghammer'; 'Stephen Hines'; 'Tim Northover'; '<a href="mailto:kanheim@a-bix.com" target="_blank">kanheim@a-bix.com</a>'; 'Chad Rosier'; 'Evgenii Stepanov'; 'Adhemerval Zanella'; 'LLVM Dev'; 'Jason Kim'; 'Kristof Beyls'; 'James Molloy'<br><b>Subject:</b> RE: TSAN hack on AArch64 for Android<u></u><u></u></span></p></div></div><p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal"><u></u> <u></u></p><div><span class=""><p class="MsoNormal">IMO having to disable 2/3 of the tests means the patch isn't ready yet.<u></u><u></u></p></span><p class="MsoNormal"><b><i><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1f497d">[ jasonk --> ]  OK, that’s a fair view.  However, please keep in mind the follwing:<u></u><u></u></span></i></b></p><span class=""><p class="MsoNormal"><b><i><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1f497d"><u></u> <u></u></span></i></b></p><p class="MsoNormal"><b><i><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1f497d">(*) The tests weren’t all stable to begin with. <u></u><u></u></span></i></b></p><p class="MsoNormal"><b><i><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1f497d">(*) I am perfectly willing to triage the failures, but please keep in mind that any “threshold” that we cross from NOT testing TSAN on aarch64+android to actually testing on it is going to be a large hurdle. <u></u><u></u></span></i></b></p><p class="MsoNormal"><b><i><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1f497d">(*) BIONIC is NOT glibc!<u></u><u></u></span></i></b></p><p class="MsoNormal"><b><i><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1f497d">(*) Its pretty darn near impossible to  have the perfect patch, which is [a] small [b] doesn’t break anything [c] adds major new features. <u></u><u></u></span></i></b></p><p class="MsoNormal"><b><i><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1f497d">(*) What I have currently is: [a] not very big [b] doesn’t break anything VISIBLY [c] adds major new features<u></u><u></u></span></i></b></p><p class="MsoNormal"><b><i><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1f497d">(*) The new feature (TSAN on aarch64-linux-androideabi) is currently turned OFF by default<u></u><u></u></span></i></b></p><p class="MsoNormal"><b><i><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1f497d">(*) There are plenty of cases in prior history in llvm where a set of tests were turned off temporarily, especially when a major new platform is being introduced. I don’t think its unreasonable to have that as a starting point.</span></i></b><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1f497d"><u></u><u></u></span></p></span></div><div><p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal"><span style="color:#1f497d"><u></u> <u></u></span></p><p class="MsoNormal"><b><i><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1f497d">[ jasonk --> ] I don’t know if this is getting through, but the tests I marked UNSUPPORTED, are ONLY unsupported on aarch64-linux-androideabi. They are run, as is, on other configurations of compiler-rt.</span></i></b><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1f497d"><u></u><u></u></span></p></div></div></div></div></blockquote></div><br></div></div>