================
@@ -1703,12 +1703,23 @@ class HlfirBuilder {
     BinaryOp<D> binaryOp;
     auto left = hlfir::loadTrivialScalar(loc, builder, gen(op.left()));
     auto right = hlfir::loadTrivialScalar(loc, builder, gen(op.right()));
+
+    bool noReassoc = getConverter().getLoweringOptions().getProtectParens() &&
+                     (left.template getDefiningOp<hlfir::NoReassocOp>() ||
+                      right.template getDefiningOp<hlfir::NoReassocOp>());
----------------
eugeneepshteyn wrote:

@jeanPerier , I only made changes to support array operations. Could you please 
check if the new direction makes sense? 

I will address other PR comments separately.

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