[PATCH] D150843: [clang][Diagnostics] Refactor printableTextForNextCharacter

Timm Bäder via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed May 17 22:56:46 PDT 2023


tbaeder added inline comments.


================
Comment at: clang/lib/Frontend/TextDiagnostic.cpp:120
-  begin = reinterpret_cast<unsigned char const *>(&*(SourceLine.begin() + *i));
-  end = begin + (SourceLine.size() - *i);
-
----------------
I don't know what this computation of `end` means, but from the debug output I added, it meant that a call to this function always converted the entire line. I think.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150843



More information about the cfe-commits mailing list