[PATCH] D78644: [LSan] Enable for SystemZ

MaskRay added a comment. In D78644#4268685 , @iii wrote: > I guess the intention is dropping the special case? The following patch > passes regtests: > > --- a/compiler-rt/lib/lsan/lsan_allocator.h > +++ b/compiler-rt/lib/lsan/lsan_allocator.h > @@

[PATCH] D78644: [LSan] Enable for SystemZ

iii added a comment. I guess the intention is dropping the special case? The following patch passes regtests: --- a/compiler-rt/lib/lsan/lsan_allocator.h +++ b/compiler-rt/lib/lsan/lsan_allocator.h @@ -68,9 +68,6 @@ using PrimaryAllocator = PrimaryAllocatorASVT; # if SANITIZER_FUCHSIA

[PATCH] D78644: [LSan] Enable for SystemZ

iii added a comment. That's where the mappings normally end. Example: $ cat /proc/self/maps 2aa-2aa2000 r--p 5e:01 668061 /usr/bin/cat 2aa2000-2aa6000 r-xp 2000 5e:01 668061 /usr/bin/cat

[PATCH] D78644: [LSan] Enable for SystemZ

jonpa added a comment. Update clang/docs/LeakSanitizer.rst (Supported Platforms). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D78644/new/ https://reviews.llvm.org/D78644 ___ cfe-commits mailing list

[PATCH] D78644: [LSan] Enable for SystemZ

MaskRay added inline comments. Herald added a subscriber: Enna1. Herald added a project: All. Comment at: compiler-rt/lib/lsan/lsan_allocator.h:76 +#elif defined(__s390x__) +const uptr kAllocatorSpace = 0x400ULL; +const uptr kAllocatorSize = 0x400ULL; // 4T.

[PATCH] D78644: [LSan] Enable for SystemZ

This revision was automatically updated to reflect the committed changes. Closed by commit rGccd8b7b10347: [LSan] Enable for SystemZ (authored by iii). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D78644/new/ https://reviews.llvm.org/D78644 Files:

[PATCH] D78644: [LSan] Enable for SystemZ

uweigand accepted this revision. uweigand added a comment. This revision is now accepted and ready to land. This all looks good to me. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D78644/new/ https://reviews.llvm.org/D78644

[PATCH] D78644: [LSan] Enable for SystemZ

iii updated this revision to Diff 270771. iii added a comment. Rebased. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D78644/new/ https://reviews.llvm.org/D78644 Files: clang/lib/Driver/ToolChains/Linux.cpp compiler-rt/cmake/config-ix.cmake

[PATCH] D78644: [LSan] Enable for SystemZ

iii created this revision. iii added reviewers: vitalybuka, eugenis, uweigand, jonpa. Herald added subscribers: Sanitizers, cfe-commits, mgorny. Herald added projects: clang, Sanitizers. Add runtime support, adjust the tests and enable LSan. Repository: rG LLVM Github Monorepo