================
@@ -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