[libcxx-commits] [PATCH] D132320: [libc++] Mark everything in <deque> as _LIBCPP_HIDE_FROM_ABI and replace _LIBCPP_INLINE_VISIBILITY

[Mark de Wever via Phabricator via libcxx-commits]

Mordante added a comment.

In D132320#3738045 <https://reviews.llvm.org/D132320#3738045>, @philnik wrote:

> In D132320#3738037 <https://reviews.llvm.org/D132320#3738037>, @Mordante wrote:
>
>> I only skimmed the file and see no direct oddities.
>> But this patch is an ABI break on multiple platforms.
>
> It's not an ABI break. I added that test in the parent commit and didn't get the numbers right yet.

It wasn't clear from this patch that it was put on top of a "broken" patch. It would have been nice to add that as a hint for the reviewers.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D132320