On Wed, Apr 21, 2021 at 2:09 PM Uros Bizjak via Gcc-patches
<gcc-patches@gcc.gnu.org> wrote:
>
> On Wed, Apr 21, 2021 at 10:28 AM Martin Liska <mli...@suse.cz> wrote:
> >
> >
> > gcc/ChangeLog:
> >
> >         * config/i386/i386.h (PTA_*): Remove.
> >         (enum pta_flag): New.
> >         (DEF_PTA): Generate PTA_* values from i386.def.
> >         * wide-int-bitmask.h (WIDE_INT_BITMASK_FROM_NTH): New macro.
> >         * config/i386/i386.def: New file.
>
> Shouldn't this file be rather named i386-isa.def?
>
> Otherwise LGTM, but someone have to approve wide-int-bitmask.h change.

Hmm, such macro isn't "nice" and as it seems you could have inlined it
at the single use in the source file?

> Uros.
>
> > ---
> >  gcc/config/i386/i386.def | 110 +++++++++++++++++++++++++++++++++++++++
> >  gcc/config/i386/i386.h   | 106 ++++++-------------------------------
> >  gcc/wide-int-bitmask.h   |   3 ++
> >  3 files changed, 128 insertions(+), 91 deletions(-)
> >  create mode 100644 gcc/config/i386/i386.def
> >

Reply via email to