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

Reply via email to