[PATCH] D149675: [Demangle] convert itaniumDemangle and nonMicrosoftDemangle to use std::string_view

Tue May 2 15:51:43 PDT 2023

efriedma accepted this revision.
efriedma added a comment.
This revision is now accepted and ready to land.

LGTM

In D149675#4313944 <https://reviews.llvm.org/D149675#4313944>, @nickdesaulniers wrote:

> In D149675#4313933 <https://reviews.llvm.org/D149675#4313933>, @efriedma wrote:
>
>> Maybe it makes sense to revert D149104 <https://reviews.llvm.org/D149104> while you're fixing the various demanglers to correctly respect the input length?
>
> This should fix-forwards the test. Sorry it took me so long to root cause.

If I understand the situation correctly, all the other demanglers read past the end of the buffer too; we just get lucky that none of the tests on the buildbots use the API in a way which exposes that.  I'd prefer to revert to known working, then fix each demangler, then fix the generic demangle() API.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D149675