[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