You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Follow the `call` and `call_opaque` operations, as well as `apply`,
which already are marked as `CExpression` even though they have side
effects.
Even though `emitc.load` can be included inside the `emitc.expression`,
the inlining and `--form-expression` pass won't actually inline them
inside other expression due to it having a side effect, thus unless
the user manually writes the `emitc.load` inside the `emitc.expression`
it won't appear there.
Also update the `hasSideEffects` to account for `LoadOp` instead of
`VariableOp`. The `VariableOp` was used before the `LoadOp` was
introduced, but at the present time, the `LoadOp` is what is being
used to actually load and emit the instruction that does the loading.
0 commit comments