balazske added a comment. In D75045#1891056 <https://reviews.llvm.org/D75045#1891056>, @NoQ wrote:
> In D75045#1891055 <https://reviews.llvm.org/D75045#1891055>, @NoQ wrote: > > > How many such platforms can you name? > > > I mean, does Clang even support such targets? The problem can occur with the "wide" variants of the getc functions when `wchar_t` is same width as `wint_t`, this can be more often (but I am not sure). Even then the character set should contain a character similar to `WEOF`, that is not true for UTF-16 and 32. So there may be low probability for the problem to occur. Is this reason to not have this check at all? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D75045/new/ https://reviews.llvm.org/D75045 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits