[PATCH] Separate macro-fusible instructions from not for -mbranches-within-32B-boundaries

H.J. Lu hjl.tools@gmail.com
Tue Mar 3 14:12:00 GMT 2020


On Tue, Mar 3, 2020 at 12:38 AM Jan Beulich <jbeulich@suse.com> wrote:
>
> On 03.03.2020 04:40, Hongtao Liu wrote:
> > Update my patch.
>
> Thanks again, looks good to me now with just one minor comment:
>
> >+enum mf_cmp_kind
> >+  {
> >+    mf_cmp_test_and,  /* test/cmp */
>
> If there is a comment here, I think it should also mention AND. But
> of course given the enumerator's name I'm not sure a comment is
> needed in the first place. But you'll have to wait for H.J.'s
> comments / okay anyway.
>

LGTM.  I am checking it in for Hongtao.

Thank you both.

-- 
H.J.



More information about the Binutils mailing list