[PATCH] D100907: [lsan][docs] Clarify supported platforms

2021-04-21 Thread Fangrui Song via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG4cbe4881882e: [lsan][docs] Clarify supported platforms (authored by MaskRay). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D100907/new/ https://reviews.llvm

[PATCH] D100907: [lsan][docs] Clarify supported platforms

2021-04-20 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay marked an inline comment as done. MaskRay added subscribers: fjricci, delcypher. MaskRay added inline comments. Comment at: clang/docs/LeakSanitizer.rst:49 + +* Android aarch64/i386/x86_64 +* Linux arm/aarch64/mips64/ppc64/ppc64le/s390x/i386/x86\_64 vital

[PATCH] D100907: [lsan][docs] Clarify supported platforms

2021-04-20 Thread Vitaly Buka via Phabricator via cfe-commits
vitalybuka accepted this revision. vitalybuka added inline comments. This revision is now accepted and ready to land. Comment at: clang/docs/LeakSanitizer.rst:49 + +* Android aarch64/i386/x86_64 +* Linux arm/aarch64/mips64/ppc64/ppc64le/s390x/i386/x86\_64 MaskRay

[PATCH] D100907: [lsan][docs] Clarify supported platforms

2021-04-20 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay updated this revision to Diff 339045. MaskRay added a comment. Fuchsia -> Fuchsia aarch64/x86_64 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D100907/new/ https://reviews.llvm.org/D100907 Files: clang/docs/LeakSanitizer.rst Index: clan

[PATCH] D100907: [lsan][docs] Clarify supported platforms

2021-04-20 Thread Petr Hosek via Phabricator via cfe-commits
phosek added inline comments. Comment at: clang/docs/LeakSanitizer.rst:50 +* Android aarch64/i386/x86_64 +* Fuchsia +* Linux arm/aarch64/mips64/ppc64/ppc64le/riscv64/s390x/i386/x86\_64 You can clarify that Fuchsia supports aarch64 and x86_64. Repository: rG L

[PATCH] D100907: [lsan][docs] Clarify supported platforms

2021-04-20 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay updated this revision to Diff 339039. MaskRay added a comment. Herald added subscribers: phosek, s.egerton, simoncook. Add Linux riscv64. Add Fuchsia Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D100907/new/ https://reviews.llvm.org/D100907

[PATCH] D100907: [lsan][docs] Clarify supported platforms

2021-04-20 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay added inline comments. Comment at: clang/docs/LeakSanitizer.rst:49 + +* Android aarch64/i386/x86_64 +* Linux arm/aarch64/mips64/ppc64/ppc64le/s390x/i386/x86\_64 vitalybuka wrote: > Apple is incomplete and riscv is missing > > ``` > if(APPLE) > set(ALL_

[PATCH] D100907: [lsan][docs] Clarify supported platforms

2021-04-20 Thread Vitaly Buka via Phabricator via cfe-commits
vitalybuka added inline comments. Comment at: clang/docs/LeakSanitizer.rst:49 + +* Android aarch64/i386/x86_64 +* Linux arm/aarch64/mips64/ppc64/ppc64le/s390x/i386/x86\_64 Apple is incomplete and riscv is missing ``` if(APPLE) set(ALL_LSAN_SUPPORTED_ARCH ${X86

[PATCH] D100907: [lsan][docs] Clarify supported platforms

2021-04-20 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay updated this revision to Diff 339032. MaskRay added a comment. add i386/aarch64 to macOS Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D100907/new/ https://reviews.llvm.org/D100907 Files: clang/docs/LeakSanitizer.rst Index: clang/docs/L

[PATCH] D100907: [lsan][docs] Clarify supported platforms

2021-04-20 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay created this revision. MaskRay added reviewers: Sanitizers, vitalybuka. MaskRay requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D100907 Files: clang/docs/LeakSanitiz