[llvm] [CGP] Fix missing sign extension for base offset in optimizeMemoryInst (PR #161377)

Vladimir Radosavljevic via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 10 03:12:50 PDT 2025


vladimirradosavljevic wrote:

> Nice catch! Should we also use sign extend here?
> 
> https://github.com/llvm/llvm-project/blob/4c2b1d495aa26fa80833fa1508736df984b6f55e/llvm/lib/CodeGen/CodeGenPrepare.cpp#L3197

Yes, we also have to use sign extend there. Nice catch!
Thanks for the review! Could you please merge this PR on my behalf?

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


More information about the llvm-commits mailing list