[libcxx-commits] [PATCH] D119385: Use trivial relocation operations in std::vector, by porting D67524 and part of D61761 to work on top of the changes in D114732.
Devin Jeanpierre via Phabricator via libcxx-commits
libcxx-commits at lists.llvm.org
Wed Feb 9 16:05:01 PST 2022
devin.jeanpierre added a comment.
@Quuxplusone I'm not sure if this is ready for proper review yet (I only ran tests locally, couldn't figure out how to make it stop using the default gcc backend for tests, so it might be a bit broken at head), but I wanted your feedback on how this is structured. Does this look ~basically OK?
I'm also not sure how to do the crediting so that you get fair credit for this -- really all I did was copy, paste, and make some tweaks here and there. :)
Otherwise, I'm going to wait for buildbots to come back, and figure out how to run tests locally against my local build of clang (which includes `__is_trivially_relocatable`).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D119385/new/
https://reviews.llvm.org/D119385
More information about the libcxx-commits
mailing list