[PATCH] D146771: [support] Provide overload to PrintNumber that use C++ types

Paul Kirth via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 24 16:52:19 PDT 2023


paulkirth added a comment.

after looking at this for a bit, I think this is probably the way to go. Using a template in ScopedPrinter.h has some problems, because there are types like `ulittle32_t` that won't satisfy `is_integral`. Using conversion also has its issues, like making some calls ambiguous.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D146771



More information about the llvm-commits mailing list