* [PATCH] rs6000: add support for sanitizers on FreeBSD
@ 2022-05-15 10:57 Piotr Kubaj
2022-05-22 21:36 ` Piotr Kubaj
0 siblings, 1 reply; 4+ messages in thread
From: Piotr Kubaj @ 2022-05-15 10:57 UTC (permalink / raw)
To: gcc-patches, segher, dje.gcc; +Cc: Piotr Kubaj
GCC's f732bf6a603721f61102a08ad2d023c7c2670870 merged LLVM's
315d792130258a9b7250494be8d002ebb427b08f, which added sanitizers support for
PowerPC on FreeBSD, so this commit only enables building it.
Enabled sanitizers are the same as on powerpc*-*-linux*.
libsanitizer
* configure.tgt: add powerpc*-*-freebsd* as supported
---
libsanitizer/ChangeLog | 4 ++++
libsanitizer/configure.tgt | 2 ++
2 files changed, 6 insertions(+)
diff --git a/libsanitizer/ChangeLog b/libsanitizer/ChangeLog
index 52050be9476..17cc395aea1 100644
--- a/libsanitizer/ChangeLog
+++ b/libsanitizer/ChangeLog
@@ -1,3 +1,7 @@
+2022-05-15 Piotr Kubaj <pkubaj@FreeBSD.org>
+
+ * configure.tgt: add powerpc*-*-freebsd*
+
2022-05-05 Martin Liska <mliska@suse.cz>
* LOCAL_PATCHES: Update.
diff --git a/libsanitizer/configure.tgt b/libsanitizer/configure.tgt
index fb89df4935c..affe8964f84 100644
--- a/libsanitizer/configure.tgt
+++ b/libsanitizer/configure.tgt
@@ -31,6 +31,8 @@ case "${target}" in
TSAN_TARGET_DEPENDENT_OBJECTS=tsan_rtl_amd64.lo
fi
;;
+ powerpc*-*-freebsd*)
+ ;;
powerpc*-*-linux*)
if test x$ac_cv_sizeof_void_p = x8; then
TSAN_SUPPORTED=yes
--
2.36.0
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] rs6000: add support for sanitizers on FreeBSD
2022-05-15 10:57 [PATCH] rs6000: add support for sanitizers on FreeBSD Piotr Kubaj
@ 2022-05-22 21:36 ` Piotr Kubaj
2022-05-23 2:39 ` Kewen.Lin
0 siblings, 1 reply; 4+ messages in thread
From: Piotr Kubaj @ 2022-05-22 21:36 UTC (permalink / raw)
To: gcc-patches, segher, dje.gcc
[-- Attachment #1: Type: text/plain, Size: 1717 bytes --]
Ping.
FreeBSD has already enabled sanitizers in its base system in https://cgit.freebsd.org/src/commit/?id=f5024381ac16ba43d37a8bd32d54c27f6a6afa66
The upstream (LLVM) patch was reviewed by a couple of people already.
Our llvm-devel port also has sanitizers enabled by default since https://cgit.freebsd.org/ports/commit/?id=70d8d9125c9f5b1da36b134408b3d580a39f1aa8
On 22-05-15 12:57:13, Piotr Kubaj wrote:
> GCC's f732bf6a603721f61102a08ad2d023c7c2670870 merged LLVM's
> 315d792130258a9b7250494be8d002ebb427b08f, which added sanitizers support for
> PowerPC on FreeBSD, so this commit only enables building it.
>
> Enabled sanitizers are the same as on powerpc*-*-linux*.
>
> libsanitizer
> * configure.tgt: add powerpc*-*-freebsd* as supported
> ---
> libsanitizer/ChangeLog | 4 ++++
> libsanitizer/configure.tgt | 2 ++
> 2 files changed, 6 insertions(+)
>
> diff --git a/libsanitizer/ChangeLog b/libsanitizer/ChangeLog
> index 52050be9476..17cc395aea1 100644
> --- a/libsanitizer/ChangeLog
> +++ b/libsanitizer/ChangeLog
> @@ -1,3 +1,7 @@
> +2022-05-15 Piotr Kubaj <pkubaj@FreeBSD.org>
> +
> + * configure.tgt: add powerpc*-*-freebsd*
> +
> 2022-05-05 Martin Liska <mliska@suse.cz>
>
> * LOCAL_PATCHES: Update.
> diff --git a/libsanitizer/configure.tgt b/libsanitizer/configure.tgt
> index fb89df4935c..affe8964f84 100644
> --- a/libsanitizer/configure.tgt
> +++ b/libsanitizer/configure.tgt
> @@ -31,6 +31,8 @@ case "${target}" in
> TSAN_TARGET_DEPENDENT_OBJECTS=tsan_rtl_amd64.lo
> fi
> ;;
> + powerpc*-*-freebsd*)
> + ;;
> powerpc*-*-linux*)
> if test x$ac_cv_sizeof_void_p = x8; then
> TSAN_SUPPORTED=yes
> --
> 2.36.0
>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] rs6000: add support for sanitizers on FreeBSD
2022-05-22 21:36 ` Piotr Kubaj
@ 2022-05-23 2:39 ` Kewen.Lin
0 siblings, 0 replies; 4+ messages in thread
From: Kewen.Lin @ 2022-05-23 2:39 UTC (permalink / raw)
To: Piotr Kubaj; +Cc: gcc-patches, segher, dje.gcc
Hi Piotr,
on 2022/5/23 05:36, Piotr Kubaj wrote:
> Ping.
>
> FreeBSD has already enabled sanitizers in its base system in https://cgit.freebsd.org/src/commit/?id=f5024381ac16ba43d37a8bd32d54c27f6a6afa66
> The upstream (LLVM) patch was reviewed by a couple of people already.
>
> Our llvm-devel port also has sanitizers enabled by default since https://cgit.freebsd.org/ports/commit/?id=70d8d9125c9f5b1da36b134408b3d580a39f1aa8
>
Thanks for the patch. I think you also need to introduce a new subtarget
ASAN_SHADOW_OFFSET for freebsd? One reference case is the
SUBTARGET_SHADOW_OFFSET in freebsd.h under i386.
BR,
Kewen
> On 22-05-15 12:57:13, Piotr Kubaj wrote:
>> GCC's f732bf6a603721f61102a08ad2d023c7c2670870 merged LLVM's
>> 315d792130258a9b7250494be8d002ebb427b08f, which added sanitizers support for
>> PowerPC on FreeBSD, so this commit only enables building it.
>>
>> Enabled sanitizers are the same as on powerpc*-*-linux*.
>>
>> libsanitizer
>> * configure.tgt: add powerpc*-*-freebsd* as supported
>> ---
>> libsanitizer/ChangeLog | 4 ++++
>> libsanitizer/configure.tgt | 2 ++
>> 2 files changed, 6 insertions(+)
>>
>> diff --git a/libsanitizer/ChangeLog b/libsanitizer/ChangeLog
>> index 52050be9476..17cc395aea1 100644
>> --- a/libsanitizer/ChangeLog
>> +++ b/libsanitizer/ChangeLog
>> @@ -1,3 +1,7 @@
>> +2022-05-15 Piotr Kubaj <pkubaj@FreeBSD.org>
>> +
>> + * configure.tgt: add powerpc*-*-freebsd*
>> +
>> 2022-05-05 Martin Liska <mliska@suse.cz>
>>
>> * LOCAL_PATCHES: Update.
>> diff --git a/libsanitizer/configure.tgt b/libsanitizer/configure.tgt
>> index fb89df4935c..affe8964f84 100644
>> --- a/libsanitizer/configure.tgt
>> +++ b/libsanitizer/configure.tgt
>> @@ -31,6 +31,8 @@ case "${target}" in
>> TSAN_TARGET_DEPENDENT_OBJECTS=tsan_rtl_amd64.lo
>> fi
>> ;;
>> + powerpc*-*-freebsd*)
>> + ;;
>> powerpc*-*-linux*)
>> if test x$ac_cv_sizeof_void_p = x8; then
>> TSAN_SUPPORTED=yes
>> --
>> 2.36.0
>>
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH] rs6000: add support for sanitizers on FreeBSD
@ 2022-05-15 9:42 Piotr Kubaj
0 siblings, 0 replies; 4+ messages in thread
From: Piotr Kubaj @ 2022-05-15 9:42 UTC (permalink / raw)
To: gcc-patches, segher, dje.gcc; +Cc: Piotr Kubaj
GCC's f732bf6a603721f61102a08ad2d023c7c2670870 merged LLVM's
315d792130258a9b7250494be8d002ebb427b08f, which added sanitizers support for
PowerPC on FreeBSD, so this commit only enables building it.
Enabled sanitizers are the same as on powerpc*-*-linux*.
libsanitizer
* configure.tgt: add powerpc*-*-freebsd* as supported
---
libsanitizer/ChangeLog | 4 ++++
libsanitizer/configure.tgt | 2 ++
2 files changed, 6 insertions(+)
diff --git a/libsanitizer/ChangeLog b/libsanitizer/ChangeLog
index 52050be9476..17cc395aea1 100644
--- a/libsanitizer/ChangeLog
+++ b/libsanitizer/ChangeLog
@@ -1,3 +1,7 @@
+2022-05-15 Piotr Kubaj <pkubaj@FreeBSD.org>
+
+ * configure.tgt: add powerpc*-*-freebsd*
+
2022-05-05 Martin Liska <mliska@suse.cz>
* LOCAL_PATCHES: Update.
diff --git a/libsanitizer/configure.tgt b/libsanitizer/configure.tgt
index fb89df4935c..affe8964f84 100644
--- a/libsanitizer/configure.tgt
+++ b/libsanitizer/configure.tgt
@@ -31,6 +31,8 @@ case "${target}" in
TSAN_TARGET_DEPENDENT_OBJECTS=tsan_rtl_amd64.lo
fi
;;
+ powerpc*-*-freebsd*)
+ ;;
powerpc*-*-linux*)
if test x$ac_cv_sizeof_void_p = x8; then
TSAN_SUPPORTED=yes
--
2.36.0
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2022-05-23 2:39 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-05-15 10:57 [PATCH] rs6000: add support for sanitizers on FreeBSD Piotr Kubaj
2022-05-22 21:36 ` Piotr Kubaj
2022-05-23 2:39 ` Kewen.Lin
-- strict thread matches above, loose matches on Subject: below --
2022-05-15 9:42 Piotr Kubaj
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).