[Mlir-commits] [mlir] [mlir][emitc] Deferred emission as expressions (PR #159975)

[Gil Rapaport]

================
@@ -1691,6 +1707,12 @@ LogicalResult CppEmitter::emitOperand(Value value, bool isInBrackets) {
   if (expressionOp && shouldBeInlined(expressionOp))
     return emitExpression(expressionOp);
 
+  if (auto cExpression =
+          dyn_cast_if_present<CExpressionInterface>(value.getDefiningOp())) {
+    if (cExpression.alwaysInline())
+      return emitExpression(value.getDefiningOp());
+  }
----------------
aniragil wrote:

> shouldBeInlined already encodes "alwaysInline CExpression or inlinable ExpressionOp"

Good catch!

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