[llvm] [InstCombine] Ignore offset when removing redundant alignment assumptions (PR #202423)
Nikita Popov via llvm-commits
llvm-commits at lists.llvm.org
Mon Jun 8 12:59:58 PDT 2026
nikic wrote:
> We don't currently use that information anywhere, so we
should just drop the assume in this case.
I believe the AlignmentFromAssumptions pass uses this knowledge. Though we probably don't generate align assumes with offset anywhere?
I think we should either be retaining the offset information or dropping support for offsets entirely. But dropping the assume if we only know the alignment of the base pointer imho defeats the point of the offset argument.
https://github.com/llvm/llvm-project/pull/202423
More information about the llvm-commits
mailing list