[Mlir-commits] [mlir] [mlir][emitc] mark `emitc.load` with `CExpression` (PR #130802)

[Kirill Chibisov]

kchibisov wrote:

@mgehre-amd @simon-camp I've updated PR to make a correct `hasSideEffects` check, since it does matter now. The old one was a left over from the time where `Variable` was the only way to define something that will actually have a loading.

The loosening of what kind of load is considered a side effect I'll do separately. The same with e.g. `call` if we want that.

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