akirilov-arm commented on issue #4331:
Also, I noticed that both
breduce
andireduce
were defined for vectors. Let's ignore the issue of boolean vectors and just look atireduce
- that operation is still problematic because of cases such as reductions fromI62X2
toI8X2
; on the other hand, as far as I can tell there are neither implementations nor tests of the vector variants. Should we just restrict the IR operation definitions to scalars?cc @abrown @cfallin
afonso360 commented on issue #4331:
Please, add runtests for the identity mappings, e.g. ireduce from I64 to I64.
It turns out that the verifier forbids these instructions. I've removed the rules for those.
Should we just restrict the IR operation definitions to scalars?
This sounds like a good idea, I haven't seen non 128bit vectors used anywhere and from what I've seen they are pretty much guaranteed not to compile despite being technically legal.
afonso360 commented on issue #4331:
CI is stuck. Closing and reopening
akirilov-arm commented on issue #4331:
@afonso360 Do you mind making the change for both operations?
Last updated: Dec 23 2024 at 12:05 UTC