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 > >