cor3ntin added inline comments.

================
Comment at: clang/lib/Basic/Diagnostic.cpp:837
+          llvm::sys::unicode::isFormatting(CodepointValue)) {
+        OutStr.append(CodepointBegin, CodepointEnd);
+        continue;
----------------
cor3ntin wrote:
> erichkeane wrote:
> > What about...?
> I doesn't seem much better but sure, I can do that
actually, you would need something like
`OutStream.write(reinterpret_cast<const char*>(CodepointBegin), 
std::distance(CodepointBegin, CodepointEnd));`

Unless you insist, I'd rather not change it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108469

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to