On 24.03.21 11:52, houzj.f...@fujitsu.com wrote:
+                               if (!has_nonconst_input)
+                                       return ece_evaluate_expr(expr);

That's not okay without a further check to see if the comparison function used
by the node is immutable.  Compare ScalarArrayOpExpr, for instance.

Thanks for pointing it out.
Attaching new patch with this change.

This patch looks okay to me and addresses all the feedback that was given. If there are no more comments, I'll commit it in a few days.


Reply via email to