[mlir][linalg][transform][python] Extend mix-in for Vectorize

[martin-luecke]

Summary

Extends the existing mix-in for VectorizeOp with support for the missing unit attributes.

Also fixes the unintuitive implementation where
structured.VectorizeOp(target=target, vectorize_padding=False) still resulted in the creation of the UnitAttr vectorize_padding.

[ingomueller-net]

mlir/python/mlir/dialects/_structured_transform_ops_ext.py

793–794

That cast should not be necessary; you just tested if it's an instance of bool, no?

793–794

Is all of this really necessary? Doesn't the generated code handle Booleans? I one case, I see this:

if bool(bufferize_destination_only): attributes["bufferize_destination_only"] = _ods_ir.UnitAttr.get(
  _ods_get_default_loc_context(loc))

Doesn't this handle all cases as desired if you remove the whole 12 lines?

[martin-luecke]

Address review comments:

• simplify mix-in to rely more on the generated constructor

[martin-luecke]

Thank you for your comments @ingomueller-net.

I do not yet have commit privileges so if you are happy with the state I would appreciate it if you could commit this on my behalf.

[ingomueller-net]

OK, done! Unfortunately, I show up as an author now on git. I spend a bit of time trying to find out who was going to be the author and how to make sure it remains you but I didn't find anything. Turns out that the one who lands becomes the author...