On 4/21/21 3:03 PM, Richard Biener wrote:
> 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?

I would use a static method, but that doesn't work with constexpr (at least not
in -std=c++11).

Let's move the macro to the usage place.

Thanks,
Martin

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