[libcxx-commits] [PATCH] D143010: [libc++abi][AIX] Skip non-C++ EH aware frames when retrieving exception object

Xing Xue via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Thu Feb 2 09:05:05 PST 2023


xingxue marked 2 inline comments as done.
xingxue added inline comments.


================
Comment at: libcxxabi/src/aix_state_tab_eh.inc:673
+  _LIBCXXABI_TRACE_STATETAB0("skipNonCxxEHAwareFrames() reached the end of stack frames, aborting\n");
+  std::abort();
+}
----------------
daltenty wrote:
> Prefer just un-namespaced `abort`, since we don't have the standard library headers in the libcxxabi context.
Good point, thanks!


================
Comment at: libcxxabi/src/aix_state_tab_eh.inc:685
+// Note: make sure __xlc_excpetion_handle() is a non-leaf function. Currently
+// it calls skipNonCxxEHAwareFrames(), which in turn calls std::abort().
 _LIBCXXABI_FUNC_VIS uintptr_t __xlc_exception_handle() {
----------------
daltenty wrote:
> 
Changed as suggested, thanks!


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D143010/new/

https://reviews.llvm.org/D143010



More information about the libcxx-commits mailing list