Re: [PATCH v2] tree-optimization: Recognize add/sub absolute-value idiom [PR56223]

--0000000000003056df0658482652
Content-Type: text/plain; charset="UTF-8"

Hi Jeff,

Thanks for reviewing the patch and suggested modifications.

>> I think we can side-step this class of problems by checking if (GIMPLE)
in addition to the type tests your doing.
Done.

>> With the additional check to force these patterns to only fire for
>> GIMPLE this is OK to push to the trunk after the usual testing.
Modified and tested with no regressions.
Pushed the patch to the trunk.

Thanks,
Naveen

--0000000000003056df0658482652
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr">Hi Jeff,<div><br></div><div>Thanks for reviewing the patch=
 and suggested modifications.<br><br>&gt;&gt; I think we can side-step this=
 class of problems by checking if (GIMPLE) in addition to the type tests yo=
ur doing.=C2=A0</div><div>Done.<br><br>&gt;&gt; With the additional check t=
o force these patterns to only fire for <br>&gt;&gt; GIMPLE this is OK to p=
ush to the trunk after the usual testing.<br>Modified and tested with no re=
gressions.</div><div>Pushed the patch to the trunk.</div><div><br></div><di=
v>Thanks,</div><div>Naveen<br><br></div></div>

--0000000000003056df0658482652--