[PATCH] D39149: [libc++] Prevent tautological comparisons

[Marshall Clow via Phabricator via cfe-commits]

mclow.lists added a comment.

In https://reviews.llvm.org/D39149#958378, @bcain wrote:

> Let's resurrect these changes since https://reviews.llvm.org/D39462 was not the right short-term approach.


Let's not.  IMHO, these changes are:

- a bad idea in principle - clang should not be warning here, and
- a bad idea in execution - we don't want to be using `__SIZEOF_INT__` and `__SIZEOF_LONG__`

I'm fine with the realization that "https://reviews.llvm.org/D39462 was not the right short-term approach", but that doesn't make this the right solution.


https://reviews.llvm.org/D39149