[PATCH] D48892: [libc++] Replace uses of _LIBCPP_ALWAYS_INLINE by _LIBCPP_INLINE_VISIBILITY
Louis Dionne via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Jul 6 08:33:41 PDT 2018
ldionne updated this revision to Diff 154415.
ldionne added a comment.
This revision to the patch fixes a problem where __pbump had been applied
_LIBCPP_EXTERN_TEMPLATE_INLINE_VISIBILITY instead of _LIBCPP_INLINE_VISIBILITY,
which caused the symbols exported in the ABI to change and broke the CI.
Independently, LLDB tests that were failing because they relied on
_LIBCPP_INLINE_VISIBILITY were fixed so that this change should not
break them again.
I still haven't been able to run the ABI tests locally (they fail on my
machine), and I'd like to do that before I submit this patch.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 87676 bytes
Desc: not available
More information about the cfe-commits