thurstond wrote: > That's what I though. The patch is LGTM, as it's relatively simple. But maybe we don't need it. The patch uses KnownBits value analysis, which is beneficial to shareholders. https://github.com/llvm/llvm-project/pull/122206