[llvm] [InstCombine] Handle more scalable geps in EmitGEPOffset (PR #71699)
Nikita Popov via llvm-commits
llvm-commits at lists.llvm.org
Fri Nov 10 05:40:31 PST 2023
nikic wrote:
> It appears that the `& PtrSizeMask` can be removed without altering any of the lit tests or any of the programs I tried across AArch64/Arm. Let me know if you think I should re-add it.
I think this is okay. I confirmed that the APInt uint64_t ctor will implicitly truncate the value, rather than asserting. As such, an explicit mask should not be necessary.
https://github.com/llvm/llvm-project/pull/71699
More information about the llvm-commits
mailing list