[all-commits] [llvm/llvm-project] f5371e: [Damangle] convert dlangDemangle to use std::strin...

Nick Desaulniers via All-commits all-commits at lists.llvm.org
Fri Jun 2 15:20:03 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: f5371eb3d3aed06ba84a69533586a60243ad2f24
      https://github.com/llvm/llvm-project/commit/f5371eb3d3aed06ba84a69533586a60243ad2f24
  Author: Nick Desaulniers <ndesaulniers at google.com>
  Date:   2023-06-02 (Fri, 02 Jun 2023)

  Changed paths:
    M lldb/source/Core/Mangled.cpp
    M llvm/include/llvm/Demangle/Demangle.h
    M llvm/lib/Demangle/DLangDemangle.cpp
    M llvm/lib/Demangle/Demangle.cpp
    M llvm/tools/llvm-dlang-demangle-fuzzer/llvm-dlang-demangle-fuzzer.cpp
    M llvm/unittests/Demangle/DLangDemangleTest.cpp

  Log Message:
  -----------
  [Damangle] convert dlangDemangle to use std::string_view

I was doing this API conversion to use std::string_view top-down in
D149104, but this exposed issues in individual demanglers that needed to
get fixed first. There's no issue with the conversion for the D language
demangler, so convert it.

I have a more aggressive refactoring of the entire D language demangler
to use std::string_view more extensively, but the interface with
llvm::nonMicrosoftDemangle is the more interesting one.

Reviewed By: MaskRay

Differential Revision: https://reviews.llvm.org/D151003




More information about the All-commits mailing list