[Mlir-commits] [mlir] [mlir] fix unregistered op's populateInherentAttrs (PR #77718)

Yuanqiang Liu llvmlistbot at llvm.org
Fri Feb 2 01:53:14 PST 2024


qingyunqu wrote:

> It's not clear to me this is something we can consistently support: the properties attribute may not be a dictionary and this will work in some cases but not in general.

I know it may not to a dictionary. But it previous was assumed to be a dictionary:
https://github.com/llvm/llvm-project/blob/main/mlir/lib/IR/MLIRContext.cpp#L844C1-L853C2

https://github.com/llvm/llvm-project/pull/77718


More information about the Mlir-commits mailing list