[Mlir-commits] [mlir] [mlir][emitc] Inline constant when translate (PR #143485)

[Kirill Chibisov]

kchibisov wrote:

>I think not only for constantOp but also other ops, we should mark it have no side effect only for builtin types. Because the op with opaque type may use overloaded operator that may access memory or have other side effect.

The other op I think is `load` at this point, the rest already have checks in place. The point of this interface was to make it also for load where load uses primitive type, so it won't have side effect, instead of prohibiting everyload as we do now.

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