[libcxx-commits] [PATCH] D132092: [2a/3][ASan][libcxx] std::deque annotations

Tacet via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Tue Mar 21 22:52:27 PDT 2023


AdvenamTacet updated this revision to Diff 507238.
AdvenamTacet added a comment.

This update is a big refactor of code responsible for annotations.
This update reduced size of the patch significantly.
There are only two helper function (one wrapper), both are minimal.

I added a lot of comment to the (new) code, if you want a smaller patch those comments may be removed, but I think it's good to have them.

I belive, the code is ready for a review. I tested it locally (but only with one C++ standard, so there may be some failure, I will fix it asap).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D132092

Files:
  libcxx/include/__config
  libcxx/include/deque
  libcxx/test/std/containers/sequences/deque/deque.capacity/access.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.capacity/empty.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.capacity/max_size.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.capacity/resize_size.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.capacity/resize_size_value.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.capacity/shrink_to_fit.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.capacity/size.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.cons/alloc.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.cons/assign_initializer_list.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.cons/assign_iter_iter.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.cons/assign_size_value.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.cons/copy.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.cons/copy_alloc.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.cons/deduct.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.cons/default.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.cons/initializer_list.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.cons/initializer_list_alloc.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.cons/iter_iter.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.cons/iter_iter_alloc.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.cons/move.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.cons/move_alloc.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.cons/move_assign.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.cons/op_equal.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.cons/op_equal_initializer_list.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.cons/size.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.cons/size_value.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.cons/size_value_alloc.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.erasure/erase.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.erasure/erase_if.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.modifiers/clear.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.modifiers/emplace.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.modifiers/emplace_back.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.modifiers/emplace_front.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.modifiers/erase_iter.invalidation.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.modifiers/erase_iter.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.modifiers/erase_iter_iter.invalidation.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.modifiers/erase_iter_iter.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.modifiers/insert_iter_initializer_list.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.modifiers/insert_iter_iter.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.modifiers/insert_rvalue.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.modifiers/insert_size_value.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.modifiers/insert_value.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.modifiers/pop_back.invalidation.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.modifiers/pop_back.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.modifiers/pop_front.invalidation.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.modifiers/pop_front.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.modifiers/push_back.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.modifiers/push_back_rvalue.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.modifiers/push_front.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.modifiers/push_front_rvalue.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.special/copy.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.special/copy_backward.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.special/move.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.special/move_backward.pass.cpp
  libcxx/test/std/containers/sequences/deque/deque.special/swap.pass.cpp
  libcxx/test/support/asan_testing.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D132092.507238.patch
Type: text/x-patch
Size: 112114 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20230322/c9a8fa8f/attachment-0001.bin>


More information about the libcxx-commits mailing list