[Mlir-commits] [mlir] [mlir][vector] Allow integer indices in vector.extract/insert ops (PR #115808)

[Andrzej Warzyński]

banach-space wrote:

> The end goal is to be able to default to the widest type but use narrower types when needed and it's safe.

So I've been trying to figure out the right mechanism to select the right index size 😅 Suggestions are much appreciated :) At a very coarse grain level we could use the architecture pointer size, but this way we'd be mostly switching between 32 and 64 bits. #not-good-enough :) 

> However, note that vector.gather is a memory related op, vector.extract/insert are not.

Are you thinking that the default for `vector.extract/insert` should be narrower than for `vector.gather`? For example, `i32` and `i64`, respectively, on a 64 bit machine?

https://github.com/llvm/llvm-project/pull/115808