[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