[libcxx-commits] [PATCH] D132769: [2b/3][ASan][libcxx] std::basic_string annotations

Tacet via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Wed Apr 19 02:00:33 PDT 2023


AdvenamTacet added inline comments.


================
Comment at: libcxx/include/string:2385
     __set_long_cap(__allocation.count);
+    __set_long_size(__old_sz);
+    __annotate_new(__old_sz);
----------------
ldionne wrote:
> Isn't that a pre-existing problem that should be fixed in its own patch?
I created a separate revision with that change: D148693
I will remove it from this patch in the next update.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D132769



More information about the libcxx-commits mailing list