[PATCH] D94377: [StringExtras] Add a helper class for comma-separated lists

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 12 08:46:56 PST 2021


dblaikie added a comment.

In D94377#2491956 <https://reviews.llvm.org/D94377#2491956>, @MaskRay wrote:

> The name `SubsequentDelim` looks a bit strange to me. Maybe others have suggestions on the name?
> (personally I might choose `AddDelimUnlessFirst`)

Yeah, I don't have any particularly compelling names - maybe "StreamableSeparator"


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D94377



More information about the llvm-commits mailing list