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?