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

[Nikita Popov via Phabricator via llvm-commits]

nikic added a comment.

Canonicalizing this sounds generally reasonable, but why towards i64 rather than i32? We require i32 for shuffle masks, and it's the canonical type for struct indices, so it seems like the more natural choice, and would probably result in less test diffs.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115387