Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
mlir/lib/CAPI/IR/StandardTypes.cpp | ||
---|---|---|
321–335 | Nit: consider adding explicit casts between signed (intptr_t) and unsigned integers to avoid warnings. |
mlir/lib/Bindings/Python/IRModules.cpp | ||
---|---|---|
1300 | I notice that some def/def_static take py::arg, while others don't take. When do we need to use py::arg? What is the purpose of using py::arg? |
mlir/lib/Bindings/Python/IRModules.cpp | ||
---|---|---|
1300 | It is generally a good idea to include py::arg wherever there is an API where people may want to invoke things with keywords or have other hints to that effect (i.e. most of them). Any that are missing should not be considered intentional unless if it is a purely positional/internal/dunder API. |
I notice that some def/def_static take py::arg, while others don't take. When do we need to use py::arg? What is the purpose of using py::arg?