================
@@ -1733,27 +1751,29 @@ class HlfirBuilder {
       assert(right.isArray() && "must have at least one array operand");
       shape = hlfir::genShape(loc, builder, right);
     }
-    auto genKernel = [&op, &left, &right, &binaryOp](
+    // Array operations may need to handle reassoc flag inside of
+    // hlfir.elemental
----------------
jeanPerier wrote:

nit: I do not think you need to add a comment to justify `noReassoc` needs to 
be captured. At least for me this would distract me from the big picture of 
what genKernel is doing when reading this code outside of the context of this 
PR.

https://github.com/llvm/llvm-project/pull/170505
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to