Removes uses of StrEnumAttr in core dialects
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
Indeed. Surprisingly, some dialects required very little work to change (StrEnumAttr -> EnumAttr and fix the BUILD file and that's it).
mlir/include/mlir/Dialect/GPU/GPUOps.td | ||
---|---|---|
40 | Unrelated here, but we have namespace here and dialect specified below (which has a namespace), feels a bit weird |
mlir/include/mlir/Dialect/GPU/GPUOps.td | ||
---|---|---|
40 | Yeah. That's because enums aren't tied to any particular dialect. I would like to redo enum gen entirely once all enums are moved over to AttrDef. |
Is the m(docm) here an accident?