[PATCH] D115387: [instcombine] Canonicalize constant index type to i64 for extractelement/insertelement

[Craig Topper via Phabricator via llvm-commits]

craig.topper added a comment.

In D115387#3183235 <https://reviews.llvm.org/D115387#3183235>, @reames wrote:

> Sounds like the consensus is i32?  I'll give it another day for further discussion, and will then update the patch if that's the direction we want to go in.

Would we update IRBuilder to use i32 and perhaps change the interface from uint64_t to unsigned?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D115387/new/

https://reviews.llvm.org/D115387