Skip to content

Commit

Permalink
[libcxxabi] Use the right calling convention for exception destructor…
Browse files Browse the repository at this point in the history
…s on i386 Windows

On Windows on i386, C++ member functions use a different calling
convention (`__thiscall`) than the default one for regular functions
(`__cdecl`). (On Windows on architectures other than i386, both calling
convention attributes are no-ops.)

This matches how libstdc++ declares these types.

This fixes the std/thread/futures/futures.{shared,unique}_future/dtor.pass.cpp
tests on i386 mingw.

Differential Revision: https://reviews.llvm.org/D124990
  • Loading branch information
mstorsjo committed May 5, 2022
1 parent 448eabd commit aeb4907
Show file tree
Hide file tree
Showing 4 changed files with 10 additions and 4 deletions.
6 changes: 6 additions & 0 deletions libcxxabi/include/__cxxabi_config.h
Expand Up @@ -97,4 +97,10 @@
# define _LIBCXXABI_NO_EXCEPTIONS
#endif

#if defined(_WIN32)
#define _LIBCXXABI_DTOR_FUNC __thiscall
#else
#define _LIBCXXABI_DTOR_FUNC
#endif

#endif // ____CXXABI_CONFIG_H
2 changes: 1 addition & 1 deletion libcxxabi/include/cxxabi.h
Expand Up @@ -47,7 +47,7 @@ __cxa_free_exception(void *thrown_exception) throw();
// 2.4.3 Throwing the Exception Object
extern _LIBCXXABI_FUNC_VIS _LIBCXXABI_NORETURN void
__cxa_throw(void *thrown_exception, std::type_info *tinfo,
void (*dest)(void *));
void (_LIBCXXABI_DTOR_FUNC *dest)(void *));

// 2.5.3 Exception Handlers
extern _LIBCXXABI_FUNC_VIS void *
Expand Down
2 changes: 1 addition & 1 deletion libcxxabi/src/cxa_exception.cpp
Expand Up @@ -254,7 +254,7 @@ will call terminate, assuming that there was no handler for the
exception.
*/
void
__cxa_throw(void *thrown_object, std::type_info *tinfo, void (*dest)(void *)) {
__cxa_throw(void *thrown_object, std::type_info *tinfo, void (_LIBCXXABI_DTOR_FUNC *dest)(void *)) {
__cxa_eh_globals *globals = __cxa_get_globals();
__cxa_exception* exception_header = cxa_exception_from_thrown_object(thrown_object);

Expand Down
4 changes: 2 additions & 2 deletions libcxxabi/src/cxa_exception.h
Expand Up @@ -43,7 +43,7 @@ struct _LIBCXXABI_HIDDEN __cxa_exception {

// Manage the exception object itself.
std::type_info *exceptionType;
void (*exceptionDestructor)(void *);
void (_LIBCXXABI_DTOR_FUNC *exceptionDestructor)(void *);
std::unexpected_handler unexpectedHandler;
std::terminate_handler terminateHandler;

Expand Down Expand Up @@ -81,7 +81,7 @@ struct _LIBCXXABI_HIDDEN __cxa_dependent_exception {
#endif

std::type_info *exceptionType;
void (*exceptionDestructor)(void *);
void (_LIBCXXABI_DTOR_FUNC *exceptionDestructor)(void *);
std::unexpected_handler unexpectedHandler;
std::terminate_handler terminateHandler;

Expand Down

0 comments on commit aeb4907

Please sign in to comment.