[PATCH] D151300: [clang][Diagnostics][NFC] Remove unnecessary StringRef

Wed May 24 05:18:18 PDT 2023

aaron.ballman added a comment.

In D151300#4367939 <https://reviews.llvm.org/D151300#4367939>, @tbaeder wrote:

> In D151300#4367884 <https://reviews.llvm.org/D151300#4367884>, @aaron.ballman wrote:
>
>> LGTM, but it's worth noting that `std::string::pop_back()` calls `erase()` and there's no guarantee that there's not an extra allocation involved as a result. However, I've not seen evidence that STLs actually do an allocation (looking at libc++ and MSVC STL, they don't appear to allocate), so I think this is fine.
>
> Is the allocation only relevant for performance reasons or something else?

Performance was my only concern there.

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

https://reviews.llvm.org/D151300