[PATCH] D124341: [clang-tidy][NFC] Replace many instances of std::string where a StringRef would suffice.

Mikael Holmén via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed May 11 01:23:58 PDT 2022


uabelho added a comment.

In D124341#3504427 <https://reviews.llvm.org/D124341#3504427>, @njames93 wrote:

> In D124341#3502659 <https://reviews.llvm.org/D124341#3502659>, @uabelho wrote:
>
>> Hi,
>>
>> I noticed that this patch isn't NFC.
>
> Whoops, good catch. I left in some debugging code, fixed in rGa308a55720249749 <https://reviews.llvm.org/rGa308a557202497495c10355279b5b9975bbee98e>

Thanks!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D124341



More information about the cfe-commits mailing list