[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