This is an archive of the discontinued LLVM Phabricator instance.

Differential D108171

[sanitizer] Add hexagon support to lsan
AcceptedPublic

Authored by bcain on Aug 16 2021, 5:08 PM.

Details

Reviewers
vitalybuka
Summary

Adds build support for hexagon linux to lsan.

Diff Detail

Event Timeline

bcain created this revision.Aug 16 2021, 5:08 PM
Herald added subscribers: s.egerton, simoncook, mgorny. · View Herald TranscriptAug 16 2021, 5:08 PM
bcain requested review of this revision.Aug 16 2021, 5:08 PM
Herald added a subscriber: Restricted Project. · View Herald TranscriptAug 16 2021, 5:08 PM
bcain retitled this revision from [sanitizer] Add hexagon support to lsan to [PATCH 2/8] [sanitizer] Add hexagon support to lsan.Aug 16 2021, 5:50 PM
Harbormaster completed remote builds in B119806: Diff 366767.Aug 16 2021, 5:55 PM
vitalybuka accepted this revision.Aug 17 2021, 10:17 AM
vitalybuka added inline comments.
compiler-rt/cmake/Modules/AllSupportedArchDefs.cmake
52
This revision is now accepted and ready to land.Aug 17 2021, 10:17 AM
bcain updated this revision to Diff 366986.Aug 17 2021, 12:52 PM
bcain retitled this revision from [PATCH 2/8] [sanitizer] Add hexagon support to lsan to [sanitizer] Add hexagon support to lsan.

Line-wrap update

Harbormaster completed remote builds in B119961: Diff 366986.Aug 17 2021, 1:22 PM

Revision Contents

PathSize
compiler-rt/
cmake/
Modules/
3 lines
lib/
lsan/
2 lines

Diff 366986

compiler-rt/cmake/Modules/AllSupportedArchDefs.cmake

Context not available.
if(APPLE) if(APPLE)
set(ALL_LSAN_SUPPORTED_ARCH ${X86} ${X86_64} ${MIPS64} ${ARM64}) set(ALL_LSAN_SUPPORTED_ARCH ${X86} ${X86_64} ${MIPS64} ${ARM64})
else() else()
vitalybukaUnsubmitted
Not Done
ReplyInline Actions
else()
- set(ALL_LSAN_SUPPORTED_ARCH ${X86} ${X86_64} ${MIPS64} ${ARM64} ${ARM32} ${PPC64} ${S390X} ${RISCV64} ${HEXAGON})
- endif()
+ set(ALL_LSAN_SUPPORTED_ARCH ${X86} ${X86_64} ${MIPS64} ${ARM64} ${ARM32}
+ ${PPC64} ${S390X} ${RISCV64} ${HEXAGON})endif()
vitalybuka:
set(ALL_LSAN_SUPPORTED_ARCH ${X86} ${X86_64} ${MIPS64} ${ARM64} ${ARM32} ${PPC64} ${S390X} ${RISCV64}) set(ALL_LSAN_SUPPORTED_ARCH ${X86} ${X86_64} ${MIPS64} ${ARM64} ${ARM32}
${PPC64} ${S390X} ${RISCV64} ${HEXAGON})
endif() endif()
set(ALL_MSAN_SUPPORTED_ARCH ${X86_64} ${MIPS64} ${ARM64} ${PPC64} ${S390X}) set(ALL_MSAN_SUPPORTED_ARCH ${X86_64} ${MIPS64} ${ARM64} ${PPC64} ${S390X})
set(ALL_HWASAN_SUPPORTED_ARCH ${X86_64} ${ARM64}) set(ALL_HWASAN_SUPPORTED_ARCH ${X86_64} ${ARM64})
Context not available.

compiler-rt/lib/lsan/lsan_allocator.h

Context not available.
}; };
#if defined(__mips64) || defined(__aarch64__) || defined(__i386__) || \ #if defined(__mips64) || defined(__aarch64__) || defined(__i386__) || \
defined(__arm__) || SANITIZER_RISCV64 defined(__arm__) || SANITIZER_RISCV64 || defined(__hexagon__)
template <typename AddressSpaceViewTy> template <typename AddressSpaceViewTy>
struct AP32 { struct AP32 {
static const uptr kSpaceBeg = 0; static const uptr kSpaceBeg = 0;
Context not available.