- 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 | ||
---|---|---|
58 | Nit: if the mapping already exists | |
64 | Same nit | |
71 | Nit: "an the" | |
81 | 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 | ||
---|---|---|
81 | 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. |
clang-tidy: warning: header guard does not follow preferred style [llvm-header-guard]
not useful