Index attributes had no default value, which means the attribute values had to be set on the operation. This revision adds a default parameter to IndexAttrDef. After the change, every index attribute has to define a default value. For example, we may define the following strides attribute:
When using the operation the default stride is used if the strides attribute is not set. The mechanism is implemented using DefaultValuedAttr.
Additionally, the revision uses the naming index attribute instead of attribute more consistently, which is a preparation for follow up revisions that will introduce function attributes.
Depends On D119125
I would have expected this to be NFC. What am I missing?