[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