[PATCH] D83321: [Support] Fix utf16 path's index upper bound

Andrew Ng via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 9 03:53:49 PDT 2020


andrewng added a comment.

In D83321#2140620 <https://reviews.llvm.org/D83321#2140620>, @danix800 wrote:

> Full context attached @andrewng .


Thanks for doing that, makes it easier for reviewers to see around the code without having to load up the file in question.

Perhaps this particular use case needs to be added to the unit testing ("llvm/unittests/Support/Path.cpp")?


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

https://reviews.llvm.org/D83321





More information about the llvm-commits mailing list