PiotrZSL added a comment. "Do you think that the change can land like in its current state first? Or would you prefer that the configuration option is added at the same time?"
Code is fine, probably If would would write this, then I would bother to split into C++20 and C++2B simply because `std::print` wouldn't compile if it wouldn't be available. So I would just use hasAnyName. But code looks fine, configuration could be added later... CHANGES SINCE LAST ACTION https://reviews.llvm.org/D143342/new/ https://reviews.llvm.org/D143342 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
