ftynse wrote: > Aren't we losing some type-safety with this change? C API that does not differentiate any Attribute or Type subclasses. I don't see why we would have a privileged StringAttr here. https://github.com/llvm/llvm-project/pull/188222