[Mlir-commits] [llvm] [mlir] [Linalg] Fix crash in vectorizeScalableVectorPrecondition with undersized vector sizes (PR #205493)

[Andrzej Warzyński]

banach-space wrote:

Thank you for the updates!

> move the vectorizeScalableVectorPrecondition call below the TypeSwitch. Both have to pass, so only the order in which they fail changes.

+1

> or hoist the isValidMaskedInputVector call into vectorizeOpPrecondition for the LinalgOp case. For that case only: vectorizePackOpPrecondition, vectorizeUnPackOpPrecondition and the pad and insert_slice hooks each compare against a rank of their own, not against the loop count.

Yeah, this option would be a bit more involved. 

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