* [PATCH, sanitizer] Wrap rethrow_primary_exception (PR 87880).
@ 2019-06-14 14:38 Iain Sandoe
2019-06-14 14:48 ` Jakub Jelinek
0 siblings, 1 reply; 8+ messages in thread
From: Iain Sandoe @ 2019-06-14 14:38 UTC (permalink / raw)
To: GCC Patches; +Cc: Jakub Jelinek
For some Darwin versions the absence of the rethrow_primary_exception
symbol causes almost all sanitizer tests to fail.
The following patch wraps it as suggested by Jakub in the PR trail, such that
if the gate is not defined, it’s assumed to be available.
OK for trunk?
Iain
libsanitizer/
2019-06-14 Iain Sandoe <iain@sandoe.co.uk>
PR libsanitizer/87880
* asan/asan_interceptors.h:
(ASAN_INTERCEPT___CXA_RETHROW_PRIMARY_EXCEPTION): New.
* asan/Makefile.am (DEFS): Add
ASAN_HAS_CXA_RETHROW_PRIMARY_EXCEPTION, defined to 0.
* asan/Makefile.in: Regenerated.
diff --git a/libsanitizer/asan/Makefile.am b/libsanitizer/asan/Makefile.am
index 867240d..b18ab2a 100644
--- a/libsanitizer/asan/Makefile.am
+++ b/libsanitizer/asan/Makefile.am
@@ -3,7 +3,7 @@ AM_CPPFLAGS = -I $(top_srcdir)/include -I $(top_srcdir)
# May be used by toolexeclibdir.
gcc_version := $(shell @get_gcc_base_ver@ $(top_srcdir)/../gcc/BASE-VER)
-DEFS = -D_GNU_SOURCE -D_DEBUG -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS -DASAN_HAS_EXCEPTIONS=1 -DASAN_NEEDS_SEGV=1 -DCAN_SANITIZE_UB=0
+DEFS = -D_GNU_SOURCE -D_DEBUG -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS -DASAN_HAS_EXCEPTIONS=1 -DASAN_NEEDS_SEGV=1 -DCAN_SANITIZE_UB=0 -DASAN_HAS_CXA_RETHROW_PRIMARY_EXCEPTION=0
if USING_MAC_INTERPOSE
DEFS += -DMAC_INTERPOSE_FUNCTIONS -DMISSING_BLOCKS_SUPPORT
endif
diff --git a/libsanitizer/asan/asan_interceptors.h b/libsanitizer/asan/asan_interceptors.h
index b599ebb..beb1dc9 100644
--- a/libsanitizer/asan/asan_interceptors.h
+++ b/libsanitizer/asan/asan_interceptors.h
@@ -79,7 +79,12 @@ void InitializePlatformInterceptors();
#if ASAN_HAS_EXCEPTIONS && !SANITIZER_WINDOWS && !SANITIZER_SOLARIS && \
!SANITIZER_NETBSD
# define ASAN_INTERCEPT___CXA_THROW 1
-# define ASAN_INTERCEPT___CXA_RETHROW_PRIMARY_EXCEPTION 1
+# if ! defined(ASAN_HAS_CXA_RETHROW_PRIMARY_EXCEPTION) \
+ || ASAN_HAS_CXA_RETHROW_PRIMARY_EXCEPTION
+# define ASAN_INTERCEPT___CXA_RETHROW_PRIMARY_EXCEPTION 1
+# else
+# define ASAN_INTERCEPT___CXA_RETHROW_PRIMARY_EXCEPTION 0
+# endif
# if defined(_GLIBCXX_SJLJ_EXCEPTIONS) || (SANITIZER_IOS && defined(__arm__))
# define ASAN_INTERCEPT__UNWIND_SJLJ_RAISEEXCEPTION 1
# else
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [PATCH, sanitizer] Wrap rethrow_primary_exception (PR 87880).
2019-06-14 14:38 [PATCH, sanitizer] Wrap rethrow_primary_exception (PR 87880) Iain Sandoe
@ 2019-06-14 14:48 ` Jakub Jelinek
2019-06-16 18:54 ` Iain Sandoe
0 siblings, 1 reply; 8+ messages in thread
From: Jakub Jelinek @ 2019-06-14 14:48 UTC (permalink / raw)
To: Iain Sandoe; +Cc: GCC Patches
On Fri, Jun 14, 2019 at 03:38:05PM +0100, Iain Sandoe wrote:
> For some Darwin versions the absence of the rethrow_primary_exception
> symbol causes almost all sanitizer tests to fail.
>
> The following patch wraps it as suggested by Jakub in the PR trail, such that
> if the gate is not defined, it’s assumed to be available.
I wonder if we shouldn't bump libasan soname because of this, as this change
is removing an exported symbol from it.
Otherwise LGTM (but bumping soname would mean it is not backportable).
> 2019-06-14 Iain Sandoe <iain@sandoe.co.uk>
>
> PR libsanitizer/87880
> * asan/asan_interceptors.h:
> (ASAN_INTERCEPT___CXA_RETHROW_PRIMARY_EXCEPTION): New.
> * asan/Makefile.am (DEFS): Add
> ASAN_HAS_CXA_RETHROW_PRIMARY_EXCEPTION, defined to 0.
> * asan/Makefile.in: Regenerated.
Jakub
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH, sanitizer] Wrap rethrow_primary_exception (PR 87880).
2019-06-14 14:48 ` Jakub Jelinek
@ 2019-06-16 18:54 ` Iain Sandoe
2019-06-16 18:58 ` Jakub Jelinek
0 siblings, 1 reply; 8+ messages in thread
From: Iain Sandoe @ 2019-06-16 18:54 UTC (permalink / raw)
To: Jakub Jelinek, Jonathan Wakely; +Cc: GCC Patches
Hi Jakub,
> On 14 Jun 2019, at 15:47, Jakub Jelinek <jakub@redhat.com> wrote:
>
> On Fri, Jun 14, 2019 at 03:38:05PM +0100, Iain Sandoe wrote:
>> For some Darwin versions the absence of the rethrow_primary_exception
>> symbol causes almost all sanitizer tests to fail.
>>
>> The following patch wraps it as suggested by Jakub in the PR trail, such that
>> if the gate is not defined, it’s assumed to be available.
>
> I wonder if we shouldn't bump libasan soname because of this, as this change
> is removing an exported symbol from it.
>
> Otherwise LGTM (but bumping soname would mean it is not backportable).
So, I guess, unless Jonathan has plans to add __cxa_rethrow_primary_exception
during the 10 time-frame, it’s correct to exclude the symbol anyway and we should
bump the so version and apply trunk.
Actually, because the way in which interposing works for Darwin is different, the only
symbol change in the library on Darwin is removing an "undefined dynamic lookup".
So, for back-ports, I can could up with some Darwin-specific Makefike change that
only adds the ASAN_HAS_CXA_RETHROW_PRIMARY_EXCEPTION=0 for Darwin.
So - OK for trunk with a bumped soname?
(and a TODO to figure a Darwin-only backport)
thanks
Iain
>
>> 2019-06-14 Iain Sandoe <iain@sandoe.co.uk>
>>
>> PR libsanitizer/87880
>> * asan/asan_interceptors.h:
>> (ASAN_INTERCEPT___CXA_RETHROW_PRIMARY_EXCEPTION): New.
>> * asan/Makefile.am (DEFS): Add
>> ASAN_HAS_CXA_RETHROW_PRIMARY_EXCEPTION, defined to 0.
>> * asan/Makefile.in: Regenerated.
>
> Jakub
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH, sanitizer] Wrap rethrow_primary_exception (PR 87880).
2019-06-16 18:54 ` Iain Sandoe
@ 2019-06-16 18:58 ` Jakub Jelinek
2019-06-17 8:07 ` Jonathan Wakely
2019-08-19 1:11 ` Iain Sandoe
0 siblings, 2 replies; 8+ messages in thread
From: Jakub Jelinek @ 2019-06-16 18:58 UTC (permalink / raw)
To: Iain Sandoe; +Cc: Jonathan Wakely, GCC Patches
On Sun, Jun 16, 2019 at 07:54:42PM +0100, Iain Sandoe wrote:
> So, I guess, unless Jonathan has plans to add __cxa_rethrow_primary_exception
> during the 10 time-frame, it’s correct to exclude the symbol anyway and we should
> bump the so version and apply trunk.
I don't understand why they've added it, it should be called
std::rethrow_exception and that is how it is called in libstdc++.
> Actually, because the way in which interposing works for Darwin is different, the only
> symbol change in the library on Darwin is removing an "undefined dynamic lookup".
> So, for back-ports, I can could up with some Darwin-specific Makefike change that
> only adds the ASAN_HAS_CXA_RETHROW_PRIMARY_EXCEPTION=0 for Darwin.
>
> So - OK for trunk with a bumped soname?
Yes.
> (and a TODO to figure a Darwin-only backport)
Yeah.
Jakub
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH, sanitizer] Wrap rethrow_primary_exception (PR 87880).
2019-06-16 18:58 ` Jakub Jelinek
@ 2019-06-17 8:07 ` Jonathan Wakely
2019-06-18 8:21 ` Iain Sandoe
2019-08-19 1:11 ` Iain Sandoe
1 sibling, 1 reply; 8+ messages in thread
From: Jonathan Wakely @ 2019-06-17 8:07 UTC (permalink / raw)
To: Jakub Jelinek; +Cc: Iain Sandoe, GCC Patches
On 16/06/19 20:58 +0200, Jakub Jelinek wrote:
>On Sun, Jun 16, 2019 at 07:54:42PM +0100, Iain Sandoe wrote:
>> So, I guess, unless Jonathan has plans to add __cxa_rethrow_primary_exception
>> during the 10 time-frame, it’s correct to exclude the symbol anyway and we should
>> bump the so version and apply trunk.
>
>I don't understand why they've added it, it should be called
>std::rethrow_exception and that is how it is called in libstdc++.
std::rethrow_exception was new in C++11, so maybe they wanted to be
able to use it in C++03 code as well? (Just a guess).
>> Actually, because the way in which interposing works for Darwin is different, the only
>> symbol change in the library on Darwin is removing an "undefined dynamic lookup".
>> So, for back-ports, I can could up with some Darwin-specific Makefike change that
>> only adds the ASAN_HAS_CXA_RETHROW_PRIMARY_EXCEPTION=0 for Darwin.
>>
>> So - OK for trunk with a bumped soname?
>
>Yes.
>
>> (and a TODO to figure a Darwin-only backport)
>
>Yeah.
>
> Jakub
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH, sanitizer] Wrap rethrow_primary_exception (PR 87880).
2019-06-17 8:07 ` Jonathan Wakely
@ 2019-06-18 8:21 ` Iain Sandoe
0 siblings, 0 replies; 8+ messages in thread
From: Iain Sandoe @ 2019-06-18 8:21 UTC (permalink / raw)
To: GCC Patches; +Cc: Jakub Jelinek
> On 17 Jun 2019, at 09:07, Jonathan Wakely <jwakely@redhat.com> wrote:
>
> On 16/06/19 20:58 +0200, Jakub Jelinek wrote:
>> On Sun, Jun 16, 2019 at 07:54:42PM +0100, Iain Sandoe wrote:
>>> So, I guess, unless Jonathan has plans to add __cxa_rethrow_primary_exception
>>> during the 10 time-frame, it’s correct to exclude the symbol anyway and we should
>>> bump the so version and apply trunk.
>>
>> I don't understand why they've added it, it should be called
>> std::rethrow_exception and that is how it is called in libstdc++.
>
> std::rethrow_exception was new in C++11, so maybe they wanted to be
> able to use it in C++03 code as well? (Just a guess).
>
>>> Actually, because the way in which interposing works for Darwin is different, the only
>>> symbol change in the library on Darwin is removing an "undefined dynamic lookup".
>>> So, for back-ports, I can could up with some Darwin-specific Makefike change that
>>> only adds the ASAN_HAS_CXA_RETHROW_PRIMARY_EXCEPTION=0 for Darwin.
>>>
>>> So - OK for trunk with a bumped soname?
>>
>> Yes.
this is what I applied.
Iain
2019-06-18 Iain Sandoe <iain@sandoe.co.uk>
PR libsanitizer/87880
* asan/asan_interceptors.h:
(ASAN_INTERCEPT___CXA_RETHROW_PRIMARY_EXCEPTION): New.
* asan/Makefile.am (DEFS): Add
ASAN_HAS_CXA_RETHROW_PRIMARY_EXCEPTION, defined to 0.
* asan/Makefile.in: Regenerated.
* asan/libtool-version: Bump version.
diff --git a/libsanitizer/asan/Makefile.am b/libsanitizer/asan/Makefile.am
index 867240d..b18ab2a 100644
--- a/libsanitizer/asan/Makefile.am
+++ b/libsanitizer/asan/Makefile.am
@@ -3,7 +3,7 @@ AM_CPPFLAGS = -I $(top_srcdir)/include -I $(top_srcdir)
# May be used by toolexeclibdir.
gcc_version := $(shell @get_gcc_base_ver@ $(top_srcdir)/../gcc/BASE-VER)
-DEFS = -D_GNU_SOURCE -D_DEBUG -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS -DASAN_HAS_EXCEPTIONS=1 -DASAN_NEEDS_SEGV=1 -DCAN_SANITIZE_UB=0
+DEFS = -D_GNU_SOURCE -D_DEBUG -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS -DASAN_HAS_EXCEPTIONS=1 -DASAN_NEEDS_SEGV=1 -DCAN_SANITIZE_UB=0 -DASAN_HAS_CXA_RETHROW_PRIMARY_EXCEPTION=0
if USING_MAC_INTERPOSE
DEFS += -DMAC_INTERPOSE_FUNCTIONS -DMISSING_BLOCKS_SUPPORT
endif
diff --git a/libsanitizer/asan/asan_interceptors.h b/libsanitizer/asan/asan_interceptors.h
index b599ebb..beb1dc9 100644
--- a/libsanitizer/asan/asan_interceptors.h
+++ b/libsanitizer/asan/asan_interceptors.h
@@ -79,7 +79,12 @@ void InitializePlatformInterceptors();
#if ASAN_HAS_EXCEPTIONS && !SANITIZER_WINDOWS && !SANITIZER_SOLARIS && \
!SANITIZER_NETBSD
# define ASAN_INTERCEPT___CXA_THROW 1
-# define ASAN_INTERCEPT___CXA_RETHROW_PRIMARY_EXCEPTION 1
+# if ! defined(ASAN_HAS_CXA_RETHROW_PRIMARY_EXCEPTION) \
+ || ASAN_HAS_CXA_RETHROW_PRIMARY_EXCEPTION
+# define ASAN_INTERCEPT___CXA_RETHROW_PRIMARY_EXCEPTION 1
+# else
+# define ASAN_INTERCEPT___CXA_RETHROW_PRIMARY_EXCEPTION 0
+# endif
# if defined(_GLIBCXX_SJLJ_EXCEPTIONS) || (SANITIZER_IOS && defined(__arm__))
# define ASAN_INTERCEPT__UNWIND_SJLJ_RAISEEXCEPTION 1
# else
diff --git a/libsanitizer/asan/libtool-version b/libsanitizer/asan/libtool-version
index e3138f3..c509757 100644
--- a/libsanitizer/asan/libtool-version
+++ b/libsanitizer/asan/libtool-version
@@ -3,4 +3,4 @@
# a separate file so that version updates don't involve re-running
# automake.
# CURRENT:REVISION:AGE
-5:0:0
+6:0:0
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH, sanitizer] Wrap rethrow_primary_exception (PR 87880).
2019-06-16 18:58 ` Jakub Jelinek
2019-06-17 8:07 ` Jonathan Wakely
@ 2019-08-19 1:11 ` Iain Sandoe
2019-08-19 1:16 ` Iain Sandoe
1 sibling, 1 reply; 8+ messages in thread
From: Iain Sandoe @ 2019-08-19 1:11 UTC (permalink / raw)
To: Jakub Jelinek; +Cc: GCC Patches
> On 16 Jun 2019, at 19:58, Jakub Jelinek <jakub@redhat.com> wrote:
>
> On Sun, Jun 16, 2019 at 07:54:42PM +0100, Iain Sandoe wrote:
>> Actually, because the way in which interposing works for Darwin is different, the only
>> symbol change in the library on Darwin is removing an "undefined dynamic lookup".
>> So, for back-ports, I can could up with some Darwin-specific Makefike change that
>> only adds the ASAN_HAS_CXA_RETHROW_PRIMARY_EXCEPTION=0 for Darwin.
>>
>> So - OK for trunk with a bumped soname?
>
> Yes.
>
>> (and a TODO to figure a Darwin-only backport)
>
> Yeah.
This turned out to be easier than I’d expected, since there was already a Darwin-specific
guard in the Makefile.
Tested on x86_64-darwin{11,13,14,16,17,18}, x86_64,powerpc64-linux-gnu
(that the symbol is not removed for the Linux cases).
Applied to the 9 branch for 9.3
Iain
libsanitizer/
2019-08-18 Iain Sandoe <iain@sandoe.co.uk>
Backport from mainline
2019-06-18 Iain Sandoe <iain@sandoe.co.uk>
PR libsanitizer/87880
* asan/asan_interceptors.h:
(ASAN_INTERCEPT___CXA_RETHROW_PRIMARY_EXCEPTION): New.
* asan/Makefile.am (DEFS): Add (for Darwin only)
ASAN_HAS_CXA_RETHROW_PRIMARY_EXCEPTION, defined to 0.
* asan/Makefile.in: Regenerated.
diff --git a/libsanitizer/asan/Makefile.am b/libsanitizer/asan/Makefile.am
index 867240d244..6efbc1df7f 100644
--- a/libsanitizer/asan/Makefile.am
+++ b/libsanitizer/asan/Makefile.am
@@ -5,7 +5,7 @@ gcc_version := $(shell @get_gcc_base_ver@ $(top_srcdir)/../gcc/BASE-VER)
DEFS = -D_GNU_SOURCE -D_DEBUG -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS -DASAN_HAS_EXCEPTIONS=1 -DASAN_NEEDS_SEGV=1 -DCAN_SANITIZE_UB=0
if USING_MAC_INTERPOSE
-DEFS += -DMAC_INTERPOSE_FUNCTIONS -DMISSING_BLOCKS_SUPPORT
+DEFS += -DMAC_INTERPOSE_FUNCTIONS -DMISSING_BLOCKS_SUPPORT -DASAN_HAS_CXA_RETHROW_PRIMARY_EXCEPTION=0
endif
AM_CXXFLAGS = -Wall -W -Wno-unused-parameter -Wwrite-strings -pedantic -Wno-long-long -fPIC -fno-builtin -fno-exceptions -fno-rtti -fomit-frame-pointer -funwind-tables -fvisibility=hidden -Wno-variadic-macros -fno-ipa-icf
AM_CXXFLAGS += $(LIBSTDCXX_RAW_CXX_CXXFLAGS)
diff --git a/libsanitizer/asan/asan_interceptors.h b/libsanitizer/asan/asan_interceptors.h
index b599ebb0ba..beb1dc9532 100644
--- a/libsanitizer/asan/asan_interceptors.h
+++ b/libsanitizer/asan/asan_interceptors.h
@@ -79,7 +79,12 @@ void InitializePlatformInterceptors();
#if ASAN_HAS_EXCEPTIONS && !SANITIZER_WINDOWS && !SANITIZER_SOLARIS && \
!SANITIZER_NETBSD
# define ASAN_INTERCEPT___CXA_THROW 1
-# define ASAN_INTERCEPT___CXA_RETHROW_PRIMARY_EXCEPTION 1
+# if ! defined(ASAN_HAS_CXA_RETHROW_PRIMARY_EXCEPTION) \
+ || ASAN_HAS_CXA_RETHROW_PRIMARY_EXCEPTION
+# define ASAN_INTERCEPT___CXA_RETHROW_PRIMARY_EXCEPTION 1
+# else
+# define ASAN_INTERCEPT___CXA_RETHROW_PRIMARY_EXCEPTION 0
+# endif
# if defined(_GLIBCXX_SJLJ_EXCEPTIONS) || (SANITIZER_IOS && defined(__arm__))
# define ASAN_INTERCEPT__UNWIND_SJLJ_RAISEEXCEPTION 1
# else
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [PATCH, sanitizer] Wrap rethrow_primary_exception (PR 87880).
2019-08-19 1:11 ` Iain Sandoe
@ 2019-08-19 1:16 ` Iain Sandoe
0 siblings, 0 replies; 8+ messages in thread
From: Iain Sandoe @ 2019-08-19 1:16 UTC (permalink / raw)
To: Jakub Jelinek; +Cc: GCC Patches
> On 16 Jun 2019, at 19:58, Jakub Jelinek <jakub@redhat.com> wrote:
>
> On Sun, Jun 16, 2019 at 07:54:42PM +0100, Iain Sandoe wrote:
>> Actually, because the way in which interposing works for Darwin is different, the only
>> symbol change in the library on Darwin is removing an "undefined dynamic lookup".
>> So, for back-ports, I can could up with some Darwin-specific Makefike change that
>> only adds the ASAN_HAS_CXA_RETHROW_PRIMARY_EXCEPTION=0 for Darwin.
>>
>> So - OK for trunk with a bumped soname?
>
> Yes.
>
>> (and a TODO to figure a Darwin-only backport)
>
> Yeah.
This turned out to be easier than I’d expected, since there was already a Darwin-specific
guard in the Makefile.
Tested on x86_64-darwin{11,13,14,16,17,18}, x86_64,powerpc64-linux-gnu
(that the symbol is not removed for the Linux cases).
Applied to the 9 branch for 9.3
Iain
libsanitizer/
2019-08-18 Iain Sandoe <iain@sandoe.co.uk>
Backport from mainline
2019-06-18 Iain Sandoe <iain@sandoe.co.uk>
PR libsanitizer/87880
* asan/asan_interceptors.h:
(ASAN_INTERCEPT___CXA_RETHROW_PRIMARY_EXCEPTION): New.
* asan/Makefile.am (DEFS): Add (for Darwin only)
ASAN_HAS_CXA_RETHROW_PRIMARY_EXCEPTION, defined to 0.
* asan/Makefile.in: Regenerated.
diff --git a/libsanitizer/asan/Makefile.am b/libsanitizer/asan/Makefile.am
index 867240d244..6efbc1df7f 100644
--- a/libsanitizer/asan/Makefile.am
+++ b/libsanitizer/asan/Makefile.am
@@ -5,7 +5,7 @@ gcc_version := $(shell @get_gcc_base_ver@ $(top_srcdir)/../gcc/BASE-VER)
DEFS = -D_GNU_SOURCE -D_DEBUG -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS -DASAN_HAS_EXCEPTIONS=1 -DASAN_NEEDS_SEGV=1 -DCAN_SANITIZE_UB=0
if USING_MAC_INTERPOSE
-DEFS += -DMAC_INTERPOSE_FUNCTIONS -DMISSING_BLOCKS_SUPPORT
+DEFS += -DMAC_INTERPOSE_FUNCTIONS -DMISSING_BLOCKS_SUPPORT -DASAN_HAS_CXA_RETHROW_PRIMARY_EXCEPTION=0
endif
AM_CXXFLAGS = -Wall -W -Wno-unused-parameter -Wwrite-strings -pedantic -Wno-long-long -fPIC -fno-builtin -fno-exceptions -fno-rtti -fomit-frame-pointer -funwind-tables -fvisibility=hidden -Wno-variadic-macros -fno-ipa-icf
AM_CXXFLAGS += $(LIBSTDCXX_RAW_CXX_CXXFLAGS)
diff --git a/libsanitizer/asan/asan_interceptors.h b/libsanitizer/asan/asan_interceptors.h
index b599ebb0ba..beb1dc9532 100644
--- a/libsanitizer/asan/asan_interceptors.h
+++ b/libsanitizer/asan/asan_interceptors.h
@@ -79,7 +79,12 @@ void InitializePlatformInterceptors();
#if ASAN_HAS_EXCEPTIONS && !SANITIZER_WINDOWS && !SANITIZER_SOLARIS && \
!SANITIZER_NETBSD
# define ASAN_INTERCEPT___CXA_THROW 1
-# define ASAN_INTERCEPT___CXA_RETHROW_PRIMARY_EXCEPTION 1
+# if ! defined(ASAN_HAS_CXA_RETHROW_PRIMARY_EXCEPTION) \
+ || ASAN_HAS_CXA_RETHROW_PRIMARY_EXCEPTION
+# define ASAN_INTERCEPT___CXA_RETHROW_PRIMARY_EXCEPTION 1
+# else
+# define ASAN_INTERCEPT___CXA_RETHROW_PRIMARY_EXCEPTION 0
+# endif
# if defined(_GLIBCXX_SJLJ_EXCEPTIONS) || (SANITIZER_IOS && defined(__arm__))
# define ASAN_INTERCEPT__UNWIND_SJLJ_RAISEEXCEPTION 1
# else
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2019-08-18 19:20 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-06-14 14:38 [PATCH, sanitizer] Wrap rethrow_primary_exception (PR 87880) Iain Sandoe
2019-06-14 14:48 ` Jakub Jelinek
2019-06-16 18:54 ` Iain Sandoe
2019-06-16 18:58 ` Jakub Jelinek
2019-06-17 8:07 ` Jonathan Wakely
2019-06-18 8:21 ` Iain Sandoe
2019-08-19 1:11 ` Iain Sandoe
2019-08-19 1:16 ` Iain Sandoe
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).