<div dir="ltr">I'll look into it.</div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Apr 25, 2017 at 9:05 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">This is causing several test case failures on ppc64le:<br>
<br>
<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><div class="HOEnZb"><div class="h5"><br>
<br>
On 04/21/2017 04:59 PM, Alex Shlyapnikov via llvm-commits wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
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>
Log:<br>
[lsan] Enable LSan on PowerPC64.<br>
<br>
Summary: Re-landing reverted D31995 with suppressions 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>
<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: <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>
==============================<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 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, ByteMap><br>
     PrimaryAllocator;<br>
-#elif defined(__x86_64__)<br>
+#elif defined(__x86_64__) || defined(__powerpc64__)<br>
 struct AP64 {  // Allocator64 parameters. Deliberately 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: <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>
==============================<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 16:59:53 2017<br>
@@ -32,7 +32,8 @@<br>
 // new architecture inside sanitizer library.<br>
 #if (SANITIZER_LINUX && !SANITIZER_ANDROID || SANITIZER_MAC) && \<br>
     (SANITIZER_WORDSIZE == 64) &&                               \<br>
-    (defined(__x86_64__) || defined(__mips64) || defined(__aarch64__))<br>
+    (defined(__x86_64__) || defined(__mips64) || 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: <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>
==============================<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 16:59:53 2017<br>
@@ -67,8 +67,9 @@ config.substitutions.append( ("%clangxx<br>
 config.substitutions.append( ("%clang_lsan ", build_invocation(clang_lsan_cf<wbr>lags)) )<br>
 config.substitutions.append( ("%clangxx_lsan ", build_invocation(clang_lsan_cx<wbr>xflags)) )<br>
<br>
-# LeakSanitizer tests are currently supported on x86-64 Linux, arm Linux, and mips64 Linux only.<br>
-if config.host_os not in ['Linux'] or config.host_arch not in ['x86_64', 'mips64', 'arm', 'armhf', 'armv7l']:<br>
+# LeakSanitizer tests are currently supported on x86-64 Linux, PowerPC64 Linux, arm Linux, and mips64 Linux only.<br>
+supported_linux = config.host_os is 'Linux' and config.host_arch in ['x86_64', 'ppc64', 'mips64', 'arm', '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>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.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><br>
<br>
</blockquote>
<br>
<br></div></div><span class="HOEnZb"><font color="#888888">
-- <br>
<br>
-Bill Seurer<br>
<br>
</font></span></blockquote></div><br></div>