[libcxx-commits] [PATCH] D90551: [libcxx] Implement P1956 rename low-level bit functions

Louis Dionne via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Mon Nov 16 06:38:12 PST 2020


ldionne added a comment.

In D90551#2386348 <https://reviews.llvm.org/D90551#2386348>, @Mordante wrote:

> Thanks for the review.
>
> I kept the old names to aid people transition to the new names since I don't know how many people relay on the old names. (I don't expect too many.) If the test at Apple and Google shows no or little breakage I think it would be good to remove the old names.

Removing the old names causes no breakage internally. Let's go with that, and we can always come back and add the old names back as a temporary measure if we notice breakage -- but it's not clear to me that's going to happen.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90551



More information about the libcxx-commits mailing list