[PATCH] D91968: llvm/ADT/StringExtras.h hexDigitValue - Init of integer buffer

Duncan P. N. Exon Smith via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 30 17:54:17 PST 2020


dexonsmith added a comment.

In D91968#2412092 <https://reviews.llvm.org/D91968#2412092>, @joerg wrote:

> Do we really want to change the code here? It is perfectly well defined behavior.

I weakly agree with @joerg, I would have thought we have lots of cases of `-1U` since it's a common / well-understood way to get this value. Is the warning possible to turn off? Have you reported a compiler bug?


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

https://reviews.llvm.org/D91968



More information about the llvm-commits mailing list