[libcxx-commits] [PATCH] D138527: [libc++abi] Simplify __cxa_get_globals_fast callsites

Shoaib Meenai via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Tue Nov 22 14:37:20 PST 2022


smeenai created this revision.
Herald added a project: All.
smeenai requested review of this revision.
Herald added a project: libc++abi.
Herald added a subscriber: libcxx-commits.
Herald added a reviewer: libc++abi.

Now that `__cxa_get_globals` is unconditionally using TLS (if we support
threads), it's equivalent to `__cxa_get_globals_fast`, and we don't need
to check if the return value of the latter is null. Update the test to
check this property as well.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D138527

Files:
  libcxxabi/src/cxa_default_handlers.cpp
  libcxxabi/src/cxa_exception.cpp
  libcxxabi/src/cxa_handlers.cpp
  libcxxabi/test/test_exception_storage.pass.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D138527.477303.patch
Type: text/x-patch
Size: 4850 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20221122/2a7c5423/attachment.bin>


More information about the libcxx-commits mailing list