cor3ntin added a comment. In D133807#3792525 <https://reviews.llvm.org/D133807#3792525>, @tahonermann wrote:
> Structurally, these changes look like what I would expect. I didn't try to > validate any of the code point ranges. > > Are there useful tests that could be modified or added in order to validate > (probably on a spot check basis) Unicode 15 support for regression purposes? > For example, adding a test for `\N{}` for one of the newly added names or > some case folding tests for new characters. I considered it, and I can add a few if you insist but... I'm not sure adding random tests tell us much except that the specific tested characters are supported. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D133807/new/ https://reviews.llvm.org/D133807 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits