tbaeder added a comment. In D150843#4401189 <https://reviews.llvm.org/D150843#4401189>, @tahonermann wrote:
> The summary states that the changes are not quite NFC. In that case, we would > ideally have a test that demonstrates the changed behavior. Would adding such > a test be challenging? They are NFC, it's just not 100% only a refactoring, since it adds the new ASCII-only case. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D150843/new/ https://reviews.llvm.org/D150843 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits