I had another patch that would skip converting symbols to dimension when it is multiplied/divided by another dimension however this prevented the transformation in the original version of symbolic_semi_affine. Please let me know if you think there is a better solution as I'm not super familiar with affine expressions.

removed DefaultResource

rename test function

Address review comments.

In D94318#2487343, @aartbik wrote:

do we need similar extensions for the other load/store flavors (masked load/store, gather/scatter, compress/expand)?
[if so, please let's wait until my CL is in to avoid conflicts, since I have been making some mods to those ops]

Address review feedback.

Address review comment

Fix bug caught in review and add extra test. Address more review feedback.

Change function name to extractVectorTypeFromShapedValue.

In D93120#2471069, @bondhugula wrote:

@ThomasRaoux, not on a directly related note, but this file VectorOps.cpp is taking nearly 12-15s to build on a fast workstation -- directly increasing build times / critical path towards mlir-opt and other targets for everyone. It'd be great to consider refactoring among next steps.

Thanks Aart and Mahesh

Address more review comments.

Address review comments.

Thanks, Aart.

ping, if anybody is around :)

ping, if anybody is around :)

Add review comments.

Address review comments.

In D92540#2432328, @burmako wrote:

Thanks a lot for your work! This will be really useful :)

I'd also like to share the credit that you kindly gave to me with fedelebron@ and agrue@.

Add comment and spell out vector types in test.

Fix typo

Adding todo to relax indexing map restrictions.

Use ElementwiseMappable.

Address review comments.

Thanks Aart.

Address review comments

Address review comments.