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
_______________________________________________
llvm-branch-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits

Reply via email to