[PATCH] D143942: [NFC] Replace -1U{LL} expressions with appropriate *_MAX macros in Support library.

[Pavel Kopyl via Phabricator via llvm-commits]

pavelkopyl added a comment.

In D143942#4124074 <https://reviews.llvm.org/D143942#4124074>, @dblaikie wrote:

> Mixed feelings - if the code is valid/does what the developer intended, and the MSVC warning is just giving false positives/being too chatty, then we should probably disable the MSVC warning.
>
> Equally, this does seem more readable, at the cost of including another header - but I guess there aren't too many TUs that don't include climits somewhere along the way.
>
> I'm OK with this, but also open to hearing other people's perspectives.

Actually, this warning is already suppressed in mainline, but we have a bit customized build where it would more desirable to "fix" it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143942