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?