Remove redundant `OptionalAttr` modifier on Unit attributes

[dime10]

Oh one (minor) related thing I am realizing now, is OptionalAttr<UnitAttr>:$adjoint not redundant? As in, a unit attribute would by definition need to be optional to have meaning?

Originally posted by @dime10 in #1695 (comment)