[PATCH] D123064: [Clang][C++23] P2071 Named universal character escapes
Aaron Ballman via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Apr 11 10:56:12 PDT 2022
aaron.ballman added inline comments.
================
Comment at: llvm/include/llvm/ADT/StringExtras.h:329
+std::string to_hexString(uint64_t Value, bool UpperCase = true);
+
----------------
cor3ntin wrote:
> aaron.ballman wrote:
> > `utohexstr()` already exists on line 152 -- any reason we can't reuse that?
> I guess I missed that. Why do we have 2 functions doing the same thing?
> How do you want me to clean that up?
> I think we should have a separate nfc patch to clean that afterwards
In this patch, I'd replace your new calls to `to_hexString()` with calls to `utohexstr()` and then in an NFC change, I'd get rid of `to_hexString()` entirely.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D123064/new/
https://reviews.llvm.org/D123064
More information about the cfe-commits
mailing list