[PATCH] D110535: [llvm] [ADT] Update llvm::Split() per Pavel Labath's suggestions

Michał Górny via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 20 09:36:59 PDT 2021


mgorny added a comment.

In D110535#3075563 <https://reviews.llvm.org/D110535#3075563>, @lattner wrote:

> Do you have commit access?  If not, please read the llvm developer policy and follow the instructions, thanks!

That's not the problem. The problem is that @labath and @joerg seem not to agree on how it should be done, and I don't know whether to hack support for `char` delimiter to preserve the old syntax as @joerg wanted, or to keep it as is (i.e. without support for `char` delimiter) as @labath originally suggested.


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

https://reviews.llvm.org/D110535



More information about the llvm-commits mailing list