[PATCH] D50170: [libcxxabi] Fix test_exception_address_alignment test for ARM

Yvan Roux via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 7 01:22:42 PDT 2018


yroux added a comment.

I dug a bit and can give more context.  The usage of _LIBUNWIND_ARM_EHABI was introduced by https://reviews.llvm.org/D31178 but from what we are seeing with the release builds, libunwind headers are not picked up (at least in these configs) and since r309226 <http://llvm.org/viewvc/llvm-project?view=revision&revision=309226> was committed, clang and libunwind unwind.h versions are close enough (and are the same w/r to alignment) to rely on libcxxabi macro _LIBCXXABI_ARM_EHABI to choose the alignment no matter which unwind header is used.


Repository:
  rCXXA libc++abi

https://reviews.llvm.org/D50170





More information about the llvm-commits mailing list