This is an archive of the discontinued LLVM Phabricator instance.

[libc++] Replace incorrect visibility on a streambuf method
AbandonedPublic

Authored by ldionne on Jul 5 2018, 11:18 AM.

Details

Summary

In r336369, I replaced _LIBCPP_ALWAYS_INLINE by _LIBCPP_INLINE_VISIBILITY.
However, in one place, I replaced it by _LIBCPP_EXTERN_TEMPLATE_INLINE_VISIBILITY
and forgot to put it back before sending the review. Nobody spotted the subtle
mistake because the diff was so large. This broke the check-cxx-abilist test.

This commit fixes that.

Diff Detail

Event Timeline

ldionne created this revision.Jul 5 2018, 11:18 AM

It's pretty obvious that this is the problem when looking at https://reviews.llvm.org/D48892 (grep for __pbump), but TBH I haven't been able to run the check-cxx-abilist test locally. Mine fails with like 500 differences, so there's got to be something I'm doing wrong.

This revision is now accepted and ready to land.Jul 5 2018, 11:29 AM

It's pretty obvious that this is the problem when looking at https://reviews.llvm.org/D48892 (grep for __pbump), but TBH I haven't been able to run the check-cxx-abilist test locally. Mine fails with like 500 differences, so there's got to be something I'm doing wrong.

It's possible no one has gotten that test working on Darwin before. I didn't know about it, anyway.

ldionne abandoned this revision.Jul 5 2018, 11:47 AM

I'm dropping this because I reverted https://reviews.llvm.org/D48892 until I figure out why it's breaking LLDB. The next time I submit https://reviews.llvm.org/D48892, I'll make sure to do it without this problem.