[PATCH] D100907: [lsan][docs] Clarify supported platforms
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.org/D100907 Files: clang/docs/LeakSanitizer.rst Index: clang/docs/LeakSanitizer.rst === --- clang/docs/LeakSanitizer.rst +++ clang/docs/LeakSanitizer.rst @@ -17,8 +17,8 @@ Usage = -LeakSanitizer is supported on x86\_64 Linux and macOS. In order to use it, -simply build your program with :doc:`AddressSanitizer`: +:doc:`AddressSanitizer`: integrates LeakSanitizer and enables it by default on +supported platforms. .. code-block:: console @@ -43,6 +43,15 @@ link step, so that it would link in proper LeakSanitizer run-time library into the final executable. +Supported Platforms +=== + +* Android aarch64/i386/x86_64 +* Fuchsia aarch64/x86_64 +* Linux arm/aarch64/mips64/ppc64/ppc64le/riscv64/s390x/i386/x86\_64 +* macOS aarch64/i386/x86\_64 +* NetBSD i386/x86_64 + More Information Index: clang/docs/LeakSanitizer.rst === --- clang/docs/LeakSanitizer.rst +++ clang/docs/LeakSanitizer.rst @@ -17,8 +17,8 @@ Usage = -LeakSanitizer is supported on x86\_64 Linux and macOS. In order to use it, -simply build your program with :doc:`AddressSanitizer`: +:doc:`AddressSanitizer`: integrates LeakSanitizer and enables it by default on +supported platforms. .. code-block:: console @@ -43,6 +43,15 @@ link step, so that it would link in proper LeakSanitizer run-time library into the final executable. +Supported Platforms +=== + +* Android aarch64/i386/x86_64 +* Fuchsia aarch64/x86_64 +* Linux arm/aarch64/mips64/ppc64/ppc64le/riscv64/s390x/i386/x86\_64 +* macOS aarch64/i386/x86\_64 +* NetBSD i386/x86_64 + More Information ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D100907: [lsan][docs] Clarify supported platforms
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
vitalybuka wrote:
> MaskRay wrote:
> > vitalybuka wrote:
> > > Apple is incomplete and riscv is missing
> > >
> > > ```
> > > if(APPLE)
> > > set(ALL_LSAN_SUPPORTED_ARCH ${X86} ${X86_64} ${MIPS64} ${ARM64})
> > > else()
> > > set(ALL_LSAN_SUPPORTED_ARCH ${X86} ${X86_64} ${MIPS64} ${ARM64}
> > > ${ARM32} ${PPC64} ${S390X} ${RISCV64})
> > > endif()
> > > ```
> > I'll add riscv64 for Linux.
> >
> > But wait, lib/sanitizer_common/sanitizer_stoptheworld_mac.cpp doesn't
> > mention mips64...
> maybe work in progress
Likely a copy-paste error from de3b9a2ecc7e9ebe3fc1780aede0223821c52d2a
I can delete it.
CC @delcypher @fjricci who have touched the macOS code.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D100907/new/
https://reviews.llvm.org/D100907
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D100907: [lsan][docs] Clarify supported platforms
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 wrote:
> vitalybuka wrote:
> > Apple is incomplete and riscv is missing
> >
> > ```
> > if(APPLE)
> > set(ALL_LSAN_SUPPORTED_ARCH ${X86} ${X86_64} ${MIPS64} ${ARM64})
> > else()
> > set(ALL_LSAN_SUPPORTED_ARCH ${X86} ${X86_64} ${MIPS64} ${ARM64} ${ARM32}
> > ${PPC64} ${S390X} ${RISCV64})
> > endif()
> > ```
> I'll add riscv64 for Linux.
>
> But wait, lib/sanitizer_common/sanitizer_stoptheworld_mac.cpp doesn't mention
> mips64...
maybe work in progress
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D100907/new/
https://reviews.llvm.org/D100907
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D100907: [lsan][docs] Clarify supported platforms
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: clang/docs/LeakSanitizer.rst === --- clang/docs/LeakSanitizer.rst +++ clang/docs/LeakSanitizer.rst @@ -17,8 +17,8 @@ Usage = -LeakSanitizer is supported on x86\_64 Linux and macOS. In order to use it, -simply build your program with :doc:`AddressSanitizer`: +:doc:`AddressSanitizer`: integrates LeakSanitizer and enables it by default on +supported platforms. .. code-block:: console @@ -43,6 +43,15 @@ link step, so that it would link in proper LeakSanitizer run-time library into the final executable. +Supported Platforms +=== + +* Android aarch64/i386/x86_64 +* Fuchsia aarch64/x86_64 +* Linux arm/aarch64/mips64/ppc64/ppc64le/riscv64/s390x/i386/x86\_64 +* macOS aarch64/i386/x86\_64 +* NetBSD i386/x86_64 + More Information Index: clang/docs/LeakSanitizer.rst === --- clang/docs/LeakSanitizer.rst +++ clang/docs/LeakSanitizer.rst @@ -17,8 +17,8 @@ Usage = -LeakSanitizer is supported on x86\_64 Linux and macOS. In order to use it, -simply build your program with :doc:`AddressSanitizer`: +:doc:`AddressSanitizer`: integrates LeakSanitizer and enables it by default on +supported platforms. .. code-block:: console @@ -43,6 +43,15 @@ link step, so that it would link in proper LeakSanitizer run-time library into the final executable. +Supported Platforms +=== + +* Android aarch64/i386/x86_64 +* Fuchsia aarch64/x86_64 +* Linux arm/aarch64/mips64/ppc64/ppc64le/riscv64/s390x/i386/x86\_64 +* macOS aarch64/i386/x86\_64 +* NetBSD i386/x86_64 + More Information ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D100907: [lsan][docs] Clarify supported platforms
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 LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D100907/new/ https://reviews.llvm.org/D100907 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D100907: [lsan][docs] Clarify supported platforms
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 Files: clang/docs/LeakSanitizer.rst Index: clang/docs/LeakSanitizer.rst === --- clang/docs/LeakSanitizer.rst +++ clang/docs/LeakSanitizer.rst @@ -17,8 +17,8 @@ Usage = -LeakSanitizer is supported on x86\_64 Linux and macOS. In order to use it, -simply build your program with :doc:`AddressSanitizer`: +:doc:`AddressSanitizer`: integrates LeakSanitizer and enables it by default on +supported platforms. .. code-block:: console @@ -43,6 +43,15 @@ link step, so that it would link in proper LeakSanitizer run-time library into the final executable. +Supported Platforms +=== + +* Android aarch64/i386/x86_64 +* Fuchsia +* Linux arm/aarch64/mips64/ppc64/ppc64le/riscv64/s390x/i386/x86\_64 +* macOS aarch64/i386/x86\_64 +* NetBSD i386/x86_64 + More Information Index: clang/docs/LeakSanitizer.rst === --- clang/docs/LeakSanitizer.rst +++ clang/docs/LeakSanitizer.rst @@ -17,8 +17,8 @@ Usage = -LeakSanitizer is supported on x86\_64 Linux and macOS. In order to use it, -simply build your program with :doc:`AddressSanitizer`: +:doc:`AddressSanitizer`: integrates LeakSanitizer and enables it by default on +supported platforms. .. code-block:: console @@ -43,6 +43,15 @@ link step, so that it would link in proper LeakSanitizer run-time library into the final executable. +Supported Platforms +=== + +* Android aarch64/i386/x86_64 +* Fuchsia +* Linux arm/aarch64/mips64/ppc64/ppc64le/riscv64/s390x/i386/x86\_64 +* macOS aarch64/i386/x86\_64 +* NetBSD i386/x86_64 + More Information ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D100907: [lsan][docs] Clarify supported platforms
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_LSAN_SUPPORTED_ARCH ${X86} ${X86_64} ${MIPS64} ${ARM64})
> else()
> set(ALL_LSAN_SUPPORTED_ARCH ${X86} ${X86_64} ${MIPS64} ${ARM64} ${ARM32}
> ${PPC64} ${S390X} ${RISCV64})
> endif()
> ```
I'll add riscv64 for Linux.
But wait, lib/sanitizer_common/sanitizer_stoptheworld_mac.cpp doesn't mention
mips64...
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D100907/new/
https://reviews.llvm.org/D100907
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D100907: [lsan][docs] Clarify supported platforms
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} ${X86_64} ${MIPS64} ${ARM64})
else()
set(ALL_LSAN_SUPPORTED_ARCH ${X86} ${X86_64} ${MIPS64} ${ARM64} ${ARM32}
${PPC64} ${S390X} ${RISCV64})
endif()
```
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D100907/new/
https://reviews.llvm.org/D100907
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D100907: [lsan][docs] Clarify supported platforms
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/LeakSanitizer.rst === --- clang/docs/LeakSanitizer.rst +++ clang/docs/LeakSanitizer.rst @@ -17,8 +17,8 @@ Usage = -LeakSanitizer is supported on x86\_64 Linux and macOS. In order to use it, -simply build your program with :doc:`AddressSanitizer`: +:doc:`AddressSanitizer`: integrates LeakSanitizer and enables it by default on +supported platforms. .. code-block:: console @@ -43,6 +43,14 @@ link step, so that it would link in proper LeakSanitizer run-time library into the final executable. +Supported Platforms +=== + +* Android aarch64/i386/x86_64 +* Linux arm/aarch64/mips64/ppc64/ppc64le/s390x/i386/x86\_64 +* macOS aarch64/i386/x86\_64 +* NetBSD i386/x86_64 + More Information Index: clang/docs/LeakSanitizer.rst === --- clang/docs/LeakSanitizer.rst +++ clang/docs/LeakSanitizer.rst @@ -17,8 +17,8 @@ Usage = -LeakSanitizer is supported on x86\_64 Linux and macOS. In order to use it, -simply build your program with :doc:`AddressSanitizer`: +:doc:`AddressSanitizer`: integrates LeakSanitizer and enables it by default on +supported platforms. .. code-block:: console @@ -43,6 +43,14 @@ link step, so that it would link in proper LeakSanitizer run-time library into the final executable. +Supported Platforms +=== + +* Android aarch64/i386/x86_64 +* Linux arm/aarch64/mips64/ppc64/ppc64le/s390x/i386/x86\_64 +* macOS aarch64/i386/x86\_64 +* NetBSD i386/x86_64 + More Information ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D100907: [lsan][docs] Clarify supported platforms
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/LeakSanitizer.rst Index: clang/docs/LeakSanitizer.rst === --- clang/docs/LeakSanitizer.rst +++ clang/docs/LeakSanitizer.rst @@ -17,8 +17,8 @@ Usage = -LeakSanitizer is supported on x86\_64 Linux and macOS. In order to use it, -simply build your program with :doc:`AddressSanitizer`: +:doc:`AddressSanitizer`: integrates LeakSanitizer and enables it by default on +supported platforms. .. code-block:: console @@ -43,6 +43,14 @@ link step, so that it would link in proper LeakSanitizer run-time library into the final executable. +Supported Platforms +=== + +* Android aarch64/i386/x86_64 +* Linux arm/aarch64/mips64/ppc64/ppc64le/s390x/i386/x86\_64 +* macOS x86\_64 +* NetBSD i386/x86_64 + More Information Index: clang/docs/LeakSanitizer.rst === --- clang/docs/LeakSanitizer.rst +++ clang/docs/LeakSanitizer.rst @@ -17,8 +17,8 @@ Usage = -LeakSanitizer is supported on x86\_64 Linux and macOS. In order to use it, -simply build your program with :doc:`AddressSanitizer`: +:doc:`AddressSanitizer`: integrates LeakSanitizer and enables it by default on +supported platforms. .. code-block:: console @@ -43,6 +43,14 @@ link step, so that it would link in proper LeakSanitizer run-time library into the final executable. +Supported Platforms +=== + +* Android aarch64/i386/x86_64 +* Linux arm/aarch64/mips64/ppc64/ppc64le/s390x/i386/x86\_64 +* macOS x86\_64 +* NetBSD i386/x86_64 + More Information ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
