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

[Sanjay Patel via Phabricator via llvm-commits]

spatel added a comment.

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

> 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.

No objection to i64 from me.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115387