[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