Historically the builtin dialect has had an empty namespace. This has unfortunately created a very awkward situation, where many utilities either have to special case the empty namespace, or just don't work at all right now. This revision adds a namespace to the builtin dialect, and starts to cleanup some of the utilities to no longer handle empty namespaces. For now, the assembly form of builtin operations does not require the builtin. prefix. (This should likely be re-evaluated though)
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Maybe irrelevant: is "module_terminator" still a thing? If so, should it also be builtin?