[PATCH] D150539: [clang-format] Handle <chrono> ud suffixes in IntegerLiteralSeparator

[Owen Pan via Phabricator via cfe-commits]

owenpan added inline comments.


================
Comment at: clang/lib/Format/IntegerLiteralSeparatorFixer.cpp:116-117
     if (Style.isCpp()) {
-      if (const auto Pos = Text.find_first_of("_i"); Pos != StringRef::npos) {
+      // FIXME: This doesn't work for ud-suffix d from std::chrono::day.
+      if (const auto Pos = Text.find_first_of("_himnsuy");
+          Pos != StringRef::npos) {
----------------
HazardyKnusperkeks wrote:
> What about this?
Good idea!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150539