RKSimon added a comment. In D103888#2811305 <https://reviews.llvm.org/D103888#2811305>, @dblaikie wrote:
> Sounds OK. > > I wouldn't mind the places that can use op<< to use that - not sure > preserving the explicit radix argument is super high value. (I think people > would generally assume that's the default) > Possible we could call it `to_string`, is std::to_string meant to be an ADL > extension point, so that other types expose a to_string in their own > associated namespace, etc? We already have a llvm::to_string(V) wrapper inside ScopedPrinter.h that uses a temp llvm::raw_string_ostream to create the string - do you think that'd be good enough? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D103888/new/ https://reviews.llvm.org/D103888 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
