On Thu, Nov 14, 2024 at 5:30 PM Jeff Law <jeffreya...@gmail.com> wrote: > > > > On 11/11/24 9:32 PM, Eikansh Gupta wrote: > > Hi, > > > > > It seems to me this ought to work when the min/max reversed > > as well, or > > > am I missing something? > > > > Yes, it should work when min/max are reversed. > So are you going to add support for both? Seems to me like we'd want to > support both orders.
Both orders are already supported: + (op:c (min:c @0 @1) (max @0 @1)) Notice the `:c` on op. It means both `(plus (min @0 @1) (max @0 @1) )` and `(plus (max @0 @1) (min @0 @1) )` will match. Unless I am misunderstanding something here too. Thanks, Andrew Pinski > > jeff >