[PATCH] D143267: [LoopVectorize] Use DataLayout::getIndexType instead of i32 for non-constant GEP indices.
Paul Walker via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Feb 5 06:58:58 PST 2023
paulwalker-arm accepted this revision.
paulwalker-arm added a comment.
This revision is now accepted and ready to land.
Patch looks good to me albeit a little clunky to not use a consistent type regardless of constness. We should just emit the IR instcombine considers canonical (I did something similar recently when it came to splat indices) but I take your point about the significant test update burden.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D143267/new/
https://reviews.llvm.org/D143267
More information about the llvm-commits
mailing list