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

[Philip Reames via Phabricator via llvm-commits]

reames added a comment.

Can we go with I64 since some canonicalization is better than nothing, and if someone cares about making it I32, do that in a separate patch?  Aside from some large test churn, I don't really see any downside to changing this (arbitrary) choice later.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115387