[PATCH] D148348: [StringView] remove dropFront

Nick Desaulniers via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 14 10:05:12 PDT 2023


nickdesaulniers planned changes to this revision.
nickdesaulniers added inline comments.


================
Comment at: llvm/include/llvm/Demangle/ItaniumDemangle.h:1587
+      assert(sizeof("basic_") - 1 <= SV.size());
+      SV = SV.substr(sizeof("basic_") - 1);
     }
----------------
nickdesaulniers wrote:
> nickdesaulniers wrote:
> > anywhere we assign the result back to the string view object we should probably use StringView::remove_prefix instead.
> Nevermind; there is no StringView::remove_prefix. These can be changed to std::string_view::remove_prefix after conversion.
ah @Maskray added that in 3ece37b3fa2c14157ad02967b867570c6a0c08e8


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D148348



More information about the llvm-commits mailing list