- Still rough edges that need more sugar but the bones are there. Notes left in the test case for things that can be improved.
- Does not actually yield custom OpViews yet for traversing. Will rework that in a followup.
Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
mlir/lib/Bindings/Python/Globals.h | ||
---|---|---|
59 | Nit: if the mapping already exists | |
65 | Same nit | |
72 | Nit: "an the" | |
82 | What is the convention for bool values? true = loaded, false = not found, no-value = not attempted? | |
mlir/lib/Bindings/Python/IRModules.h | ||
473 | Is this change intentional? | |
mlir/lib/Bindings/Python/MainModule.cpp | ||
57 | Can this ever contain false? If not, we rather want a StringSet<> for this. |
mlir/lib/Bindings/Python/Globals.h | ||
---|---|---|
82 | Changed to StringSet. | |
mlir/lib/Bindings/Python/IRModules.h | ||
473 | Thanks from a previous iteration of the patch where I had to capture the module in the method, but since I've pulled this out to the PyGlobals now, it is unnecessary. Reverted. | |
mlir/lib/Bindings/Python/MainModule.cpp | ||
57 | Thanks. Originally was mapping to the module, removed the mapped value and should have dropped it back to a set then. Done. |
Nit: if the mapping already exists