VachanVY wrote:

> Please use existing facilities to add thousands-separators, not implement a 
> new one.

@zwuis do you mean `APInt::toString(..., /*InsertSeparators=*/true` (exposed 
via `llvm::toString` in `llvm/ADT/StringExtras.h`) ?

https://github.com/llvm/llvm-project/pull/211785
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to