[llvm-branch-commits] [llvm] GlobalISel: Improve MMO extended LLT caclulation from size (PR #196398)

[Petar Avramovic via llvm-branch-commits]

petar-avramovic wrote:

> Which do you mean? Do you mean selection doesn't work through tablegen patterns or are you seeing problems elsewhere?

Selection is fine, it just now requires explicit, for example integer GILLT_i32, type checks for most of the opcodes.
So to make it work, I need to make sure lowering uses i32 not s32.

For some cases in particular, load itself is not the problem nor type in MMO, but this was the source of some other instructions using s32. So I wanted to fix the source.

I also noticed that MMOs that come from irtranslator now use i32/f32, only exceptions are MMOs generated during lowering that is why I wanted to make it consistent. Think that if you wanted to irtranslate MMO type as s32, this patch would give you scalarLLT for lowered MMOs

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