[libcxx-commits] [PATCH] D96966: [libc++] Remove temporary stack-allocated array pointer

Louis Dionne via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Thu Feb 18 13:45:36 PST 2021


ldionne added inline comments.


================
Comment at: libcxx/include/string:1561
         {
-            size_type (&__a)[__n_words] = __r_.first().__r.__words;
             for (unsigned __i = 0; __i < __n_words; ++__i)
----------------
Actually, that's a reference to an array. So I don't think we should be creating any temporary here.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96966



More information about the libcxx-commits mailing list