[PATCH] D47406: [ADT] Add `StringRef::rsplit(StringRef Separator)`.

Henry Wong via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 7 08:24:54 PDT 2018


MTC added a comment.

In https://reviews.llvm.org/D47406#1120360, @zturner wrote:

> rsplit(char) already exists. That said, another way to reuse code would be
>  to have rsplit(char) call the StringRef version


Thanks for your advice, @zturner! I have refactored `split(char)` and `rsplit(char)` to use `StringRef` version. I think the code is more clean now.


Repository:
  rL LLVM

https://reviews.llvm.org/D47406





More information about the llvm-commits mailing list