[compiler-rt] r301048 - [lsan] Enable LSan on PowerPC64.
Aleksey Shlyapnikov via llvm-commits
llvm-commits at lists.llvm.org
Wed Apr 26 10:27:22 PDT 2017
Sorry for the breakage, D32545 should fix that.
On Wed, Apr 26, 2017 at 8:10 AM, Bill Seurer <seurer at linux.vnet.ibm.com>
wrote:
> It looks like all the failing tests now succeed except
>
> FAIL: AddressSanitizer-powerpc64le-linux::stress_dtls.c
>
> This one also started failing with 301048.
>
>
>
> On 04/25/2017 12:38 PM, Aleksey Shlyapnikov wrote:
>
>> https://reviews.llvm.org/rL301335 should take care of it.
>>
>> On Tue, Apr 25, 2017 at 10:16 AM, Aleksey Shlyapnikov
>> <alekseys at google.com <mailto:alekseys at google.com>> wrote:
>>
>> I'll look into it.
>>
>> On Tue, Apr 25, 2017 at 9:05 AM, Bill Seurer
>> <seurer at linux.vnet.ibm.com <mailto:seurer at linux.vnet.ibm.com>> wrote:
>>
>> This is causing several test case failures on ppc64le:
>>
>> http://lab.llvm.org:8011/builders/clang-ppc64le-linux/builds/5614
>> <http://lab.llvm.org:8011/builders/clang-ppc64le-linux/builds/5614>
>>
>>
>>
>> On 04/21/2017 04:59 PM, Alex Shlyapnikov via llvm-commits wrote:
>>
>> Author: alekseyshl
>> Date: Fri Apr 21 16:59:53 2017
>> New Revision: 301048
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=301048&view=rev
>> <http://llvm.org/viewvc/llvm-project?rev=301048&view=rev>
>> Log:
>> [lsan] Enable LSan on PowerPC64.
>>
>> Summary: Re-landing reverted D31995 with suppressions
>> defined in D32303 and D32377.
>>
>> Reviewers: eugenis
>>
>> Subscribers: nemanjai, llvm-commits
>>
>> Differential Revision: https://reviews.llvm.org/D32379
>> <https://reviews.llvm.org/D32379>
>>
>> Modified:
>> compiler-rt/trunk/lib/lsan/lsan_allocator.h
>> compiler-rt/trunk/lib/lsan/lsan_common.h
>> compiler-rt/trunk/test/lsan/lit.common.cfg
>>
>> Modified: compiler-rt/trunk/lib/lsan/lsan_allocator.h
>> URL:
>> http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/ls
>> an/lsan_allocator.h?rev=301048&r1=301047&r2=301048&view=diff
>> <http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/l
>> san/lsan_allocator.h?rev=301048&r1=301047&r2=301048&view=diff>
>> ============================================================
>> ==================
>> --- compiler-rt/trunk/lib/lsan/lsan_allocator.h (original)
>> +++ compiler-rt/trunk/lib/lsan/lsan_allocator.h Fri Apr 21
>> 16:59:53 2017
>> @@ -59,7 +59,7 @@ typedef CompactSizeClassMap SizeClassMap
>> typedef SizeClassAllocator32<0, SANITIZER_MMAP_RANGE_SIZE,
>> sizeof(ChunkMetadata), SizeClassMap, kRegionSizeLog,
>> ByteMap>
>> PrimaryAllocator;
>> -#elif defined(__x86_64__)
>> +#elif defined(__x86_64__) || defined(__powerpc64__)
>> struct AP64 { // Allocator64 parameters. Deliberately
>> using a short name.
>> static const uptr kSpaceBeg = 0x600000000000ULL;
>> static const uptr kSpaceSize = 0x40000000000ULL; // 4T.
>>
>> Modified: compiler-rt/trunk/lib/lsan/lsan_common.h
>> URL:
>> http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/ls
>> an/lsan_common.h?rev=301048&r1=301047&r2=301048&view=diff
>> <http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/l
>> san/lsan_common.h?rev=301048&r1=301047&r2=301048&view=diff>
>> ============================================================
>> ==================
>> --- compiler-rt/trunk/lib/lsan/lsan_common.h (original)
>> +++ compiler-rt/trunk/lib/lsan/lsan_common.h Fri Apr 21
>> 16:59:53 2017
>> @@ -32,7 +32,8 @@
>> // new architecture inside sanitizer library.
>> #if (SANITIZER_LINUX && !SANITIZER_ANDROID ||
>> SANITIZER_MAC) && \
>> (SANITIZER_WORDSIZE == 64) &&
>> \
>> - (defined(__x86_64__) || defined(__mips64) ||
>> defined(__aarch64__))
>> + (defined(__x86_64__) || defined(__mips64) ||
>> defined(__aarch64__) || \
>> + defined(__powerpc64__))
>> #define CAN_SANITIZE_LEAKS 1
>> #elif defined(__i386__) && \
>> (SANITIZER_LINUX && !SANITIZER_ANDROID || SANITIZER_MAC)
>>
>> Modified: compiler-rt/trunk/test/lsan/lit.common.cfg
>> URL:
>> http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/test/
>> lsan/lit.common.cfg?rev=301048&r1=301047&r2=301048&view=diff
>> <http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/test/
>> lsan/lit.common.cfg?rev=301048&r1=301047&r2=301048&view=diff>
>> ============================================================
>> ==================
>> --- compiler-rt/trunk/test/lsan/lit.common.cfg (original)
>> +++ compiler-rt/trunk/test/lsan/lit.common.cfg Fri Apr 21
>> 16:59:53 2017
>> @@ -67,8 +67,9 @@ config.substitutions.append( ("%clangxx
>> config.substitutions.append( ("%clang_lsan ",
>> build_invocation(clang_lsan_cflags)) )
>> config.substitutions.append( ("%clangxx_lsan ",
>> build_invocation(clang_lsan_cxxflags)) )
>>
>> -# LeakSanitizer tests are currently supported on x86-64
>> Linux, arm Linux, and mips64 Linux only.
>> -if config.host_os not in ['Linux'] or config.host_arch not
>> in ['x86_64', 'mips64', 'arm', 'armhf', 'armv7l']:
>> +# LeakSanitizer tests are currently supported on x86-64
>> Linux, PowerPC64 Linux, arm Linux, and mips64 Linux only.
>> +supported_linux = config.host_os is 'Linux' and
>> config.host_arch in ['x86_64', 'ppc64', 'mips64', 'arm',
>> 'armhf', 'armv7l']
>> +if not (supported_linux):
>> config.unsupported = True
>>
>> # Don't support Thumb due to broken fast unwinder
>>
>>
>> _______________________________________________
>> llvm-commits mailing list
>> llvm-commits at lists.llvm.org <mailto:llvm-commits at lists.llv
>> m.org>
>> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>> <http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits>
>>
>>
>>
>> --
>>
>> -Bill Seurer
>>
>>
>>
>>
>
> --
>
> -Bill Seurer
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170426/d175948b/attachment.html>
More information about the llvm-commits
mailing list