[libcxx-commits] [PATCH] D132092: [2a/3][ASan][libcxx] std::deque annotations
Tacet via Phabricator via libcxx-commits
libcxx-commits at lists.llvm.org
Tue May 30 22:11:11 PDT 2023
AdvenamTacet added a comment.
Ok, I just learned that deque objects have to be assignable/movable, so `std::deque<const int>` is not correct (from what I understand). Therefore tests for it cannot exist. But it's not my intention to change behavior of `std::deque<const T>` and therefore I created D151777 <https://reviews.llvm.org/D151777>. The new patch adds a few `const`. All examples working previously seem to work with D151777 <https://reviews.llvm.org/D151777>, but I found some not working even before this patch.
As - if I'm correct - deque object cannot be of const type, I think I did as much as I could.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D132092/new/
https://reviews.llvm.org/D132092
More information about the libcxx-commits
mailing list