- 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. | |
clang-tidy: warning: header guard does not follow preferred style [llvm-header-guard]
not useful