[PATCH] D59927: [clangd] Support UTF-32 (i.e. codepoint) offsets.

[Ilya Biryukov via Phabricator via cfe-commits]

ilya-biryukov accepted this revision.
ilya-biryukov added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: unittests/clangd/SourceCodeTests.cpp:164
+                       llvm::Failed()); // out of range
+  // middle line
+  EXPECT_THAT_EXPECTED(positionToOffset(File, position(1, -1)),
----------------
small NIT: the `.` is missing at the end of the comment. Same for lots of comments in the file, I guess.


Repository:
  rCTE Clang Tools Extra

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

https://reviews.llvm.org/D59927