<div dir="ltr"><div>Sorry for the breakage, D32545 should fix that.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Apr 26, 2017 at 8:10 AM, Bill Seurer <span dir="ltr"><<a href="mailto:seurer@linux.vnet.ibm.com" target="_blank">seurer@linux.vnet.ibm.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">It looks like all the failing tests now succeed except<br>
<br>
FAIL: AddressSanitizer-powerpc64le-l<wbr>inux::stress_dtls.c<br>
<br>
This one also started failing with 301048.<span class=""><br>
<br>
<br>
<br>
On 04/25/2017 12:38 PM, Aleksey Shlyapnikov wrote:<br>
</span><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
<a href="https://reviews.llvm.org/rL301335" rel="noreferrer" target="_blank">https://reviews.llvm.org/rL301<wbr>335</a> should take care of it.<br>
<br>
On Tue, Apr 25, 2017 at 10:16 AM, Aleksey Shlyapnikov<br></span><span class="">
<<a href="mailto:alekseys@google.com" target="_blank">alekseys@google.com</a> <mailto:<a href="mailto:alekseys@google.com" target="_blank">alekseys@google.com</a>>> wrote:<br>
<br>
    I'll look into it.<br>
<br>
    On Tue, Apr 25, 2017 at 9:05 AM, Bill Seurer<br></span><span class="">
    <<a href="mailto:seurer@linux.vnet.ibm.com" target="_blank">seurer@linux.vnet.ibm.com</a> <mailto:<a href="mailto:seurer@linux.vnet.ibm.com" target="_blank">seurer@linux.vnet.ibm.<wbr>com</a>>> wrote:<br>
<br>
        This is causing several test case failures on ppc64le:<br>
<br></span>
        <a href="http://lab.llvm.org:8011/builders/clang-ppc64le-linux/builds/5614" rel="noreferrer" target="_blank">http://lab.llvm.org:8011/build<wbr>ers/clang-ppc64le-linux/builds<wbr>/5614</a> <<a href="http://lab.llvm.org:8011/builders/clang-ppc64le-linux/builds/5614" rel="noreferrer" target="_blank">http://lab.llvm.org:8011/buil<wbr>ders/clang-ppc64le-linux/build<wbr>s/5614</a>><div><div class="h5"><br>
<br>
<br>
        On 04/21/2017 04:59 PM, Alex Shlyapnikov via llvm-commits wrote:<br>
<br>
            Author: alekseyshl<br>
            Date: Fri Apr 21 16:59:53 2017<br>
            New Revision: 301048<br>
<br>
            URL: <a href="http://llvm.org/viewvc/llvm-project?rev=301048&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-pr<wbr>oject?rev=301048&view=rev</a><br>
            <<a href="http://llvm.org/viewvc/llvm-project?rev=301048&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-p<wbr>roject?rev=301048&view=rev</a>><br>
            Log:<br>
            [lsan] Enable LSan on PowerPC64.<br>
<br>
            Summary: Re-landing reverted D31995 with suppressions<br>
            defined in D32303 and D32377.<br>
<br>
            Reviewers: eugenis<br>
<br>
            Subscribers: nemanjai, llvm-commits<br>
<br>
            Differential Revision: <a href="https://reviews.llvm.org/D32379" rel="noreferrer" target="_blank">https://reviews.llvm.org/D3237<wbr>9</a><br>
            <<a href="https://reviews.llvm.org/D32379" rel="noreferrer" target="_blank">https://reviews.llvm.org/D323<wbr>79</a>><br>
<br>
            Modified:<br>
                compiler-rt/trunk/lib/lsan/lsa<wbr>n_allocator.h<br>
                compiler-rt/trunk/lib/lsan/lsa<wbr>n_common.h<br>
                compiler-rt/trunk/test/lsan/li<wbr>t.common.cfg<br>
