[llvm] [InstCombine] Add support for GEPs in `simplifyNonNullOperand` (PR #128365)

Alexander Kornienko via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 27 06:46:01 PST 2025


alexfh wrote:

> > Google is seeing some miscompiles that track back to this PR. I'm working on a sharable test case, but just wanted to put that out there in case someone else sees something similar.
> 
> I got an automatically reduced version of the code @Sterling-Augustine is talking about here. I'm not quite sure the interesting bits weren't distorted during reduction, but maybe this will help figuring out where the problem is: https://gcc.godbolt.org/z/YYcvG69Tf

I found a UB in the original C++ code, so maybe there's no miscompilation, just wrong user code. Nevertheless, please double-check if the input I provided is handled as intended.

https://github.com/llvm/llvm-project/pull/128365


More information about the llvm-commits mailing list