[PATCH] D142826: [Clang] Add -Wtype-limits to -Wextra for GCC compatibility

Thu Feb 2 02:39:23 PST 2023

hans added a comment.

In D142826#4098014 <https://reviews.llvm.org/D142826#4098014>, @chapuni wrote:

> This discovers a warning in https://reviews.llvm.org/rGa68d4b11465f5b3326be1dd820f59fac275b7581
> I think its checking is valid if size_t is uint32_t (eg. -m32)
>
> Could you guys teach me what the right fix for it? I don't know canonical fixes for it.

+1 we have some code like this too and I'm not sure what's the best way to rewrite it to pacify this warning.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D142826/new/

https://reviews.llvm.org/D142826