<br>
            Modified: compiler-rt/trunk/lib/lsan/lsa<wbr>n_allocator.h<br>
            URL:<br>
            <a href="http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/lsan/lsan_allocator.h?rev=301048&r1=301047&r2=301048&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-pr<wbr>oject/compiler-rt/trunk/lib/ls<wbr>an/lsan_allocator.h?rev=301048<wbr>&r1=301047&r2=301048&view=diff</a><br>
            <<a href="http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/lsan/lsan_allocator.h?rev=301048&r1=301047&r2=301048&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-p<wbr>roject/compiler-rt/trunk/lib/l<wbr>san/lsan_allocator.h?rev=30104<wbr>8&r1=301047&r2=301048&view=<wbr>diff</a>><br>
            ==============================<wbr>==============================<wbr>==================<br>
            --- compiler-rt/trunk/lib/lsan/lsa<wbr>n_allocator.h (original)<br>
            +++ compiler-rt/trunk/lib/lsan/lsa<wbr>n_allocator.h Fri Apr 21<br>
            16:59:53 2017<br>
            @@ -59,7 +59,7 @@ typedef CompactSizeClassMap SizeClassMap<br>
             typedef SizeClassAllocator32<0, SANITIZER_MMAP_RANGE_SIZE,<br>
                 sizeof(ChunkMetadata), SizeClassMap, kRegionSizeLog,<br>
            ByteMap><br>
                 PrimaryAllocator;<br>
            -#elif defined(__x86_64__)<br>
            +#elif defined(__x86_64__) || defined(__powerpc64__)<br>
             struct AP64 {  // Allocator64 parameters. Deliberately<br>
            using a short name.<br>
               static const uptr kSpaceBeg = 0x600000000000ULL;<br>
               static const uptr kSpaceSize =  0x40000000000ULL; // 4T.<br>
<br>
            Modified: compiler-rt/trunk/lib/lsan/lsa<wbr>n_common.h<br>
            URL:<br>
            <a href="http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/lsan/lsan_common.h?rev=301048&r1=301047&r2=301048&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-pr<wbr>oject/compiler-rt/trunk/lib/ls<wbr>an/lsan_common.h?rev=301048&r1<wbr>=301047&r2=301048&view=diff</a><br>
            <<a href="http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/lsan/lsan_common.h?rev=301048&r1=301047&r2=301048&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-p<wbr>roject/compiler-rt/trunk/lib/l<wbr>san/lsan_common.h?rev=301048&r<wbr>1=301047&r2=301048&view=diff</a>><br>
            ==============================<wbr>==============================<wbr>==================<br>
            --- compiler-rt/trunk/lib/lsan/lsa<wbr>n_common.h (original)<br>
            +++ compiler-rt/trunk/lib/lsan/lsa<wbr>n_common.h Fri Apr 21<br>
            16:59:53 2017<br>
            @@ -32,7 +32,8 @@<br>
             // new architecture inside sanitizer library.<br>
             #if (SANITIZER_LINUX && !SANITIZER_ANDROID ||<br>
            SANITIZER_MAC) && \<br>
                 (SANITIZER_WORDSIZE == 64) &&<br>
                 \<br>
            -    (defined(__x86_64__) || defined(__mips64) ||<br>
            defined(__aarch64__))<br>
            +    (defined(__x86_64__) || defined(__mips64) ||<br>
            defined(__aarch64__) || \<br>
            +     defined(__powerpc64__))<br>
             #define CAN_SANITIZE_LEAKS 1<br>
             #elif defined(__i386__) && \<br>
                 (SANITIZER_LINUX && !SANITIZER_ANDROID || SANITIZER_MAC)<br>
<br>
            Modified: compiler-rt/trunk/test/lsan/li<wbr>t.common.cfg<br>
            URL:<br>
            <a href="http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/test/lsan/lit.common.cfg?rev=301048&r1=301047&r2=301048&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-pr<wbr>oject/compiler-rt/trunk/test/<wbr>lsan/lit.common.cfg?rev=301048<wbr>&r1=301047&r2=301048&view=diff</a><br>
            <<a href="http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/test/lsan/lit.common.cfg?rev=301048&r1=301047&r2=301048&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-p<wbr>roject/compiler-rt/trunk/test/<wbr>lsan/lit.common.cfg?rev=301048<wbr>&r1=301047&r2=301048&view=diff</a><wbr>><br>
            ==============================<wbr>==============================<wbr>==================<br>
            --- compiler-rt/trunk/test/lsan/li<wbr>t.common.cfg (original)<br>
            +++ compiler-rt/trunk/test/lsan/li<wbr>t.common.cfg Fri Apr 21<br>
            16:59:53 2017<br>
            @@ -67,8 +67,9 @@ config.substitutions.append( ("%clangxx<br>
             config.substitutions.append( ("%clang_lsan ",<br>
            build_invocation(clang_lsan_cf<wbr>lags)) )<br>
             config.substitutions.append( ("%clangxx_lsan ",<br>
            build_invocation(clang_lsan_cx<wbr>xflags)) )<br>
<br>
            -# LeakSanitizer tests are currently supported on x86-64<br>
            Linux, arm Linux, and mips64 Linux only.<br>
            -if config.host_os not in ['Linux'] or config.host_arch not<br>
            in ['x86_64', 'mips64', 'arm', 'armhf', 'armv7l']:<br>
            +# LeakSanitizer tests are currently supported on x86-64<br>
            Linux, PowerPC64 Linux, arm Linux, and mips64 Linux only.<br>
            +supported_linux = config.host_os is 'Linux' and<br>
            config.host_arch in ['x86_64', 'ppc64', 'mips64', 'arm',<br>
            'armhf', 'armv7l']<br>
            +if not (supported_linux):<br>
               config.unsupported = True<br>
<br>
             # Don't support Thumb due to broken fast unwinder<br>
<br>
<br>
            ______________________________<wbr>_________________<br>
            llvm-commits mailing list<br></div></div>
            <a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a> <mailto:<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llv<wbr>m.org</a>><br>
            <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/llvm-commits</a><span class=""><br>
            <<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin<wbr>/mailman/listinfo/llvm-commits</a><wbr>><br>
<br>
<br>
<br>
        --<br>
<br>
        -Bill Seurer<br>
<br>
<br>
<br>
</span></blockquote><span class="HOEnZb"><font color="#888888">
<br>
<br>
-- <br>
<br>
-Bill Seurer<br>
<br>
</font></span></blockquote></div><br></div>