* [PATCH 1/7] x86: don't leak sub-architecture accumulated strings
2022-06-30 12:52 [PATCH] gas/x86: .arch / -march= enhancements Jan Beulich
@ 2022-06-30 12:53 ` Jan Beulich
2022-06-30 22:54 ` H.J. Lu
2022-06-30 12:53 ` [PATCH 2/7] x86: de-duplicate sub-architecture strings accumulation Jan Beulich
` (5 subsequent siblings)
6 siblings, 1 reply; 17+ messages in thread
From: Jan Beulich @ 2022-06-30 12:53 UTC (permalink / raw)
To: Binutils
While it may not be necessary in i386_target_format() (but then setting
the variable to NULL also wouldn't be necessary), at least in the other
cases strings may already have accumulated.
--- a/gas/config/tc-i386.c
+++ b/gas/config/tc-i386.c
@@ -2871,6 +2871,7 @@ set_cpu_arch (int dummy ATTRIBUTE_UNUSED
check_cpu_arch_compatible (string, cpu_arch[j].flags);
cpu_arch_name = cpu_arch[j].name;
+ xfree (cpu_sub_arch_name);
cpu_sub_arch_name = NULL;
cpu_arch_flags = cpu_arch[j].flags;
if (flag_code == CODE_64BIT)
@@ -13406,6 +13407,7 @@ md_parse_option (int c, const char *arg)
continue;
cpu_arch_name = cpu_arch[j].name;
+ xfree (cpu_sub_arch_name);
cpu_sub_arch_name = NULL;
cpu_arch_flags = cpu_arch[j].flags;
cpu_arch_isa = cpu_arch[j].type;
@@ -14086,6 +14088,7 @@ i386_target_format (void)
{
static const i386_cpu_flags iamcu_flags = CPU_IAMCU_FLAGS;
cpu_arch_name = "iamcu";
+ xfree (cpu_sub_arch_name);
cpu_sub_arch_name = NULL;
cpu_arch_flags = iamcu_flags;
cpu_arch_isa = PROCESSOR_IAMCU;
^ permalink raw reply [flat|nested] 17+ messages in thread* Re: [PATCH 1/7] x86: don't leak sub-architecture accumulated strings
2022-06-30 12:53 ` [PATCH 1/7] x86: don't leak sub-architecture accumulated strings Jan Beulich
@ 2022-06-30 22:54 ` H.J. Lu
2022-07-01 10:18 ` Jan Beulich
0 siblings, 1 reply; 17+ messages in thread
From: H.J. Lu @ 2022-06-30 22:54 UTC (permalink / raw)
To: Jan Beulich; +Cc: Binutils
On Thu, Jun 30, 2022 at 5:53 AM Jan Beulich <jbeulich@suse.com> wrote:
>
> While it may not be necessary in i386_target_format() (but then setting
> the variable to NULL also wouldn't be necessary), at least in the other
> cases strings may already have accumulated.
>
> --- a/gas/config/tc-i386.c
> +++ b/gas/config/tc-i386.c
> @@ -2871,6 +2871,7 @@ set_cpu_arch (int dummy ATTRIBUTE_UNUSED
> check_cpu_arch_compatible (string, cpu_arch[j].flags);
>
> cpu_arch_name = cpu_arch[j].name;
> + xfree (cpu_sub_arch_name);
> cpu_sub_arch_name = NULL;
> cpu_arch_flags = cpu_arch[j].flags;
> if (flag_code == CODE_64BIT)
> @@ -13406,6 +13407,7 @@ md_parse_option (int c, const char *arg)
> continue;
>
> cpu_arch_name = cpu_arch[j].name;
> + xfree (cpu_sub_arch_name);
> cpu_sub_arch_name = NULL;
> cpu_arch_flags = cpu_arch[j].flags;
> cpu_arch_isa = cpu_arch[j].type;
> @@ -14086,6 +14088,7 @@ i386_target_format (void)
> {
> static const i386_cpu_flags iamcu_flags = CPU_IAMCU_FLAGS;
> cpu_arch_name = "iamcu";
> + xfree (cpu_sub_arch_name);
> cpu_sub_arch_name = NULL;
> cpu_arch_flags = iamcu_flags;
> cpu_arch_isa = PROCESSOR_IAMCU;
>
Can we just use free?
--
H.J.
^ permalink raw reply [flat|nested] 17+ messages in thread* Re: [PATCH 1/7] x86: don't leak sub-architecture accumulated strings
2022-06-30 22:54 ` H.J. Lu
@ 2022-07-01 10:18 ` Jan Beulich
2022-07-01 19:44 ` H.J. Lu
0 siblings, 1 reply; 17+ messages in thread
From: Jan Beulich @ 2022-07-01 10:18 UTC (permalink / raw)
To: H.J. Lu; +Cc: Binutils
On 01.07.2022 00:54, H.J. Lu wrote:
> On Thu, Jun 30, 2022 at 5:53 AM Jan Beulich <jbeulich@suse.com> wrote:
>> --- a/gas/config/tc-i386.c
>> +++ b/gas/config/tc-i386.c
>> @@ -2871,6 +2871,7 @@ set_cpu_arch (int dummy ATTRIBUTE_UNUSED
>> check_cpu_arch_compatible (string, cpu_arch[j].flags);
>>
>> cpu_arch_name = cpu_arch[j].name;
>> + xfree (cpu_sub_arch_name);
>> cpu_sub_arch_name = NULL;
>> cpu_arch_flags = cpu_arch[j].flags;
>> if (flag_code == CODE_64BIT)
>> @@ -13406,6 +13407,7 @@ md_parse_option (int c, const char *arg)
>> continue;
>>
>> cpu_arch_name = cpu_arch[j].name;
>> + xfree (cpu_sub_arch_name);
>> cpu_sub_arch_name = NULL;
>> cpu_arch_flags = cpu_arch[j].flags;
>> cpu_arch_isa = cpu_arch[j].type;
>> @@ -14086,6 +14088,7 @@ i386_target_format (void)
>> {
>> static const i386_cpu_flags iamcu_flags = CPU_IAMCU_FLAGS;
>> cpu_arch_name = "iamcu";
>> + xfree (cpu_sub_arch_name);
>> cpu_sub_arch_name = NULL;
>> cpu_arch_flags = iamcu_flags;
>> cpu_arch_isa = PROCESSOR_IAMCU;
>>
>
> Can we just use free?
In principle yes, but I view using free() as inconsistent when the
allocation used xmalloc() or alike. These should pair, even if right
now xfree() simply aliases to free() - this could change down the
road, and then cause unnecessary code churn just to update all such
instances.
Jan
^ permalink raw reply [flat|nested] 17+ messages in thread* Re: [PATCH 1/7] x86: don't leak sub-architecture accumulated strings
2022-07-01 10:18 ` Jan Beulich
@ 2022-07-01 19:44 ` H.J. Lu
0 siblings, 0 replies; 17+ messages in thread
From: H.J. Lu @ 2022-07-01 19:44 UTC (permalink / raw)
To: Jan Beulich; +Cc: Binutils
On Fri, Jul 1, 2022 at 3:18 AM Jan Beulich <jbeulich@suse.com> wrote:
>
> On 01.07.2022 00:54, H.J. Lu wrote:
> > On Thu, Jun 30, 2022 at 5:53 AM Jan Beulich <jbeulich@suse.com> wrote:
> >> --- a/gas/config/tc-i386.c
> >> +++ b/gas/config/tc-i386.c
> >> @@ -2871,6 +2871,7 @@ set_cpu_arch (int dummy ATTRIBUTE_UNUSED
> >> check_cpu_arch_compatible (string, cpu_arch[j].flags);
> >>
> >> cpu_arch_name = cpu_arch[j].name;
> >> + xfree (cpu_sub_arch_name);
> >> cpu_sub_arch_name = NULL;
> >> cpu_arch_flags = cpu_arch[j].flags;
> >> if (flag_code == CODE_64BIT)
> >> @@ -13406,6 +13407,7 @@ md_parse_option (int c, const char *arg)
> >> continue;
> >>
> >> cpu_arch_name = cpu_arch[j].name;
> >> + xfree (cpu_sub_arch_name);
> >> cpu_sub_arch_name = NULL;
> >> cpu_arch_flags = cpu_arch[j].flags;
> >> cpu_arch_isa = cpu_arch[j].type;
> >> @@ -14086,6 +14088,7 @@ i386_target_format (void)
> >> {
> >> static const i386_cpu_flags iamcu_flags = CPU_IAMCU_FLAGS;
> >> cpu_arch_name = "iamcu";
> >> + xfree (cpu_sub_arch_name);
> >> cpu_sub_arch_name = NULL;
> >> cpu_arch_flags = iamcu_flags;
> >> cpu_arch_isa = PROCESSOR_IAMCU;
> >>
> >
> > Can we just use free?
>
> In principle yes, but I view using free() as inconsistent when the
> allocation used xmalloc() or alike. These should pair, even if right
> now xfree() simply aliases to free() - this could change down the
> road, and then cause unnecessary code churn just to update all such
> instances.
>
I don't think we will change xfree. Just use free directly.
Thanks.
--
H.J.
^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH 2/7] x86: de-duplicate sub-architecture strings accumulation
2022-06-30 12:52 [PATCH] gas/x86: .arch / -march= enhancements Jan Beulich
2022-06-30 12:53 ` [PATCH 1/7] x86: don't leak sub-architecture accumulated strings Jan Beulich
@ 2022-06-30 12:53 ` Jan Beulich
2022-06-30 22:55 ` H.J. Lu
2022-06-30 12:53 ` [PATCH 3/7] x86: permit "default" with .arch Jan Beulich
` (4 subsequent siblings)
6 siblings, 1 reply; 17+ messages in thread
From: Jan Beulich @ 2022-06-30 12:53 UTC (permalink / raw)
To: Binutils
Introduce a helper function to replace 4 instances of similar code. Use
reconcat() to cover the previously explicit free() (which really should
have been xfree() anyway).
--- a/gas/config/tc-i386.c
+++ b/gas/config/tc-i386.c
@@ -2851,6 +2851,16 @@ check_cpu_arch_compatible (const char *n
}
static void
+extend_cpu_sub_arch_name (const char *name)
+{
+ if (cpu_sub_arch_name)
+ cpu_sub_arch_name = reconcat (cpu_sub_arch_name, cpu_sub_arch_name,
+ name, (const char *) NULL);
+ else
+ cpu_sub_arch_name = xstrdup (name);
+}
+
+static void
set_cpu_arch (int dummy ATTRIBUTE_UNUSED)
{
SKIP_WHITESPACE ();
@@ -2899,16 +2909,7 @@ set_cpu_arch (int dummy ATTRIBUTE_UNUSED
if (!cpu_flags_equal (&flags, &cpu_arch_flags))
{
- if (cpu_sub_arch_name)
- {
- char *name = cpu_sub_arch_name;
- cpu_sub_arch_name = concat (name,
- cpu_arch[j].name,
- (const char *) NULL);
- free (name);
- }
- else
- cpu_sub_arch_name = xstrdup (cpu_arch[j].name);
+ extend_cpu_sub_arch_name (string);
cpu_arch_flags = flags;
cpu_arch_isa_flags = flags;
}
@@ -2932,15 +2933,7 @@ set_cpu_arch (int dummy ATTRIBUTE_UNUSED
cpu_noarch[j].flags);
if (!cpu_flags_equal (&flags, &cpu_arch_flags))
{
- if (cpu_sub_arch_name)
- {
- char *name = cpu_sub_arch_name;
- cpu_sub_arch_name = concat (name, string,
- (const char *) NULL);
- free (name);
- }
- else
- cpu_sub_arch_name = xstrdup (string);
+ extend_cpu_sub_arch_name (string);
cpu_arch_flags = flags;
cpu_arch_isa_flags = flags;
}
@@ -13430,16 +13423,7 @@ md_parse_option (int c, const char *arg)
if (!cpu_flags_equal (&flags, &cpu_arch_flags))
{
- if (cpu_sub_arch_name)
- {
- char *name = cpu_sub_arch_name;
- cpu_sub_arch_name = concat (name,
- cpu_arch[j].name,
- (const char *) NULL);
- free (name);
- }
- else
- cpu_sub_arch_name = xstrdup (cpu_arch[j].name);
+ extend_cpu_sub_arch_name (cpu_arch[j].name);
cpu_arch_flags = flags;
cpu_arch_isa_flags = flags;
}
@@ -13463,15 +13447,7 @@ md_parse_option (int c, const char *arg)
cpu_noarch[j].flags);
if (!cpu_flags_equal (&flags, &cpu_arch_flags))
{
- if (cpu_sub_arch_name)
- {
- char *name = cpu_sub_arch_name;
- cpu_sub_arch_name = concat (arch,
- (const char *) NULL);
- free (name);
- }
- else
- cpu_sub_arch_name = xstrdup (arch);
+ extend_cpu_sub_arch_name (arch);
cpu_arch_flags = flags;
cpu_arch_isa_flags = flags;
}
^ permalink raw reply [flat|nested] 17+ messages in thread* Re: [PATCH 2/7] x86: de-duplicate sub-architecture strings accumulation
2022-06-30 12:53 ` [PATCH 2/7] x86: de-duplicate sub-architecture strings accumulation Jan Beulich
@ 2022-06-30 22:55 ` H.J. Lu
0 siblings, 0 replies; 17+ messages in thread
From: H.J. Lu @ 2022-06-30 22:55 UTC (permalink / raw)
To: Jan Beulich; +Cc: Binutils
On Thu, Jun 30, 2022 at 5:53 AM Jan Beulich <jbeulich@suse.com> wrote:
>
> Introduce a helper function to replace 4 instances of similar code. Use
> reconcat() to cover the previously explicit free() (which really should
> have been xfree() anyway).
>
> --- a/gas/config/tc-i386.c
> +++ b/gas/config/tc-i386.c
> @@ -2851,6 +2851,16 @@ check_cpu_arch_compatible (const char *n
> }
>
> static void
> +extend_cpu_sub_arch_name (const char *name)
> +{
> + if (cpu_sub_arch_name)
> + cpu_sub_arch_name = reconcat (cpu_sub_arch_name, cpu_sub_arch_name,
> + name, (const char *) NULL);
> + else
> + cpu_sub_arch_name = xstrdup (name);
> +}
> +
> +static void
> set_cpu_arch (int dummy ATTRIBUTE_UNUSED)
> {
> SKIP_WHITESPACE ();
> @@ -2899,16 +2909,7 @@ set_cpu_arch (int dummy ATTRIBUTE_UNUSED
>
> if (!cpu_flags_equal (&flags, &cpu_arch_flags))
> {
> - if (cpu_sub_arch_name)
> - {
> - char *name = cpu_sub_arch_name;
> - cpu_sub_arch_name = concat (name,
> - cpu_arch[j].name,
> - (const char *) NULL);
> - free (name);
> - }
> - else
> - cpu_sub_arch_name = xstrdup (cpu_arch[j].name);
> + extend_cpu_sub_arch_name (string);
> cpu_arch_flags = flags;
> cpu_arch_isa_flags = flags;
> }
> @@ -2932,15 +2933,7 @@ set_cpu_arch (int dummy ATTRIBUTE_UNUSED
> cpu_noarch[j].flags);
> if (!cpu_flags_equal (&flags, &cpu_arch_flags))
> {
> - if (cpu_sub_arch_name)
> - {
> - char *name = cpu_sub_arch_name;
> - cpu_sub_arch_name = concat (name, string,
> - (const char *) NULL);
> - free (name);
> - }
> - else
> - cpu_sub_arch_name = xstrdup (string);
> + extend_cpu_sub_arch_name (string);
> cpu_arch_flags = flags;
> cpu_arch_isa_flags = flags;
> }
> @@ -13430,16 +13423,7 @@ md_parse_option (int c, const char *arg)
>
> if (!cpu_flags_equal (&flags, &cpu_arch_flags))
> {
> - if (cpu_sub_arch_name)
> - {
> - char *name = cpu_sub_arch_name;
> - cpu_sub_arch_name = concat (name,
> - cpu_arch[j].name,
> - (const char *) NULL);
> - free (name);
> - }
> - else
> - cpu_sub_arch_name = xstrdup (cpu_arch[j].name);
> + extend_cpu_sub_arch_name (cpu_arch[j].name);
> cpu_arch_flags = flags;
> cpu_arch_isa_flags = flags;
> }
> @@ -13463,15 +13447,7 @@ md_parse_option (int c, const char *arg)
> cpu_noarch[j].flags);
> if (!cpu_flags_equal (&flags, &cpu_arch_flags))
> {
> - if (cpu_sub_arch_name)
> - {
> - char *name = cpu_sub_arch_name;
> - cpu_sub_arch_name = concat (arch,
> - (const char *) NULL);
> - free (name);
> - }
> - else
> - cpu_sub_arch_name = xstrdup (arch);
> + extend_cpu_sub_arch_name (arch);
> cpu_arch_flags = flags;
> cpu_arch_isa_flags = flags;
> }
>
OK.
Thanks.
--
H.J.
^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH 3/7] x86: permit "default" with .arch
2022-06-30 12:52 [PATCH] gas/x86: .arch / -march= enhancements Jan Beulich
2022-06-30 12:53 ` [PATCH 1/7] x86: don't leak sub-architecture accumulated strings Jan Beulich
2022-06-30 12:53 ` [PATCH 2/7] x86: de-duplicate sub-architecture strings accumulation Jan Beulich
@ 2022-06-30 12:53 ` Jan Beulich
2022-06-30 22:58 ` H.J. Lu
2022-06-30 12:54 ` [PATCH 4/7] x86: macro-ize cpu_arch[] entries Jan Beulich
` (3 subsequent siblings)
6 siblings, 1 reply; 17+ messages in thread
From: Jan Beulich @ 2022-06-30 12:53 UTC (permalink / raw)
To: Binutils
So far there was no way to reset the architecture to that assembly would
start with in the absence of any overrides (command line or directives).
Note that for Intel MCU "default" is merely an alias of "iamcu".
While there also zap a stray @item from the doc section, as noticed
when inspecting the generated output (which still has some quirks, but
those aren't easy to address without re-flowing almost the entire
section).
--- a/gas/config/tc-i386.c
+++ b/gas/config/tc-i386.c
@@ -933,8 +933,8 @@ const relax_typeS md_relax_table[] =
static const arch_entry cpu_arch[] =
{
- /* Do not replace the first two entries - i386_target_format()
- relies on them being there in this order. */
+ /* Do not replace the first two entries - i386_target_format() and
+ set_cpu_arch() rely on them being there in this order. */
{ STRING_COMMA_LEN ("generic32"), PROCESSOR_GENERIC32,
CPU_GENERIC32_FLAGS, 0 },
{ STRING_COMMA_LEN ("generic64"), PROCESSOR_GENERIC64,
@@ -2867,12 +2867,47 @@ set_cpu_arch (int dummy ATTRIBUTE_UNUSED
if (!is_end_of_line[(unsigned char) *input_line_pointer])
{
- char *string;
- int e = get_symbol_name (&string);
- unsigned int j;
+ char *s;
+ int e = get_symbol_name (&s);
+ const char *string = s;
+ unsigned int j = 0;
i386_cpu_flags flags;
- for (j = 0; j < ARRAY_SIZE (cpu_arch); j++)
+ if (strcmp (string, "default") == 0)
+ {
+ if (strcmp (default_arch, "iamcu") == 0)
+ string = default_arch;
+ else
+ {
+ static const i386_cpu_flags cpu_unknown_flags = CPU_UNKNOWN_FLAGS;
+
+ cpu_arch_name = NULL;
+ xfree (cpu_sub_arch_name);
+ cpu_sub_arch_name = NULL;
+ cpu_arch_flags = cpu_unknown_flags;
+ if (flag_code == CODE_64BIT)
+ {
+ cpu_arch_flags.bitfield.cpu64 = 1;
+ cpu_arch_flags.bitfield.cpuno64 = 0;
+ }
+ else
+ {
+ cpu_arch_flags.bitfield.cpu64 = 0;
+ cpu_arch_flags.bitfield.cpuno64 = 1;
+ }
+ cpu_arch_isa = PROCESSOR_UNKNOWN;
+ cpu_arch_isa_flags = cpu_arch[flag_code == CODE_64BIT].flags;
+ if (!cpu_arch_tune_set)
+ {
+ cpu_arch_tune = cpu_arch_isa;
+ cpu_arch_tune_flags = cpu_arch_isa_flags;
+ }
+
+ j = ARRAY_SIZE (cpu_arch) + 1;
+ }
+ }
+
+ for (; j < ARRAY_SIZE (cpu_arch); j++)
{
if (strcmp (string, cpu_arch[j].name) == 0)
{
@@ -2945,7 +2980,7 @@ set_cpu_arch (int dummy ATTRIBUTE_UNUSED
j = ARRAY_SIZE (cpu_arch);
}
- if (j >= ARRAY_SIZE (cpu_arch))
+ if (j == ARRAY_SIZE (cpu_arch))
as_bad (_("no such architecture: `%s'"), string);
*input_line_pointer = e;
@@ -13841,6 +13876,13 @@ show_arch (FILE *stream, int ext, int ch
p = start;
left = size - (start - message);
+
+ if (!ext && check)
+ {
+ p = output_message (stream, p, message, start, &left,
+ STRING_COMMA_LEN ("default"));
+ }
+
for (j = 0; j < ARRAY_SIZE (cpu_arch); j++)
{
/* Should it be skipped? */
--- a/gas/doc/c-i386.texi
+++ b/gas/doc/c-i386.texi
@@ -1504,6 +1504,7 @@ directive enables a warning when gas det
supported on the CPU specified. The choices for @var{cpu_type} are:
@multitable @columnfractions .20 .20 .20 .20
+@item @samp{default}
@item @samp{i8086} @tab @samp{i186} @tab @samp{i286} @tab @samp{i386}
@item @samp{i486} @tab @samp{i586} @tab @samp{i686} @tab @samp{pentium}
@item @samp{pentiumpro} @tab @samp{pentiumii} @tab @samp{pentiumiii} @tab @samp{pentium4}
@@ -1531,7 +1532,7 @@ supported on the CPU specified. The cho
@item @samp{.avx512_vpopcntdq} @tab @samp{.avx512_vbmi2} @tab @samp{.avx512_vnni}
@item @samp{.avx512_bitalg} @tab @samp{.avx512_bf16} @tab @samp{.avx512_vp2intersect}
@item @samp{.tdx} @tab @samp{.avx_vnni} @tab @samp{.avx512_fp16}
-@item @samp{.clwb} @tab @samp{.rdpid} @tab @samp{.ptwrite} @tab @item @samp{.ibt}
+@item @samp{.clwb} @tab @samp{.rdpid} @tab @samp{.ptwrite} @tab @samp{.ibt}
@item @samp{.wbnoinvd} @tab @samp{.pconfig} @tab @samp{.waitpkg} @tab @samp{.cldemote}
@item @samp{.shstk} @tab @samp{.gfni} @tab @samp{.vaes} @tab @samp{.vpclmulqdq}
@item @samp{.movdiri} @tab @samp{.movdir64b} @tab @samp{.enqcmd} @tab @samp{.tsxldtrk}
--- /dev/null
+++ b/gas/testsuite/gas/i386/arch-dflt.l
@@ -0,0 +1,19 @@
+.*: Assembler messages:
+.*:3: Error:.*`cmovl'.*
+.*:9: Error:.*`cmovg'.*
+GAS LISTING .*
+
+
+[ ]*[0-9]*[ ]+\.text
+[ ]*[0-9]*[ ]+start:
+[ ]*[0-9]*[ ]+cmovl %eax, %ecx
+[ ]*[0-9]*[ ]*
+[ ]*[0-9]*[ ]+\.arch default
+[ ]*[0-9]*[ ]+\?\?\?\? 0F4DC8[ ]+cmovnl %eax, %ecx
+[ ]*[0-9]*[ ]*
+[ ]*[0-9]*[ ]+\.arch generic32
+[ ]*[0-9]*[ ]+cmovg %eax, %ecx
+[ ]*[0-9]*[ ]*
+[ ]*[0-9]*[ ]+\.arch default
+[ ]*[0-9]*[ ]+\?\?\?\? 0F4EC8[ ]+cmovng %eax, %ecx
+#pass
--- /dev/null
+++ b/gas/testsuite/gas/i386/arch-dflt.s
@@ -0,0 +1,14 @@
+ .text
+start:
+ cmovl %eax, %ecx
+
+ .arch default
+ cmovnl %eax, %ecx
+
+ .arch generic32
+ cmovg %eax, %ecx
+
+ .arch default
+ cmovng %eax, %ecx
+
+ .end
--- a/gas/testsuite/gas/i386/i386.exp
+++ b/gas/testsuite/gas/i386/i386.exp
@@ -205,6 +205,7 @@ if [gas_32_check] then {
run_dump_test "arch-12"
run_dump_test "arch-13"
run_dump_test "arch-14"
+ run_list_test "arch-dflt" "-march=generic32 -al"
run_dump_test "8087"
run_dump_test "287"
run_dump_test "387"
^ permalink raw reply [flat|nested] 17+ messages in thread* Re: [PATCH 3/7] x86: permit "default" with .arch
2022-06-30 12:53 ` [PATCH 3/7] x86: permit "default" with .arch Jan Beulich
@ 2022-06-30 22:58 ` H.J. Lu
0 siblings, 0 replies; 17+ messages in thread
From: H.J. Lu @ 2022-06-30 22:58 UTC (permalink / raw)
To: Jan Beulich; +Cc: Binutils
On Thu, Jun 30, 2022 at 5:54 AM Jan Beulich <jbeulich@suse.com> wrote:
>
> So far there was no way to reset the architecture to that assembly would
> start with in the absence of any overrides (command line or directives).
> Note that for Intel MCU "default" is merely an alias of "iamcu".
>
> While there also zap a stray @item from the doc section, as noticed
> when inspecting the generated output (which still has some quirks, but
> those aren't easy to address without re-flowing almost the entire
> section).
>
> --- a/gas/config/tc-i386.c
> +++ b/gas/config/tc-i386.c
> @@ -933,8 +933,8 @@ const relax_typeS md_relax_table[] =
>
> static const arch_entry cpu_arch[] =
> {
> - /* Do not replace the first two entries - i386_target_format()
> - relies on them being there in this order. */
> + /* Do not replace the first two entries - i386_target_format() and
> + set_cpu_arch() rely on them being there in this order. */
> { STRING_COMMA_LEN ("generic32"), PROCESSOR_GENERIC32,
> CPU_GENERIC32_FLAGS, 0 },
> { STRING_COMMA_LEN ("generic64"), PROCESSOR_GENERIC64,
> @@ -2867,12 +2867,47 @@ set_cpu_arch (int dummy ATTRIBUTE_UNUSED
>
> if (!is_end_of_line[(unsigned char) *input_line_pointer])
> {
> - char *string;
> - int e = get_symbol_name (&string);
> - unsigned int j;
> + char *s;
> + int e = get_symbol_name (&s);
> + const char *string = s;
> + unsigned int j = 0;
> i386_cpu_flags flags;
>
> - for (j = 0; j < ARRAY_SIZE (cpu_arch); j++)
> + if (strcmp (string, "default") == 0)
> + {
> + if (strcmp (default_arch, "iamcu") == 0)
> + string = default_arch;
> + else
> + {
> + static const i386_cpu_flags cpu_unknown_flags = CPU_UNKNOWN_FLAGS;
> +
> + cpu_arch_name = NULL;
> + xfree (cpu_sub_arch_name);
Can we just use free?
> + cpu_sub_arch_name = NULL;
> + cpu_arch_flags = cpu_unknown_flags;
> + if (flag_code == CODE_64BIT)
> + {
> + cpu_arch_flags.bitfield.cpu64 = 1;
> + cpu_arch_flags.bitfield.cpuno64 = 0;
> + }
> + else
> + {
> + cpu_arch_flags.bitfield.cpu64 = 0;
> + cpu_arch_flags.bitfield.cpuno64 = 1;
> + }
> + cpu_arch_isa = PROCESSOR_UNKNOWN;
> + cpu_arch_isa_flags = cpu_arch[flag_code == CODE_64BIT].flags;
> + if (!cpu_arch_tune_set)
> + {
> + cpu_arch_tune = cpu_arch_isa;
> + cpu_arch_tune_flags = cpu_arch_isa_flags;
> + }
> +
> + j = ARRAY_SIZE (cpu_arch) + 1;
> + }
> + }
> +
> + for (; j < ARRAY_SIZE (cpu_arch); j++)
> {
> if (strcmp (string, cpu_arch[j].name) == 0)
> {
> @@ -2945,7 +2980,7 @@ set_cpu_arch (int dummy ATTRIBUTE_UNUSED
> j = ARRAY_SIZE (cpu_arch);
> }
>
> - if (j >= ARRAY_SIZE (cpu_arch))
> + if (j == ARRAY_SIZE (cpu_arch))
> as_bad (_("no such architecture: `%s'"), string);
>
> *input_line_pointer = e;
> @@ -13841,6 +13876,13 @@ show_arch (FILE *stream, int ext, int ch
>
> p = start;
> left = size - (start - message);
> +
> + if (!ext && check)
> + {
> + p = output_message (stream, p, message, start, &left,
> + STRING_COMMA_LEN ("default"));
> + }
> +
> for (j = 0; j < ARRAY_SIZE (cpu_arch); j++)
> {
> /* Should it be skipped? */
> --- a/gas/doc/c-i386.texi
> +++ b/gas/doc/c-i386.texi
> @@ -1504,6 +1504,7 @@ directive enables a warning when gas det
> supported on the CPU specified. The choices for @var{cpu_type} are:
>
> @multitable @columnfractions .20 .20 .20 .20
> +@item @samp{default}
> @item @samp{i8086} @tab @samp{i186} @tab @samp{i286} @tab @samp{i386}
> @item @samp{i486} @tab @samp{i586} @tab @samp{i686} @tab @samp{pentium}
> @item @samp{pentiumpro} @tab @samp{pentiumii} @tab @samp{pentiumiii} @tab @samp{pentium4}
> @@ -1531,7 +1532,7 @@ supported on the CPU specified. The cho
> @item @samp{.avx512_vpopcntdq} @tab @samp{.avx512_vbmi2} @tab @samp{.avx512_vnni}
> @item @samp{.avx512_bitalg} @tab @samp{.avx512_bf16} @tab @samp{.avx512_vp2intersect}
> @item @samp{.tdx} @tab @samp{.avx_vnni} @tab @samp{.avx512_fp16}
> -@item @samp{.clwb} @tab @samp{.rdpid} @tab @samp{.ptwrite} @tab @item @samp{.ibt}
> +@item @samp{.clwb} @tab @samp{.rdpid} @tab @samp{.ptwrite} @tab @samp{.ibt}
> @item @samp{.wbnoinvd} @tab @samp{.pconfig} @tab @samp{.waitpkg} @tab @samp{.cldemote}
> @item @samp{.shstk} @tab @samp{.gfni} @tab @samp{.vaes} @tab @samp{.vpclmulqdq}
> @item @samp{.movdiri} @tab @samp{.movdir64b} @tab @samp{.enqcmd} @tab @samp{.tsxldtrk}
> --- /dev/null
> +++ b/gas/testsuite/gas/i386/arch-dflt.l
> @@ -0,0 +1,19 @@
> +.*: Assembler messages:
> +.*:3: Error:.*`cmovl'.*
> +.*:9: Error:.*`cmovg'.*
> +GAS LISTING .*
> +
> +
> +[ ]*[0-9]*[ ]+\.text
> +[ ]*[0-9]*[ ]+start:
> +[ ]*[0-9]*[ ]+cmovl %eax, %ecx
> +[ ]*[0-9]*[ ]*
> +[ ]*[0-9]*[ ]+\.arch default
> +[ ]*[0-9]*[ ]+\?\?\?\? 0F4DC8[ ]+cmovnl %eax, %ecx
> +[ ]*[0-9]*[ ]*
> +[ ]*[0-9]*[ ]+\.arch generic32
> +[ ]*[0-9]*[ ]+cmovg %eax, %ecx
> +[ ]*[0-9]*[ ]*
> +[ ]*[0-9]*[ ]+\.arch default
> +[ ]*[0-9]*[ ]+\?\?\?\? 0F4EC8[ ]+cmovng %eax, %ecx
> +#pass
> --- /dev/null
> +++ b/gas/testsuite/gas/i386/arch-dflt.s
> @@ -0,0 +1,14 @@
> + .text
> +start:
> + cmovl %eax, %ecx
> +
> + .arch default
> + cmovnl %eax, %ecx
> +
> + .arch generic32
> + cmovg %eax, %ecx
> +
> + .arch default
> + cmovng %eax, %ecx
> +
> + .end
> --- a/gas/testsuite/gas/i386/i386.exp
> +++ b/gas/testsuite/gas/i386/i386.exp
> @@ -205,6 +205,7 @@ if [gas_32_check] then {
> run_dump_test "arch-12"
> run_dump_test "arch-13"
> run_dump_test "arch-14"
> + run_list_test "arch-dflt" "-march=generic32 -al"
> run_dump_test "8087"
> run_dump_test "287"
> run_dump_test "387"
>
--
H.J.
^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH 4/7] x86: macro-ize cpu_arch[] entries
2022-06-30 12:52 [PATCH] gas/x86: .arch / -march= enhancements Jan Beulich
` (2 preceding siblings ...)
2022-06-30 12:53 ` [PATCH 3/7] x86: permit "default" with .arch Jan Beulich
@ 2022-06-30 12:54 ` Jan Beulich
2022-06-30 23:00 ` H.J. Lu
2022-06-30 12:54 ` [PATCH 5/7] x86: introduce fake processor type to mark sub-arch entries in cpu_arch[] Jan Beulich
` (2 subsequent siblings)
6 siblings, 1 reply; 17+ messages in thread
From: Jan Beulich @ 2022-06-30 12:54 UTC (permalink / raw)
To: Binutils
Putting individual elements behind macros, besides (imo) improving
readability, will make subsequent (and likely also future) changes less
intrusive.
Utilize this right away to pack the table a little more tightly, by
converting "skip" to bool and putting it earlier in a group of bitfields
together with "len".
--- a/gas/config/tc-i386.c
+++ b/gas/config/tc-i386.c
@@ -126,10 +126,10 @@ sib_byte;
typedef struct
{
const char *name; /* arch name */
- unsigned int len; /* arch string length */
+ unsigned int len:8; /* arch string length */
+ bool skip:1; /* show_arch should skip this. */
enum processor_type type; /* arch type */
i386_cpu_flags flags; /* cpu feature flags */
- unsigned int skip; /* show_arch should skip this. */
}
arch_entry;
@@ -931,318 +931,174 @@ const relax_typeS md_relax_table[] =
{0, 0, 4, 0}
};
+#define ARCH(n, t, f, s) \
+ { STRING_COMMA_LEN (#n), s, PROCESSOR_ ## t, CPU_ ## f ## _FLAGS }
+#define SUBARCH(n, e, s) \
+ { STRING_COMMA_LEN ("." #n), s, PROCESSOR_UNKNOWN, CPU_ ## e ## _FLAGS }
+
static const arch_entry cpu_arch[] =
{
/* Do not replace the first two entries - i386_target_format() and
set_cpu_arch() rely on them being there in this order. */
- { STRING_COMMA_LEN ("generic32"), PROCESSOR_GENERIC32,
- CPU_GENERIC32_FLAGS, 0 },
- { STRING_COMMA_LEN ("generic64"), PROCESSOR_GENERIC64,
- CPU_GENERIC64_FLAGS, 0 },
- { STRING_COMMA_LEN ("i8086"), PROCESSOR_UNKNOWN,
- CPU_NONE_FLAGS, 0 },
- { STRING_COMMA_LEN ("i186"), PROCESSOR_UNKNOWN,
- CPU_I186_FLAGS, 0 },
- { STRING_COMMA_LEN ("i286"), PROCESSOR_UNKNOWN,
- CPU_I286_FLAGS, 0 },
- { STRING_COMMA_LEN ("i386"), PROCESSOR_I386,
- CPU_I386_FLAGS, 0 },
- { STRING_COMMA_LEN ("i486"), PROCESSOR_I486,
- CPU_I486_FLAGS, 0 },
- { STRING_COMMA_LEN ("i586"), PROCESSOR_PENTIUM,
- CPU_I586_FLAGS, 0 },
- { STRING_COMMA_LEN ("i686"), PROCESSOR_PENTIUMPRO,
- CPU_I686_FLAGS, 0 },
- { STRING_COMMA_LEN ("pentium"), PROCESSOR_PENTIUM,
- CPU_I586_FLAGS, 0 },
- { STRING_COMMA_LEN ("pentiumpro"), PROCESSOR_PENTIUMPRO,
- CPU_PENTIUMPRO_FLAGS, 0 },
- { STRING_COMMA_LEN ("pentiumii"), PROCESSOR_PENTIUMPRO,
- CPU_P2_FLAGS, 0 },
- { STRING_COMMA_LEN ("pentiumiii"),PROCESSOR_PENTIUMPRO,
- CPU_P3_FLAGS, 0 },
- { STRING_COMMA_LEN ("pentium4"), PROCESSOR_PENTIUM4,
- CPU_P4_FLAGS, 0 },
- { STRING_COMMA_LEN ("prescott"), PROCESSOR_NOCONA,
- CPU_CORE_FLAGS, 0 },
- { STRING_COMMA_LEN ("nocona"), PROCESSOR_NOCONA,
- CPU_NOCONA_FLAGS, 0 },
- { STRING_COMMA_LEN ("yonah"), PROCESSOR_CORE,
- CPU_CORE_FLAGS, 1 },
- { STRING_COMMA_LEN ("core"), PROCESSOR_CORE,
- CPU_CORE_FLAGS, 0 },
- { STRING_COMMA_LEN ("merom"), PROCESSOR_CORE2,
- CPU_CORE2_FLAGS, 1 },
- { STRING_COMMA_LEN ("core2"), PROCESSOR_CORE2,
- CPU_CORE2_FLAGS, 0 },
- { STRING_COMMA_LEN ("corei7"), PROCESSOR_COREI7,
- CPU_COREI7_FLAGS, 0 },
- { STRING_COMMA_LEN ("iamcu"), PROCESSOR_IAMCU,
- CPU_IAMCU_FLAGS, 0 },
- { STRING_COMMA_LEN ("k6"), PROCESSOR_K6,
- CPU_K6_FLAGS, 0 },
- { STRING_COMMA_LEN ("k6_2"), PROCESSOR_K6,
- CPU_K6_2_FLAGS, 0 },
- { STRING_COMMA_LEN ("athlon"), PROCESSOR_ATHLON,
- CPU_ATHLON_FLAGS, 0 },
- { STRING_COMMA_LEN ("sledgehammer"), PROCESSOR_K8,
- CPU_K8_FLAGS, 1 },
- { STRING_COMMA_LEN ("opteron"), PROCESSOR_K8,
- CPU_K8_FLAGS, 0 },
- { STRING_COMMA_LEN ("k8"), PROCESSOR_K8,
- CPU_K8_FLAGS, 0 },
- { STRING_COMMA_LEN ("amdfam10"), PROCESSOR_AMDFAM10,
- CPU_AMDFAM10_FLAGS, 0 },
- { STRING_COMMA_LEN ("bdver1"), PROCESSOR_BD,
- CPU_BDVER1_FLAGS, 0 },
- { STRING_COMMA_LEN ("bdver2"), PROCESSOR_BD,
- CPU_BDVER2_FLAGS, 0 },
- { STRING_COMMA_LEN ("bdver3"), PROCESSOR_BD,
- CPU_BDVER3_FLAGS, 0 },
- { STRING_COMMA_LEN ("bdver4"), PROCESSOR_BD,
- CPU_BDVER4_FLAGS, 0 },
- { STRING_COMMA_LEN ("znver1"), PROCESSOR_ZNVER,
- CPU_ZNVER1_FLAGS, 0 },
- { STRING_COMMA_LEN ("znver2"), PROCESSOR_ZNVER,
- CPU_ZNVER2_FLAGS, 0 },
- { STRING_COMMA_LEN ("znver3"), PROCESSOR_ZNVER,
- CPU_ZNVER3_FLAGS, 0 },
- { STRING_COMMA_LEN ("btver1"), PROCESSOR_BT,
- CPU_BTVER1_FLAGS, 0 },
- { STRING_COMMA_LEN ("btver2"), PROCESSOR_BT,
- CPU_BTVER2_FLAGS, 0 },
- { STRING_COMMA_LEN (".8087"), PROCESSOR_UNKNOWN,
- CPU_8087_FLAGS, 0 },
- { STRING_COMMA_LEN (".287"), PROCESSOR_UNKNOWN,
- CPU_287_FLAGS, 0 },
- { STRING_COMMA_LEN (".387"), PROCESSOR_UNKNOWN,
- CPU_387_FLAGS, 0 },
- { STRING_COMMA_LEN (".687"), PROCESSOR_UNKNOWN,
- CPU_687_FLAGS, 0 },
- { STRING_COMMA_LEN (".cmov"), PROCESSOR_UNKNOWN,
- CPU_CMOV_FLAGS, 0 },
- { STRING_COMMA_LEN (".fxsr"), PROCESSOR_UNKNOWN,
- CPU_FXSR_FLAGS, 0 },
- { STRING_COMMA_LEN (".mmx"), PROCESSOR_UNKNOWN,
- CPU_MMX_FLAGS, 0 },
- { STRING_COMMA_LEN (".sse"), PROCESSOR_UNKNOWN,
- CPU_SSE_FLAGS, 0 },
- { STRING_COMMA_LEN (".sse2"), PROCESSOR_UNKNOWN,
- CPU_SSE2_FLAGS, 0 },
- { STRING_COMMA_LEN (".sse3"), PROCESSOR_UNKNOWN,
- CPU_SSE3_FLAGS, 0 },
- { STRING_COMMA_LEN (".sse4a"), PROCESSOR_UNKNOWN,
- CPU_SSE4A_FLAGS, 0 },
- { STRING_COMMA_LEN (".ssse3"), PROCESSOR_UNKNOWN,
- CPU_SSSE3_FLAGS, 0 },
- { STRING_COMMA_LEN (".sse4.1"), PROCESSOR_UNKNOWN,
- CPU_SSE4_1_FLAGS, 0 },
- { STRING_COMMA_LEN (".sse4.2"), PROCESSOR_UNKNOWN,
- CPU_SSE4_2_FLAGS, 0 },
- { STRING_COMMA_LEN (".sse4"), PROCESSOR_UNKNOWN,
- CPU_SSE4_2_FLAGS, 0 },
- { STRING_COMMA_LEN (".avx"), PROCESSOR_UNKNOWN,
- CPU_AVX_FLAGS, 0 },
- { STRING_COMMA_LEN (".avx2"), PROCESSOR_UNKNOWN,
- CPU_AVX2_FLAGS, 0 },
- { STRING_COMMA_LEN (".avx512f"), PROCESSOR_UNKNOWN,
- CPU_AVX512F_FLAGS, 0 },
- { STRING_COMMA_LEN (".avx512cd"), PROCESSOR_UNKNOWN,
- CPU_AVX512CD_FLAGS, 0 },
- { STRING_COMMA_LEN (".avx512er"), PROCESSOR_UNKNOWN,
- CPU_AVX512ER_FLAGS, 0 },
- { STRING_COMMA_LEN (".avx512pf"), PROCESSOR_UNKNOWN,
- CPU_AVX512PF_FLAGS, 0 },
- { STRING_COMMA_LEN (".avx512dq"), PROCESSOR_UNKNOWN,
- CPU_AVX512DQ_FLAGS, 0 },
- { STRING_COMMA_LEN (".avx512bw"), PROCESSOR_UNKNOWN,
- CPU_AVX512BW_FLAGS, 0 },
- { STRING_COMMA_LEN (".avx512vl"), PROCESSOR_UNKNOWN,
- CPU_AVX512VL_FLAGS, 0 },
- { STRING_COMMA_LEN (".vmx"), PROCESSOR_UNKNOWN,
- CPU_VMX_FLAGS, 0 },
- { STRING_COMMA_LEN (".vmfunc"), PROCESSOR_UNKNOWN,
- CPU_VMFUNC_FLAGS, 0 },
- { STRING_COMMA_LEN (".smx"), PROCESSOR_UNKNOWN,
- CPU_SMX_FLAGS, 0 },
- { STRING_COMMA_LEN (".xsave"), PROCESSOR_UNKNOWN,
- CPU_XSAVE_FLAGS, 0 },
- { STRING_COMMA_LEN (".xsaveopt"), PROCESSOR_UNKNOWN,
- CPU_XSAVEOPT_FLAGS, 0 },
- { STRING_COMMA_LEN (".xsavec"), PROCESSOR_UNKNOWN,
- CPU_XSAVEC_FLAGS, 0 },
- { STRING_COMMA_LEN (".xsaves"), PROCESSOR_UNKNOWN,
- CPU_XSAVES_FLAGS, 0 },
- { STRING_COMMA_LEN (".aes"), PROCESSOR_UNKNOWN,
- CPU_AES_FLAGS, 0 },
- { STRING_COMMA_LEN (".pclmul"), PROCESSOR_UNKNOWN,
- CPU_PCLMUL_FLAGS, 0 },
- { STRING_COMMA_LEN (".clmul"), PROCESSOR_UNKNOWN,
- CPU_PCLMUL_FLAGS, 1 },
- { STRING_COMMA_LEN (".fsgsbase"), PROCESSOR_UNKNOWN,
- CPU_FSGSBASE_FLAGS, 0 },
- { STRING_COMMA_LEN (".rdrnd"), PROCESSOR_UNKNOWN,
- CPU_RDRND_FLAGS, 0 },
- { STRING_COMMA_LEN (".f16c"), PROCESSOR_UNKNOWN,
- CPU_F16C_FLAGS, 0 },
- { STRING_COMMA_LEN (".bmi2"), PROCESSOR_UNKNOWN,
- CPU_BMI2_FLAGS, 0 },
- { STRING_COMMA_LEN (".fma"), PROCESSOR_UNKNOWN,
- CPU_FMA_FLAGS, 0 },
- { STRING_COMMA_LEN (".fma4"), PROCESSOR_UNKNOWN,
- CPU_FMA4_FLAGS, 0 },
- { STRING_COMMA_LEN (".xop"), PROCESSOR_UNKNOWN,
- CPU_XOP_FLAGS, 0 },
- { STRING_COMMA_LEN (".lwp"), PROCESSOR_UNKNOWN,
- CPU_LWP_FLAGS, 0 },
- { STRING_COMMA_LEN (".movbe"), PROCESSOR_UNKNOWN,
- CPU_MOVBE_FLAGS, 0 },
- { STRING_COMMA_LEN (".cx16"), PROCESSOR_UNKNOWN,
- CPU_CX16_FLAGS, 0 },
- { STRING_COMMA_LEN (".ept"), PROCESSOR_UNKNOWN,
- CPU_EPT_FLAGS, 0 },
- { STRING_COMMA_LEN (".lzcnt"), PROCESSOR_UNKNOWN,
- CPU_LZCNT_FLAGS, 0 },
- { STRING_COMMA_LEN (".popcnt"), PROCESSOR_UNKNOWN,
- CPU_POPCNT_FLAGS, 0 },
- { STRING_COMMA_LEN (".hle"), PROCESSOR_UNKNOWN,
- CPU_HLE_FLAGS, 0 },
- { STRING_COMMA_LEN (".rtm"), PROCESSOR_UNKNOWN,
- CPU_RTM_FLAGS, 0 },
- { STRING_COMMA_LEN (".invpcid"), PROCESSOR_UNKNOWN,
- CPU_INVPCID_FLAGS, 0 },
- { STRING_COMMA_LEN (".clflush"), PROCESSOR_UNKNOWN,
- CPU_CLFLUSH_FLAGS, 0 },
- { STRING_COMMA_LEN (".nop"), PROCESSOR_UNKNOWN,
- CPU_NOP_FLAGS, 0 },
- { STRING_COMMA_LEN (".syscall"), PROCESSOR_UNKNOWN,
- CPU_SYSCALL_FLAGS, 0 },
- { STRING_COMMA_LEN (".rdtscp"), PROCESSOR_UNKNOWN,
- CPU_RDTSCP_FLAGS, 0 },
- { STRING_COMMA_LEN (".3dnow"), PROCESSOR_UNKNOWN,
- CPU_3DNOW_FLAGS, 0 },
- { STRING_COMMA_LEN (".3dnowa"), PROCESSOR_UNKNOWN,
- CPU_3DNOWA_FLAGS, 0 },
- { STRING_COMMA_LEN (".padlock"), PROCESSOR_UNKNOWN,
- CPU_PADLOCK_FLAGS, 0 },
- { STRING_COMMA_LEN (".pacifica"), PROCESSOR_UNKNOWN,
- CPU_SVME_FLAGS, 1 },
- { STRING_COMMA_LEN (".svme"), PROCESSOR_UNKNOWN,
- CPU_SVME_FLAGS, 0 },
- { STRING_COMMA_LEN (".sse4a"), PROCESSOR_UNKNOWN,
- CPU_SSE4A_FLAGS, 0 },
- { STRING_COMMA_LEN (".abm"), PROCESSOR_UNKNOWN,
- CPU_ABM_FLAGS, 0 },
- { STRING_COMMA_LEN (".bmi"), PROCESSOR_UNKNOWN,
- CPU_BMI_FLAGS, 0 },
- { STRING_COMMA_LEN (".tbm"), PROCESSOR_UNKNOWN,
- CPU_TBM_FLAGS, 0 },
- { STRING_COMMA_LEN (".adx"), PROCESSOR_UNKNOWN,
- CPU_ADX_FLAGS, 0 },
- { STRING_COMMA_LEN (".rdseed"), PROCESSOR_UNKNOWN,
- CPU_RDSEED_FLAGS, 0 },
- { STRING_COMMA_LEN (".prfchw"), PROCESSOR_UNKNOWN,
- CPU_PRFCHW_FLAGS, 0 },
- { STRING_COMMA_LEN (".smap"), PROCESSOR_UNKNOWN,
- CPU_SMAP_FLAGS, 0 },
- { STRING_COMMA_LEN (".mpx"), PROCESSOR_UNKNOWN,
- CPU_MPX_FLAGS, 0 },
- { STRING_COMMA_LEN (".sha"), PROCESSOR_UNKNOWN,
- CPU_SHA_FLAGS, 0 },
- { STRING_COMMA_LEN (".clflushopt"), PROCESSOR_UNKNOWN,
- CPU_CLFLUSHOPT_FLAGS, 0 },
- { STRING_COMMA_LEN (".prefetchwt1"), PROCESSOR_UNKNOWN,
- CPU_PREFETCHWT1_FLAGS, 0 },
- { STRING_COMMA_LEN (".se1"), PROCESSOR_UNKNOWN,
- CPU_SE1_FLAGS, 0 },
- { STRING_COMMA_LEN (".clwb"), PROCESSOR_UNKNOWN,
- CPU_CLWB_FLAGS, 0 },
- { STRING_COMMA_LEN (".avx512ifma"), PROCESSOR_UNKNOWN,
- CPU_AVX512IFMA_FLAGS, 0 },
- { STRING_COMMA_LEN (".avx512vbmi"), PROCESSOR_UNKNOWN,
- CPU_AVX512VBMI_FLAGS, 0 },
- { STRING_COMMA_LEN (".avx512_4fmaps"), PROCESSOR_UNKNOWN,
- CPU_AVX512_4FMAPS_FLAGS, 0 },
- { STRING_COMMA_LEN (".avx512_4vnniw"), PROCESSOR_UNKNOWN,
- CPU_AVX512_4VNNIW_FLAGS, 0 },
- { STRING_COMMA_LEN (".avx512_vpopcntdq"), PROCESSOR_UNKNOWN,
- CPU_AVX512_VPOPCNTDQ_FLAGS, 0 },
- { STRING_COMMA_LEN (".avx512_vbmi2"), PROCESSOR_UNKNOWN,
- CPU_AVX512_VBMI2_FLAGS, 0 },
- { STRING_COMMA_LEN (".avx512_vnni"), PROCESSOR_UNKNOWN,
- CPU_AVX512_VNNI_FLAGS, 0 },
- { STRING_COMMA_LEN (".avx512_bitalg"), PROCESSOR_UNKNOWN,
- CPU_AVX512_BITALG_FLAGS, 0 },
- { STRING_COMMA_LEN (".avx_vnni"), PROCESSOR_UNKNOWN,
- CPU_AVX_VNNI_FLAGS, 0 },
- { STRING_COMMA_LEN (".clzero"), PROCESSOR_UNKNOWN,
- CPU_CLZERO_FLAGS, 0 },
- { STRING_COMMA_LEN (".mwaitx"), PROCESSOR_UNKNOWN,
- CPU_MWAITX_FLAGS, 0 },
- { STRING_COMMA_LEN (".ospke"), PROCESSOR_UNKNOWN,
- CPU_OSPKE_FLAGS, 0 },
- { STRING_COMMA_LEN (".rdpid"), PROCESSOR_UNKNOWN,
- CPU_RDPID_FLAGS, 0 },
- { STRING_COMMA_LEN (".ptwrite"), PROCESSOR_UNKNOWN,
- CPU_PTWRITE_FLAGS, 0 },
- { STRING_COMMA_LEN (".ibt"), PROCESSOR_UNKNOWN,
- CPU_IBT_FLAGS, 0 },
- { STRING_COMMA_LEN (".shstk"), PROCESSOR_UNKNOWN,
- CPU_SHSTK_FLAGS, 0 },
- { STRING_COMMA_LEN (".gfni"), PROCESSOR_UNKNOWN,
- CPU_GFNI_FLAGS, 0 },
- { STRING_COMMA_LEN (".vaes"), PROCESSOR_UNKNOWN,
- CPU_VAES_FLAGS, 0 },
- { STRING_COMMA_LEN (".vpclmulqdq"), PROCESSOR_UNKNOWN,
- CPU_VPCLMULQDQ_FLAGS, 0 },
- { STRING_COMMA_LEN (".wbnoinvd"), PROCESSOR_UNKNOWN,
- CPU_WBNOINVD_FLAGS, 0 },
- { STRING_COMMA_LEN (".pconfig"), PROCESSOR_UNKNOWN,
- CPU_PCONFIG_FLAGS, 0 },
- { STRING_COMMA_LEN (".waitpkg"), PROCESSOR_UNKNOWN,
- CPU_WAITPKG_FLAGS, 0 },
- { STRING_COMMA_LEN (".cldemote"), PROCESSOR_UNKNOWN,
- CPU_CLDEMOTE_FLAGS, 0 },
- { STRING_COMMA_LEN (".amx_int8"), PROCESSOR_UNKNOWN,
- CPU_AMX_INT8_FLAGS, 0 },
- { STRING_COMMA_LEN (".amx_bf16"), PROCESSOR_UNKNOWN,
- CPU_AMX_BF16_FLAGS, 0 },
- { STRING_COMMA_LEN (".amx_tile"), PROCESSOR_UNKNOWN,
- CPU_AMX_TILE_FLAGS, 0 },
- { STRING_COMMA_LEN (".movdiri"), PROCESSOR_UNKNOWN,
- CPU_MOVDIRI_FLAGS, 0 },
- { STRING_COMMA_LEN (".movdir64b"), PROCESSOR_UNKNOWN,
- CPU_MOVDIR64B_FLAGS, 0 },
- { STRING_COMMA_LEN (".avx512_bf16"), PROCESSOR_UNKNOWN,
- CPU_AVX512_BF16_FLAGS, 0 },
- { STRING_COMMA_LEN (".avx512_vp2intersect"), PROCESSOR_UNKNOWN,
- CPU_AVX512_VP2INTERSECT_FLAGS, 0 },
- { STRING_COMMA_LEN (".tdx"), PROCESSOR_UNKNOWN,
- CPU_TDX_FLAGS, 0 },
- { STRING_COMMA_LEN (".enqcmd"), PROCESSOR_UNKNOWN,
- CPU_ENQCMD_FLAGS, 0 },
- { STRING_COMMA_LEN (".serialize"), PROCESSOR_UNKNOWN,
- CPU_SERIALIZE_FLAGS, 0 },
- { STRING_COMMA_LEN (".rdpru"), PROCESSOR_UNKNOWN,
- CPU_RDPRU_FLAGS, 0 },
- { STRING_COMMA_LEN (".mcommit"), PROCESSOR_UNKNOWN,
- CPU_MCOMMIT_FLAGS, 0 },
- { STRING_COMMA_LEN (".sev_es"), PROCESSOR_UNKNOWN,
- CPU_SEV_ES_FLAGS, 0 },
- { STRING_COMMA_LEN (".tsxldtrk"), PROCESSOR_UNKNOWN,
- CPU_TSXLDTRK_FLAGS, 0 },
- { STRING_COMMA_LEN (".kl"), PROCESSOR_UNKNOWN,
- CPU_KL_FLAGS, 0 },
- { STRING_COMMA_LEN (".widekl"), PROCESSOR_UNKNOWN,
- CPU_WIDEKL_FLAGS, 0 },
- { STRING_COMMA_LEN (".uintr"), PROCESSOR_UNKNOWN,
- CPU_UINTR_FLAGS, 0 },
- { STRING_COMMA_LEN (".hreset"), PROCESSOR_UNKNOWN,
- CPU_HRESET_FLAGS, 0 },
- { STRING_COMMA_LEN (".avx512_fp16"), PROCESSOR_UNKNOWN,
- CPU_AVX512_FP16_FLAGS, 0 },
+ ARCH (generic32, GENERIC32, GENERIC32, false),
+ ARCH (generic64, GENERIC64, GENERIC64, false),
+ ARCH (i8086, UNKNOWN, NONE, false),
+ ARCH (i186, UNKNOWN, I186, false),
+ ARCH (i286, UNKNOWN, I286, false),
+ ARCH (i386, I386, I386, false),
+ ARCH (i486, I486, I486, false),
+ ARCH (i586, PENTIUM, I586, false),
+ ARCH (i686, PENTIUMPRO, I686, false),
+ ARCH (pentium, PENTIUM, I586, false),
+ ARCH (pentiumpro, PENTIUMPRO, PENTIUMPRO, false),
+ ARCH (pentiumii, PENTIUMPRO, P2, false),
+ ARCH (pentiumiii, PENTIUMPRO, P3, false),
+ ARCH (pentium4, PENTIUM4, P4, false),
+ ARCH (prescott, NOCONA, CORE, false),
+ ARCH (nocona, NOCONA, NOCONA, false),
+ ARCH (yonah, CORE, CORE, true),
+ ARCH (core, CORE, CORE, false),
+ ARCH (merom, CORE2, CORE2, true),
+ ARCH (core2, CORE2, CORE2, false),
+ ARCH (corei7, COREI7, COREI7, false),
+ ARCH (iamcu, IAMCU, IAMCU, false),
+ ARCH (k6, K6, K6, false),
+ ARCH (k6_2, K6, K6_2, false),
+ ARCH (athlon, ATHLON, ATHLON, false),
+ ARCH (sledgehammer, K8, K8, true),
+ ARCH (opteron, K8, K8, false),
+ ARCH (k8, K8, K8, false),
+ ARCH (amdfam10, AMDFAM10, AMDFAM10, false),
+ ARCH (bdver1, BD, BDVER1, false),
+ ARCH (bdver2, BD, BDVER2, false),
+ ARCH (bdver3, BD, BDVER3, false),
+ ARCH (bdver4, BD, BDVER4, false),
+ ARCH (znver1, ZNVER, ZNVER1, false),
+ ARCH (znver2, ZNVER, ZNVER2, false),
+ ARCH (znver3, ZNVER, ZNVER3, false),
+ ARCH (btver1, BT, BTVER1, false),
+ ARCH (btver2, BT, BTVER2, false),
+
+ SUBARCH (8087, 8087, false),
+ SUBARCH (287, 287, false),
+ SUBARCH (387, 387, false),
+ SUBARCH (687, 687, false),
+ SUBARCH (cmov, CMOV, false),
+ SUBARCH (fxsr, FXSR, false),
+ SUBARCH (mmx, MMX, false),
+ SUBARCH (sse, SSE, false),
+ SUBARCH (sse2, SSE2, false),
+ SUBARCH (sse3, SSE3, false),
+ SUBARCH (sse4a, SSE4A, false),
+ SUBARCH (ssse3, SSSE3, false),
+ SUBARCH (sse4.1, SSE4_1, false),
+ SUBARCH (sse4.2, SSE4_2, false),
+ SUBARCH (sse4, SSE4_2, false),
+ SUBARCH (avx, AVX, false),
+ SUBARCH (avx2, AVX2, false),
+ SUBARCH (avx512f, AVX512F, false),
+ SUBARCH (avx512cd, AVX512CD, false),
+ SUBARCH (avx512er, AVX512ER, false),
+ SUBARCH (avx512pf, AVX512PF, false),
+ SUBARCH (avx512dq, AVX512DQ, false),
+ SUBARCH (avx512bw, AVX512BW, false),
+ SUBARCH (avx512vl, AVX512VL, false),
+ SUBARCH (vmx, VMX, false),
+ SUBARCH (vmfunc, VMFUNC, false),
+ SUBARCH (smx, SMX, false),
+ SUBARCH (xsave, XSAVE, false),
+ SUBARCH (xsaveopt, XSAVEOPT, false),
+ SUBARCH (xsavec, XSAVEC, false),
+ SUBARCH (xsaves, XSAVES, false),
+ SUBARCH (aes, AES, false),
+ SUBARCH (pclmul, PCLMUL, false),
+ SUBARCH (clmul, PCLMUL, true),
+ SUBARCH (fsgsbase, FSGSBASE, false),
+ SUBARCH (rdrnd, RDRND, false),
+ SUBARCH (f16c, F16C, false),
+ SUBARCH (bmi2, BMI2, false),
+ SUBARCH (fma, FMA, false),
+ SUBARCH (fma4, FMA4, false),
+ SUBARCH (xop, XOP, false),
+ SUBARCH (lwp, LWP, false),
+ SUBARCH (movbe, MOVBE, false),
+ SUBARCH (cx16, CX16, false),
+ SUBARCH (ept, EPT, false),
+ SUBARCH (lzcnt, LZCNT, false),
+ SUBARCH (popcnt, POPCNT, false),
+ SUBARCH (hle, HLE, false),
+ SUBARCH (rtm, RTM, false),
+ SUBARCH (invpcid, INVPCID, false),
+ SUBARCH (clflush, CLFLUSH, false),
+ SUBARCH (nop, NOP, false),
+ SUBARCH (syscall, SYSCALL, false),
+ SUBARCH (rdtscp, RDTSCP, false),
+ SUBARCH (3dnow, 3DNOW, false),
+ SUBARCH (3dnowa, 3DNOWA, false),
+ SUBARCH (padlock, PADLOCK, false),
+ SUBARCH (pacifica, SVME, true),
+ SUBARCH (svme, SVME, false),
+ SUBARCH (sse4a, SSE4A, false),
+ SUBARCH (abm, ABM, false),
+ SUBARCH (bmi, BMI, false),
+ SUBARCH (tbm, TBM, false),
+ SUBARCH (adx, ADX, false),
+ SUBARCH (rdseed, RDSEED, false),
+ SUBARCH (prfchw, PRFCHW, false),
+ SUBARCH (smap, SMAP, false),
+ SUBARCH (mpx, MPX, false),
+ SUBARCH (sha, SHA, false),
+ SUBARCH (clflushopt, CLFLUSHOPT, false),
+ SUBARCH (prefetchwt1, PREFETCHWT1, false),
+ SUBARCH (se1, SE1, false),
+ SUBARCH (clwb, CLWB, false),
+ SUBARCH (avx512ifma, AVX512IFMA, false),
+ SUBARCH (avx512vbmi, AVX512VBMI, false),
+ SUBARCH (avx512_4fmaps, AVX512_4FMAPS, false),
+ SUBARCH (avx512_4vnniw, AVX512_4VNNIW, false),
+ SUBARCH (avx512_vpopcntdq, AVX512_VPOPCNTDQ, false),
+ SUBARCH (avx512_vbmi2, AVX512_VBMI2, false),
+ SUBARCH (avx512_vnni, AVX512_VNNI, false),
+ SUBARCH (avx512_bitalg, AVX512_BITALG, false),
+ SUBARCH (avx_vnni, AVX_VNNI, false),
+ SUBARCH (clzero, CLZERO, false),
+ SUBARCH (mwaitx, MWAITX, false),
+ SUBARCH (ospke, OSPKE, false),
+ SUBARCH (rdpid, RDPID, false),
+ SUBARCH (ptwrite, PTWRITE, false),
+ SUBARCH (ibt, IBT, false),
+ SUBARCH (shstk, SHSTK, false),
+ SUBARCH (gfni, GFNI, false),
+ SUBARCH (vaes, VAES, false),
+ SUBARCH (vpclmulqdq, VPCLMULQDQ, false),
+ SUBARCH (wbnoinvd, WBNOINVD, false),
+ SUBARCH (pconfig, PCONFIG, false),
+ SUBARCH (waitpkg, WAITPKG, false),
+ SUBARCH (cldemote, CLDEMOTE, false),
+ SUBARCH (amx_int8, AMX_INT8, false),
+ SUBARCH (amx_bf16, AMX_BF16, false),
+ SUBARCH (amx_tile, AMX_TILE, false),
+ SUBARCH (movdiri, MOVDIRI, false),
+ SUBARCH (movdir64b, MOVDIR64B, false),
+ SUBARCH (avx512_bf16, AVX512_BF16, false),
+ SUBARCH (avx512_vp2intersect, AVX512_VP2INTERSECT, false),
+ SUBARCH (tdx, TDX, false),
+ SUBARCH (enqcmd, ENQCMD, false),
+ SUBARCH (serialize, SERIALIZE, false),
+ SUBARCH (rdpru, RDPRU, false),
+ SUBARCH (mcommit, MCOMMIT, false),
+ SUBARCH (sev_es, SEV_ES, false),
+ SUBARCH (tsxldtrk, TSXLDTRK, false),
+ SUBARCH (kl, KL, false),
+ SUBARCH (widekl, WIDEKL, false),
+ SUBARCH (uintr, UINTR, false),
+ SUBARCH (hreset, HRESET, false),
+ SUBARCH (avx512_fp16, AVX512_FP16, false),
};
+#undef SUBARCH
+#undef ARCH
+
static const noarch_entry cpu_noarch[] =
{
{ STRING_COMMA_LEN ("no87"), CPU_ANY_X87_FLAGS },
^ permalink raw reply [flat|nested] 17+ messages in thread* Re: [PATCH 4/7] x86: macro-ize cpu_arch[] entries
2022-06-30 12:54 ` [PATCH 4/7] x86: macro-ize cpu_arch[] entries Jan Beulich
@ 2022-06-30 23:00 ` H.J. Lu
0 siblings, 0 replies; 17+ messages in thread
From: H.J. Lu @ 2022-06-30 23:00 UTC (permalink / raw)
To: Jan Beulich; +Cc: Binutils
On Thu, Jun 30, 2022 at 5:54 AM Jan Beulich <jbeulich@suse.com> wrote:
>
> Putting individual elements behind macros, besides (imo) improving
> readability, will make subsequent (and likely also future) changes less
> intrusive.
>
> Utilize this right away to pack the table a little more tightly, by
> converting "skip" to bool and putting it earlier in a group of bitfields
> together with "len".
>
> --- a/gas/config/tc-i386.c
> +++ b/gas/config/tc-i386.c
> @@ -126,10 +126,10 @@ sib_byte;
> typedef struct
> {
> const char *name; /* arch name */
> - unsigned int len; /* arch string length */
> + unsigned int len:8; /* arch string length */
> + bool skip:1; /* show_arch should skip this. */
> enum processor_type type; /* arch type */
> i386_cpu_flags flags; /* cpu feature flags */
> - unsigned int skip; /* show_arch should skip this. */
> }
> arch_entry;
>
> @@ -931,318 +931,174 @@ const relax_typeS md_relax_table[] =
> {0, 0, 4, 0}
> };
>
> +#define ARCH(n, t, f, s) \
> + { STRING_COMMA_LEN (#n), s, PROCESSOR_ ## t, CPU_ ## f ## _FLAGS }
> +#define SUBARCH(n, e, s) \
> + { STRING_COMMA_LEN ("." #n), s, PROCESSOR_UNKNOWN, CPU_ ## e ## _FLAGS }
> +
> static const arch_entry cpu_arch[] =
> {
> /* Do not replace the first two entries - i386_target_format() and
> set_cpu_arch() rely on them being there in this order. */
> - { STRING_COMMA_LEN ("generic32"), PROCESSOR_GENERIC32,
> - CPU_GENERIC32_FLAGS, 0 },
> - { STRING_COMMA_LEN ("generic64"), PROCESSOR_GENERIC64,
> - CPU_GENERIC64_FLAGS, 0 },
> - { STRING_COMMA_LEN ("i8086"), PROCESSOR_UNKNOWN,
> - CPU_NONE_FLAGS, 0 },
> - { STRING_COMMA_LEN ("i186"), PROCESSOR_UNKNOWN,
> - CPU_I186_FLAGS, 0 },
> - { STRING_COMMA_LEN ("i286"), PROCESSOR_UNKNOWN,
> - CPU_I286_FLAGS, 0 },
> - { STRING_COMMA_LEN ("i386"), PROCESSOR_I386,
> - CPU_I386_FLAGS, 0 },
> - { STRING_COMMA_LEN ("i486"), PROCESSOR_I486,
> - CPU_I486_FLAGS, 0 },
> - { STRING_COMMA_LEN ("i586"), PROCESSOR_PENTIUM,
> - CPU_I586_FLAGS, 0 },
> - { STRING_COMMA_LEN ("i686"), PROCESSOR_PENTIUMPRO,
> - CPU_I686_FLAGS, 0 },
> - { STRING_COMMA_LEN ("pentium"), PROCESSOR_PENTIUM,
> - CPU_I586_FLAGS, 0 },
> - { STRING_COMMA_LEN ("pentiumpro"), PROCESSOR_PENTIUMPRO,
> - CPU_PENTIUMPRO_FLAGS, 0 },
> - { STRING_COMMA_LEN ("pentiumii"), PROCESSOR_PENTIUMPRO,
> - CPU_P2_FLAGS, 0 },
> - { STRING_COMMA_LEN ("pentiumiii"),PROCESSOR_PENTIUMPRO,
> - CPU_P3_FLAGS, 0 },
> - { STRING_COMMA_LEN ("pentium4"), PROCESSOR_PENTIUM4,
> - CPU_P4_FLAGS, 0 },
> - { STRING_COMMA_LEN ("prescott"), PROCESSOR_NOCONA,
> - CPU_CORE_FLAGS, 0 },
> - { STRING_COMMA_LEN ("nocona"), PROCESSOR_NOCONA,
> - CPU_NOCONA_FLAGS, 0 },
> - { STRING_COMMA_LEN ("yonah"), PROCESSOR_CORE,
> - CPU_CORE_FLAGS, 1 },
> - { STRING_COMMA_LEN ("core"), PROCESSOR_CORE,
> - CPU_CORE_FLAGS, 0 },
> - { STRING_COMMA_LEN ("merom"), PROCESSOR_CORE2,
> - CPU_CORE2_FLAGS, 1 },
> - { STRING_COMMA_LEN ("core2"), PROCESSOR_CORE2,
> - CPU_CORE2_FLAGS, 0 },
> - { STRING_COMMA_LEN ("corei7"), PROCESSOR_COREI7,
> - CPU_COREI7_FLAGS, 0 },
> - { STRING_COMMA_LEN ("iamcu"), PROCESSOR_IAMCU,
> - CPU_IAMCU_FLAGS, 0 },
> - { STRING_COMMA_LEN ("k6"), PROCESSOR_K6,
> - CPU_K6_FLAGS, 0 },
> - { STRING_COMMA_LEN ("k6_2"), PROCESSOR_K6,
> - CPU_K6_2_FLAGS, 0 },
> - { STRING_COMMA_LEN ("athlon"), PROCESSOR_ATHLON,
> - CPU_ATHLON_FLAGS, 0 },
> - { STRING_COMMA_LEN ("sledgehammer"), PROCESSOR_K8,
> - CPU_K8_FLAGS, 1 },
> - { STRING_COMMA_LEN ("opteron"), PROCESSOR_K8,
> - CPU_K8_FLAGS, 0 },
> - { STRING_COMMA_LEN ("k8"), PROCESSOR_K8,
> - CPU_K8_FLAGS, 0 },
> - { STRING_COMMA_LEN ("amdfam10"), PROCESSOR_AMDFAM10,
> - CPU_AMDFAM10_FLAGS, 0 },
> - { STRING_COMMA_LEN ("bdver1"), PROCESSOR_BD,
> - CPU_BDVER1_FLAGS, 0 },
> - { STRING_COMMA_LEN ("bdver2"), PROCESSOR_BD,
> - CPU_BDVER2_FLAGS, 0 },
> - { STRING_COMMA_LEN ("bdver3"), PROCESSOR_BD,
> - CPU_BDVER3_FLAGS, 0 },
> - { STRING_COMMA_LEN ("bdver4"), PROCESSOR_BD,
> - CPU_BDVER4_FLAGS, 0 },
> - { STRING_COMMA_LEN ("znver1"), PROCESSOR_ZNVER,
> - CPU_ZNVER1_FLAGS, 0 },
> - { STRING_COMMA_LEN ("znver2"), PROCESSOR_ZNVER,
> - CPU_ZNVER2_FLAGS, 0 },
> - { STRING_COMMA_LEN ("znver3"), PROCESSOR_ZNVER,
> - CPU_ZNVER3_FLAGS, 0 },
> - { STRING_COMMA_LEN ("btver1"), PROCESSOR_BT,
> - CPU_BTVER1_FLAGS, 0 },
> - { STRING_COMMA_LEN ("btver2"), PROCESSOR_BT,
> - CPU_BTVER2_FLAGS, 0 },
> - { STRING_COMMA_LEN (".8087"), PROCESSOR_UNKNOWN,
> - CPU_8087_FLAGS, 0 },
> - { STRING_COMMA_LEN (".287"), PROCESSOR_UNKNOWN,
> - CPU_287_FLAGS, 0 },
> - { STRING_COMMA_LEN (".387"), PROCESSOR_UNKNOWN,
> - CPU_387_FLAGS, 0 },
> - { STRING_COMMA_LEN (".687"), PROCESSOR_UNKNOWN,
> - CPU_687_FLAGS, 0 },
> - { STRING_COMMA_LEN (".cmov"), PROCESSOR_UNKNOWN,
> - CPU_CMOV_FLAGS, 0 },
> - { STRING_COMMA_LEN (".fxsr"), PROCESSOR_UNKNOWN,
> - CPU_FXSR_FLAGS, 0 },
> - { STRING_COMMA_LEN (".mmx"), PROCESSOR_UNKNOWN,
> - CPU_MMX_FLAGS, 0 },
> - { STRING_COMMA_LEN (".sse"), PROCESSOR_UNKNOWN,
> - CPU_SSE_FLAGS, 0 },
> - { STRING_COMMA_LEN (".sse2"), PROCESSOR_UNKNOWN,
> - CPU_SSE2_FLAGS, 0 },
> - { STRING_COMMA_LEN (".sse3"), PROCESSOR_UNKNOWN,
> - CPU_SSE3_FLAGS, 0 },
> - { STRING_COMMA_LEN (".sse4a"), PROCESSOR_UNKNOWN,
> - CPU_SSE4A_FLAGS, 0 },
> - { STRING_COMMA_LEN (".ssse3"), PROCESSOR_UNKNOWN,
> - CPU_SSSE3_FLAGS, 0 },
> - { STRING_COMMA_LEN (".sse4.1"), PROCESSOR_UNKNOWN,
> - CPU_SSE4_1_FLAGS, 0 },
> - { STRING_COMMA_LEN (".sse4.2"), PROCESSOR_UNKNOWN,
> - CPU_SSE4_2_FLAGS, 0 },
> - { STRING_COMMA_LEN (".sse4"), PROCESSOR_UNKNOWN,
> - CPU_SSE4_2_FLAGS, 0 },
> - { STRING_COMMA_LEN (".avx"), PROCESSOR_UNKNOWN,
> - CPU_AVX_FLAGS, 0 },
> - { STRING_COMMA_LEN (".avx2"), PROCESSOR_UNKNOWN,
> - CPU_AVX2_FLAGS, 0 },
> - { STRING_COMMA_LEN (".avx512f"), PROCESSOR_UNKNOWN,
> - CPU_AVX512F_FLAGS, 0 },
> - { STRING_COMMA_LEN (".avx512cd"), PROCESSOR_UNKNOWN,
> - CPU_AVX512CD_FLAGS, 0 },
> - { STRING_COMMA_LEN (".avx512er"), PROCESSOR_UNKNOWN,
> - CPU_AVX512ER_FLAGS, 0 },
> - { STRING_COMMA_LEN (".avx512pf"), PROCESSOR_UNKNOWN,
> - CPU_AVX512PF_FLAGS, 0 },
> - { STRING_COMMA_LEN (".avx512dq"), PROCESSOR_UNKNOWN,
> - CPU_AVX512DQ_FLAGS, 0 },
> - { STRING_COMMA_LEN (".avx512bw"), PROCESSOR_UNKNOWN,
> - CPU_AVX512BW_FLAGS, 0 },
> - { STRING_COMMA_LEN (".avx512vl"), PROCESSOR_UNKNOWN,
> - CPU_AVX512VL_FLAGS, 0 },
> - { STRING_COMMA_LEN (".vmx"), PROCESSOR_UNKNOWN,
> - CPU_VMX_FLAGS, 0 },
> - { STRING_COMMA_LEN (".vmfunc"), PROCESSOR_UNKNOWN,
> - CPU_VMFUNC_FLAGS, 0 },
> - { STRING_COMMA_LEN (".smx"), PROCESSOR_UNKNOWN,
> - CPU_SMX_FLAGS, 0 },
> - { STRING_COMMA_LEN (".xsave"), PROCESSOR_UNKNOWN,
> - CPU_XSAVE_FLAGS, 0 },
> - { STRING_COMMA_LEN (".xsaveopt"), PROCESSOR_UNKNOWN,
> - CPU_XSAVEOPT_FLAGS, 0 },
> - { STRING_COMMA_LEN (".xsavec"), PROCESSOR_UNKNOWN,
> - CPU_XSAVEC_FLAGS, 0 },
> - { STRING_COMMA_LEN (".xsaves"), PROCESSOR_UNKNOWN,
> - CPU_XSAVES_FLAGS, 0 },
> - { STRING_COMMA_LEN (".aes"), PROCESSOR_UNKNOWN,
> - CPU_AES_FLAGS, 0 },
> - { STRING_COMMA_LEN (".pclmul"), PROCESSOR_UNKNOWN,
> - CPU_PCLMUL_FLAGS, 0 },
> - { STRING_COMMA_LEN (".clmul"), PROCESSOR_UNKNOWN,
> - CPU_PCLMUL_FLAGS, 1 },
> - { STRING_COMMA_LEN (".fsgsbase"), PROCESSOR_UNKNOWN,
> - CPU_FSGSBASE_FLAGS, 0 },
> - { STRING_COMMA_LEN (".rdrnd"), PROCESSOR_UNKNOWN,
> - CPU_RDRND_FLAGS, 0 },
> - { STRING_COMMA_LEN (".f16c"), PROCESSOR_UNKNOWN,
> - CPU_F16C_FLAGS, 0 },
> - { STRING_COMMA_LEN (".bmi2"), PROCESSOR_UNKNOWN,
> - CPU_BMI2_FLAGS, 0 },
> - { STRING_COMMA_LEN (".fma"), PROCESSOR_UNKNOWN,
> - CPU_FMA_FLAGS, 0 },
> - { STRING_COMMA_LEN (".fma4"), PROCESSOR_UNKNOWN,
> - CPU_FMA4_FLAGS, 0 },
> - { STRING_COMMA_LEN (".xop"), PROCESSOR_UNKNOWN,
> - CPU_XOP_FLAGS, 0 },
> - { STRING_COMMA_LEN (".lwp"), PROCESSOR_UNKNOWN,
> - CPU_LWP_FLAGS, 0 },
> - { STRING_COMMA_LEN (".movbe"), PROCESSOR_UNKNOWN,
> - CPU_MOVBE_FLAGS, 0 },
> - { STRING_COMMA_LEN (".cx16"), PROCESSOR_UNKNOWN,
> - CPU_CX16_FLAGS, 0 },
> - { STRING_COMMA_LEN (".ept"), PROCESSOR_UNKNOWN,
> - CPU_EPT_FLAGS, 0 },
> - { STRING_COMMA_LEN (".lzcnt"), PROCESSOR_UNKNOWN,
> - CPU_LZCNT_FLAGS, 0 },
> - { STRING_COMMA_LEN (".popcnt"), PROCESSOR_UNKNOWN,
> - CPU_POPCNT_FLAGS, 0 },
> - { STRING_COMMA_LEN (".hle"), PROCESSOR_UNKNOWN,
> - CPU_HLE_FLAGS, 0 },
> - { STRING_COMMA_LEN (".rtm"), PROCESSOR_UNKNOWN,
> - CPU_RTM_FLAGS, 0 },
> - { STRING_COMMA_LEN (".invpcid"), PROCESSOR_UNKNOWN,
> - CPU_INVPCID_FLAGS, 0 },
> - { STRING_COMMA_LEN (".clflush"), PROCESSOR_UNKNOWN,
> - CPU_CLFLUSH_FLAGS, 0 },
> - { STRING_COMMA_LEN (".nop"), PROCESSOR_UNKNOWN,
> - CPU_NOP_FLAGS, 0 },
> - { STRING_COMMA_LEN (".syscall"), PROCESSOR_UNKNOWN,
> - CPU_SYSCALL_FLAGS, 0 },
> - { STRING_COMMA_LEN (".rdtscp"), PROCESSOR_UNKNOWN,
> - CPU_RDTSCP_FLAGS, 0 },
> - { STRING_COMMA_LEN (".3dnow"), PROCESSOR_UNKNOWN,
> - CPU_3DNOW_FLAGS, 0 },
> - { STRING_COMMA_LEN (".3dnowa"), PROCESSOR_UNKNOWN,
> - CPU_3DNOWA_FLAGS, 0 },
> - { STRING_COMMA_LEN (".padlock"), PROCESSOR_UNKNOWN,
> - CPU_PADLOCK_FLAGS, 0 },
> - { STRING_COMMA_LEN (".pacifica"), PROCESSOR_UNKNOWN,
> - CPU_SVME_FLAGS, 1 },
> - { STRING_COMMA_LEN (".svme"), PROCESSOR_UNKNOWN,
> - CPU_SVME_FLAGS, 0 },
> - { STRING_COMMA_LEN (".sse4a"), PROCESSOR_UNKNOWN,
> - CPU_SSE4A_FLAGS, 0 },
> - { STRING_COMMA_LEN (".abm"), PROCESSOR_UNKNOWN,
> - CPU_ABM_FLAGS, 0 },
> - { STRING_COMMA_LEN (".bmi"), PROCESSOR_UNKNOWN,
> - CPU_BMI_FLAGS, 0 },
> - { STRING_COMMA_LEN (".tbm"), PROCESSOR_UNKNOWN,
> - CPU_TBM_FLAGS, 0 },
> - { STRING_COMMA_LEN (".adx"), PROCESSOR_UNKNOWN,
> - CPU_ADX_FLAGS, 0 },
> - { STRING_COMMA_LEN (".rdseed"), PROCESSOR_UNKNOWN,
> - CPU_RDSEED_FLAGS, 0 },
> - { STRING_COMMA_LEN (".prfchw"), PROCESSOR_UNKNOWN,
> - CPU_PRFCHW_FLAGS, 0 },
> - { STRING_COMMA_LEN (".smap"), PROCESSOR_UNKNOWN,
> - CPU_SMAP_FLAGS, 0 },
> - { STRING_COMMA_LEN (".mpx"), PROCESSOR_UNKNOWN,
> - CPU_MPX_FLAGS, 0 },
> - { STRING_COMMA_LEN (".sha"), PROCESSOR_UNKNOWN,
> - CPU_SHA_FLAGS, 0 },
> - { STRING_COMMA_LEN (".clflushopt"), PROCESSOR_UNKNOWN,
> - CPU_CLFLUSHOPT_FLAGS, 0 },
> - { STRING_COMMA_LEN (".prefetchwt1"), PROCESSOR_UNKNOWN,
> - CPU_PREFETCHWT1_FLAGS, 0 },
> - { STRING_COMMA_LEN (".se1"), PROCESSOR_UNKNOWN,
> - CPU_SE1_FLAGS, 0 },
> - { STRING_COMMA_LEN (".clwb"), PROCESSOR_UNKNOWN,
> - CPU_CLWB_FLAGS, 0 },
> - { STRING_COMMA_LEN (".avx512ifma"), PROCESSOR_UNKNOWN,
> - CPU_AVX512IFMA_FLAGS, 0 },
> - { STRING_COMMA_LEN (".avx512vbmi"), PROCESSOR_UNKNOWN,
> - CPU_AVX512VBMI_FLAGS, 0 },
> - { STRING_COMMA_LEN (".avx512_4fmaps"), PROCESSOR_UNKNOWN,
> - CPU_AVX512_4FMAPS_FLAGS, 0 },
> - { STRING_COMMA_LEN (".avx512_4vnniw"), PROCESSOR_UNKNOWN,
> - CPU_AVX512_4VNNIW_FLAGS, 0 },
> - { STRING_COMMA_LEN (".avx512_vpopcntdq"), PROCESSOR_UNKNOWN,
> - CPU_AVX512_VPOPCNTDQ_FLAGS, 0 },
> - { STRING_COMMA_LEN (".avx512_vbmi2"), PROCESSOR_UNKNOWN,
> - CPU_AVX512_VBMI2_FLAGS, 0 },
> - { STRING_COMMA_LEN (".avx512_vnni"), PROCESSOR_UNKNOWN,
> - CPU_AVX512_VNNI_FLAGS, 0 },
> - { STRING_COMMA_LEN (".avx512_bitalg"), PROCESSOR_UNKNOWN,
> - CPU_AVX512_BITALG_FLAGS, 0 },
> - { STRING_COMMA_LEN (".avx_vnni"), PROCESSOR_UNKNOWN,
> - CPU_AVX_VNNI_FLAGS, 0 },
> - { STRING_COMMA_LEN (".clzero"), PROCESSOR_UNKNOWN,
> - CPU_CLZERO_FLAGS, 0 },
> - { STRING_COMMA_LEN (".mwaitx"), PROCESSOR_UNKNOWN,
> - CPU_MWAITX_FLAGS, 0 },
> - { STRING_COMMA_LEN (".ospke"), PROCESSOR_UNKNOWN,
> - CPU_OSPKE_FLAGS, 0 },
> - { STRING_COMMA_LEN (".rdpid"), PROCESSOR_UNKNOWN,
> - CPU_RDPID_FLAGS, 0 },
> - { STRING_COMMA_LEN (".ptwrite"), PROCESSOR_UNKNOWN,
> - CPU_PTWRITE_FLAGS, 0 },
> - { STRING_COMMA_LEN (".ibt"), PROCESSOR_UNKNOWN,
> - CPU_IBT_FLAGS, 0 },
> - { STRING_COMMA_LEN (".shstk"), PROCESSOR_UNKNOWN,
> - CPU_SHSTK_FLAGS, 0 },
> - { STRING_COMMA_LEN (".gfni"), PROCESSOR_UNKNOWN,
> - CPU_GFNI_FLAGS, 0 },
> - { STRING_COMMA_LEN (".vaes"), PROCESSOR_UNKNOWN,
> - CPU_VAES_FLAGS, 0 },
> - { STRING_COMMA_LEN (".vpclmulqdq"), PROCESSOR_UNKNOWN,
> - CPU_VPCLMULQDQ_FLAGS, 0 },
> - { STRING_COMMA_LEN (".wbnoinvd"), PROCESSOR_UNKNOWN,
> - CPU_WBNOINVD_FLAGS, 0 },
> - { STRING_COMMA_LEN (".pconfig"), PROCESSOR_UNKNOWN,
> - CPU_PCONFIG_FLAGS, 0 },
> - { STRING_COMMA_LEN (".waitpkg"), PROCESSOR_UNKNOWN,
> - CPU_WAITPKG_FLAGS, 0 },
> - { STRING_COMMA_LEN (".cldemote"), PROCESSOR_UNKNOWN,
> - CPU_CLDEMOTE_FLAGS, 0 },
> - { STRING_COMMA_LEN (".amx_int8"), PROCESSOR_UNKNOWN,
> - CPU_AMX_INT8_FLAGS, 0 },
> - { STRING_COMMA_LEN (".amx_bf16"), PROCESSOR_UNKNOWN,
> - CPU_AMX_BF16_FLAGS, 0 },
> - { STRING_COMMA_LEN (".amx_tile"), PROCESSOR_UNKNOWN,
> - CPU_AMX_TILE_FLAGS, 0 },
> - { STRING_COMMA_LEN (".movdiri"), PROCESSOR_UNKNOWN,
> - CPU_MOVDIRI_FLAGS, 0 },
> - { STRING_COMMA_LEN (".movdir64b"), PROCESSOR_UNKNOWN,
> - CPU_MOVDIR64B_FLAGS, 0 },
> - { STRING_COMMA_LEN (".avx512_bf16"), PROCESSOR_UNKNOWN,
> - CPU_AVX512_BF16_FLAGS, 0 },
> - { STRING_COMMA_LEN (".avx512_vp2intersect"), PROCESSOR_UNKNOWN,
> - CPU_AVX512_VP2INTERSECT_FLAGS, 0 },
> - { STRING_COMMA_LEN (".tdx"), PROCESSOR_UNKNOWN,
> - CPU_TDX_FLAGS, 0 },
> - { STRING_COMMA_LEN (".enqcmd"), PROCESSOR_UNKNOWN,
> - CPU_ENQCMD_FLAGS, 0 },
> - { STRING_COMMA_LEN (".serialize"), PROCESSOR_UNKNOWN,
> - CPU_SERIALIZE_FLAGS, 0 },
> - { STRING_COMMA_LEN (".rdpru"), PROCESSOR_UNKNOWN,
> - CPU_RDPRU_FLAGS, 0 },
> - { STRING_COMMA_LEN (".mcommit"), PROCESSOR_UNKNOWN,
> - CPU_MCOMMIT_FLAGS, 0 },
> - { STRING_COMMA_LEN (".sev_es"), PROCESSOR_UNKNOWN,
> - CPU_SEV_ES_FLAGS, 0 },
> - { STRING_COMMA_LEN (".tsxldtrk"), PROCESSOR_UNKNOWN,
> - CPU_TSXLDTRK_FLAGS, 0 },
> - { STRING_COMMA_LEN (".kl"), PROCESSOR_UNKNOWN,
> - CPU_KL_FLAGS, 0 },
> - { STRING_COMMA_LEN (".widekl"), PROCESSOR_UNKNOWN,
> - CPU_WIDEKL_FLAGS, 0 },
> - { STRING_COMMA_LEN (".uintr"), PROCESSOR_UNKNOWN,
> - CPU_UINTR_FLAGS, 0 },
> - { STRING_COMMA_LEN (".hreset"), PROCESSOR_UNKNOWN,
> - CPU_HRESET_FLAGS, 0 },
> - { STRING_COMMA_LEN (".avx512_fp16"), PROCESSOR_UNKNOWN,
> - CPU_AVX512_FP16_FLAGS, 0 },
> + ARCH (generic32, GENERIC32, GENERIC32, false),
> + ARCH (generic64, GENERIC64, GENERIC64, false),
> + ARCH (i8086, UNKNOWN, NONE, false),
> + ARCH (i186, UNKNOWN, I186, false),
> + ARCH (i286, UNKNOWN, I286, false),
> + ARCH (i386, I386, I386, false),
> + ARCH (i486, I486, I486, false),
> + ARCH (i586, PENTIUM, I586, false),
> + ARCH (i686, PENTIUMPRO, I686, false),
> + ARCH (pentium, PENTIUM, I586, false),
> + ARCH (pentiumpro, PENTIUMPRO, PENTIUMPRO, false),
> + ARCH (pentiumii, PENTIUMPRO, P2, false),
> + ARCH (pentiumiii, PENTIUMPRO, P3, false),
> + ARCH (pentium4, PENTIUM4, P4, false),
> + ARCH (prescott, NOCONA, CORE, false),
> + ARCH (nocona, NOCONA, NOCONA, false),
> + ARCH (yonah, CORE, CORE, true),
> + ARCH (core, CORE, CORE, false),
> + ARCH (merom, CORE2, CORE2, true),
> + ARCH (core2, CORE2, CORE2, false),
> + ARCH (corei7, COREI7, COREI7, false),
> + ARCH (iamcu, IAMCU, IAMCU, false),
> + ARCH (k6, K6, K6, false),
> + ARCH (k6_2, K6, K6_2, false),
> + ARCH (athlon, ATHLON, ATHLON, false),
> + ARCH (sledgehammer, K8, K8, true),
> + ARCH (opteron, K8, K8, false),
> + ARCH (k8, K8, K8, false),
> + ARCH (amdfam10, AMDFAM10, AMDFAM10, false),
> + ARCH (bdver1, BD, BDVER1, false),
> + ARCH (bdver2, BD, BDVER2, false),
> + ARCH (bdver3, BD, BDVER3, false),
> + ARCH (bdver4, BD, BDVER4, false),
> + ARCH (znver1, ZNVER, ZNVER1, false),
> + ARCH (znver2, ZNVER, ZNVER2, false),
> + ARCH (znver3, ZNVER, ZNVER3, false),
> + ARCH (btver1, BT, BTVER1, false),
> + ARCH (btver2, BT, BTVER2, false),
> +
> + SUBARCH (8087, 8087, false),
> + SUBARCH (287, 287, false),
> + SUBARCH (387, 387, false),
> + SUBARCH (687, 687, false),
> + SUBARCH (cmov, CMOV, false),
> + SUBARCH (fxsr, FXSR, false),
> + SUBARCH (mmx, MMX, false),
> + SUBARCH (sse, SSE, false),
> + SUBARCH (sse2, SSE2, false),
> + SUBARCH (sse3, SSE3, false),
> + SUBARCH (sse4a, SSE4A, false),
> + SUBARCH (ssse3, SSSE3, false),
> + SUBARCH (sse4.1, SSE4_1, false),
> + SUBARCH (sse4.2, SSE4_2, false),
> + SUBARCH (sse4, SSE4_2, false),
> + SUBARCH (avx, AVX, false),
> + SUBARCH (avx2, AVX2, false),
> + SUBARCH (avx512f, AVX512F, false),
> + SUBARCH (avx512cd, AVX512CD, false),
> + SUBARCH (avx512er, AVX512ER, false),
> + SUBARCH (avx512pf, AVX512PF, false),
> + SUBARCH (avx512dq, AVX512DQ, false),
> + SUBARCH (avx512bw, AVX512BW, false),
> + SUBARCH (avx512vl, AVX512VL, false),
> + SUBARCH (vmx, VMX, false),
> + SUBARCH (vmfunc, VMFUNC, false),
> + SUBARCH (smx, SMX, false),
> + SUBARCH (xsave, XSAVE, false),
> + SUBARCH (xsaveopt, XSAVEOPT, false),
> + SUBARCH (xsavec, XSAVEC, false),
> + SUBARCH (xsaves, XSAVES, false),
> + SUBARCH (aes, AES, false),
> + SUBARCH (pclmul, PCLMUL, false),
> + SUBARCH (clmul, PCLMUL, true),
> + SUBARCH (fsgsbase, FSGSBASE, false),
> + SUBARCH (rdrnd, RDRND, false),
> + SUBARCH (f16c, F16C, false),
> + SUBARCH (bmi2, BMI2, false),
> + SUBARCH (fma, FMA, false),
> + SUBARCH (fma4, FMA4, false),
> + SUBARCH (xop, XOP, false),
> + SUBARCH (lwp, LWP, false),
> + SUBARCH (movbe, MOVBE, false),
> + SUBARCH (cx16, CX16, false),
> + SUBARCH (ept, EPT, false),
> + SUBARCH (lzcnt, LZCNT, false),
> + SUBARCH (popcnt, POPCNT, false),
> + SUBARCH (hle, HLE, false),
> + SUBARCH (rtm, RTM, false),
> + SUBARCH (invpcid, INVPCID, false),
> + SUBARCH (clflush, CLFLUSH, false),
> + SUBARCH (nop, NOP, false),
> + SUBARCH (syscall, SYSCALL, false),
> + SUBARCH (rdtscp, RDTSCP, false),
> + SUBARCH (3dnow, 3DNOW, false),
> + SUBARCH (3dnowa, 3DNOWA, false),
> + SUBARCH (padlock, PADLOCK, false),
> + SUBARCH (pacifica, SVME, true),
> + SUBARCH (svme, SVME, false),
> + SUBARCH (sse4a, SSE4A, false),
> + SUBARCH (abm, ABM, false),
> + SUBARCH (bmi, BMI, false),
> + SUBARCH (tbm, TBM, false),
> + SUBARCH (adx, ADX, false),
> + SUBARCH (rdseed, RDSEED, false),
> + SUBARCH (prfchw, PRFCHW, false),
> + SUBARCH (smap, SMAP, false),
> + SUBARCH (mpx, MPX, false),
> + SUBARCH (sha, SHA, false),
> + SUBARCH (clflushopt, CLFLUSHOPT, false),
> + SUBARCH (prefetchwt1, PREFETCHWT1, false),
> + SUBARCH (se1, SE1, false),
> + SUBARCH (clwb, CLWB, false),
> + SUBARCH (avx512ifma, AVX512IFMA, false),
> + SUBARCH (avx512vbmi, AVX512VBMI, false),
> + SUBARCH (avx512_4fmaps, AVX512_4FMAPS, false),
> + SUBARCH (avx512_4vnniw, AVX512_4VNNIW, false),
> + SUBARCH (avx512_vpopcntdq, AVX512_VPOPCNTDQ, false),
> + SUBARCH (avx512_vbmi2, AVX512_VBMI2, false),
> + SUBARCH (avx512_vnni, AVX512_VNNI, false),
> + SUBARCH (avx512_bitalg, AVX512_BITALG, false),
> + SUBARCH (avx_vnni, AVX_VNNI, false),
> + SUBARCH (clzero, CLZERO, false),
> + SUBARCH (mwaitx, MWAITX, false),
> + SUBARCH (ospke, OSPKE, false),
> + SUBARCH (rdpid, RDPID, false),
> + SUBARCH (ptwrite, PTWRITE, false),
> + SUBARCH (ibt, IBT, false),
> + SUBARCH (shstk, SHSTK, false),
> + SUBARCH (gfni, GFNI, false),
> + SUBARCH (vaes, VAES, false),
> + SUBARCH (vpclmulqdq, VPCLMULQDQ, false),
> + SUBARCH (wbnoinvd, WBNOINVD, false),
> + SUBARCH (pconfig, PCONFIG, false),
> + SUBARCH (waitpkg, WAITPKG, false),
> + SUBARCH (cldemote, CLDEMOTE, false),
> + SUBARCH (amx_int8, AMX_INT8, false),
> + SUBARCH (amx_bf16, AMX_BF16, false),
> + SUBARCH (amx_tile, AMX_TILE, false),
> + SUBARCH (movdiri, MOVDIRI, false),
> + SUBARCH (movdir64b, MOVDIR64B, false),
> + SUBARCH (avx512_bf16, AVX512_BF16, false),
> + SUBARCH (avx512_vp2intersect, AVX512_VP2INTERSECT, false),
> + SUBARCH (tdx, TDX, false),
> + SUBARCH (enqcmd, ENQCMD, false),
> + SUBARCH (serialize, SERIALIZE, false),
> + SUBARCH (rdpru, RDPRU, false),
> + SUBARCH (mcommit, MCOMMIT, false),
> + SUBARCH (sev_es, SEV_ES, false),
> + SUBARCH (tsxldtrk, TSXLDTRK, false),
> + SUBARCH (kl, KL, false),
> + SUBARCH (widekl, WIDEKL, false),
> + SUBARCH (uintr, UINTR, false),
> + SUBARCH (hreset, HRESET, false),
> + SUBARCH (avx512_fp16, AVX512_FP16, false),
> };
>
> +#undef SUBARCH
> +#undef ARCH
> +
> static const noarch_entry cpu_noarch[] =
> {
> { STRING_COMMA_LEN ("no87"), CPU_ANY_X87_FLAGS },
>
OK.
Thanks.
--
H.J.
^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH 5/7] x86: introduce fake processor type to mark sub-arch entries in cpu_arch[]
2022-06-30 12:52 [PATCH] gas/x86: .arch / -march= enhancements Jan Beulich
` (3 preceding siblings ...)
2022-06-30 12:54 ` [PATCH 4/7] x86: macro-ize cpu_arch[] entries Jan Beulich
@ 2022-06-30 12:54 ` Jan Beulich
2022-06-30 23:03 ` H.J. Lu
2022-06-30 12:55 ` [PATCH 6/7] x86: generalize disabling of sub-architectures Jan Beulich
2022-06-30 12:55 ` [PATCH 7/7] x86: introduce a state stack for .arch Jan Beulich
6 siblings, 1 reply; 17+ messages in thread
From: Jan Beulich @ 2022-06-30 12:54 UTC (permalink / raw)
To: Binutils
This is in preparation of dropping the leading . from the strings.
While there also move PROCESSOR_GENERIC{32,64} from the middle of AMD
entries to near the top.
--- a/gas/config/tc-i386.c
+++ b/gas/config/tc-i386.c
@@ -934,7 +934,7 @@ const relax_typeS md_relax_table[] =
#define ARCH(n, t, f, s) \
{ STRING_COMMA_LEN (#n), s, PROCESSOR_ ## t, CPU_ ## f ## _FLAGS }
#define SUBARCH(n, e, s) \
- { STRING_COMMA_LEN ("." #n), s, PROCESSOR_UNKNOWN, CPU_ ## e ## _FLAGS }
+ { STRING_COMMA_LEN ("." #n), s, PROCESSOR_NONE, CPU_ ## e ## _FLAGS }
static const arch_entry cpu_arch[] =
{
@@ -1471,6 +1471,8 @@ i386_generate_nops (fragS *fragP, char *
case PROCESSOR_GENERIC32:
patt = f32_patt;
break;
+ case PROCESSOR_NONE:
+ abort ();
}
}
else
@@ -1516,6 +1518,8 @@ i386_generate_nops (fragS *fragP, char *
case PROCESSOR_GENERIC64:
patt = alt_patt;
break;
+ case PROCESSOR_NONE:
+ abort ();
}
}
--- a/gas/config/tc-i386.h
+++ b/gas/config/tc-i386.h
@@ -228,6 +228,8 @@ extern long i386_generic_table_relax_fra
enum processor_type
{
PROCESSOR_UNKNOWN,
+ PROCESSOR_GENERIC32,
+ PROCESSOR_GENERIC64,
PROCESSOR_I386,
PROCESSOR_I486,
PROCESSOR_PENTIUM,
@@ -241,12 +243,12 @@ enum processor_type
PROCESSOR_K6,
PROCESSOR_ATHLON,
PROCESSOR_K8,
- PROCESSOR_GENERIC32,
- PROCESSOR_GENERIC64,
PROCESSOR_AMDFAM10,
PROCESSOR_BD,
PROCESSOR_ZNVER,
- PROCESSOR_BT
+ PROCESSOR_BT,
+ /* Keep this last. */
+ PROCESSOR_NONE
};
extern enum processor_type cpu_arch_tune;
^ permalink raw reply [flat|nested] 17+ messages in thread* Re: [PATCH 5/7] x86: introduce fake processor type to mark sub-arch entries in cpu_arch[]
2022-06-30 12:54 ` [PATCH 5/7] x86: introduce fake processor type to mark sub-arch entries in cpu_arch[] Jan Beulich
@ 2022-06-30 23:03 ` H.J. Lu
0 siblings, 0 replies; 17+ messages in thread
From: H.J. Lu @ 2022-06-30 23:03 UTC (permalink / raw)
To: Jan Beulich; +Cc: Binutils
On Thu, Jun 30, 2022 at 5:54 AM Jan Beulich <jbeulich@suse.com> wrote:
>
> This is in preparation of dropping the leading . from the strings.
>
> While there also move PROCESSOR_GENERIC{32,64} from the middle of AMD
> entries to near the top.
>
> --- a/gas/config/tc-i386.c
> +++ b/gas/config/tc-i386.c
> @@ -934,7 +934,7 @@ const relax_typeS md_relax_table[] =
> #define ARCH(n, t, f, s) \
> { STRING_COMMA_LEN (#n), s, PROCESSOR_ ## t, CPU_ ## f ## _FLAGS }
> #define SUBARCH(n, e, s) \
> - { STRING_COMMA_LEN ("." #n), s, PROCESSOR_UNKNOWN, CPU_ ## e ## _FLAGS }
> + { STRING_COMMA_LEN ("." #n), s, PROCESSOR_NONE, CPU_ ## e ## _FLAGS }
>
> static const arch_entry cpu_arch[] =
> {
> @@ -1471,6 +1471,8 @@ i386_generate_nops (fragS *fragP, char *
> case PROCESSOR_GENERIC32:
> patt = f32_patt;
> break;
> + case PROCESSOR_NONE:
> + abort ();
> }
> }
> else
> @@ -1516,6 +1518,8 @@ i386_generate_nops (fragS *fragP, char *
> case PROCESSOR_GENERIC64:
> patt = alt_patt;
> break;
> + case PROCESSOR_NONE:
> + abort ();
> }
> }
>
> --- a/gas/config/tc-i386.h
> +++ b/gas/config/tc-i386.h
> @@ -228,6 +228,8 @@ extern long i386_generic_table_relax_fra
> enum processor_type
> {
> PROCESSOR_UNKNOWN,
> + PROCESSOR_GENERIC32,
> + PROCESSOR_GENERIC64,
> PROCESSOR_I386,
> PROCESSOR_I486,
> PROCESSOR_PENTIUM,
> @@ -241,12 +243,12 @@ enum processor_type
> PROCESSOR_K6,
> PROCESSOR_ATHLON,
> PROCESSOR_K8,
> - PROCESSOR_GENERIC32,
> - PROCESSOR_GENERIC64,
> PROCESSOR_AMDFAM10,
> PROCESSOR_BD,
> PROCESSOR_ZNVER,
> - PROCESSOR_BT
> + PROCESSOR_BT,
> + /* Keep this last. */
> + PROCESSOR_NONE
> };
>
> extern enum processor_type cpu_arch_tune;
>
OK.
Thanks.
--
H.J.
^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH 6/7] x86: generalize disabling of sub-architectures
2022-06-30 12:52 [PATCH] gas/x86: .arch / -march= enhancements Jan Beulich
` (4 preceding siblings ...)
2022-06-30 12:54 ` [PATCH 5/7] x86: introduce fake processor type to mark sub-arch entries in cpu_arch[] Jan Beulich
@ 2022-06-30 12:55 ` Jan Beulich
2022-06-30 23:11 ` H.J. Lu
2022-06-30 12:55 ` [PATCH 7/7] x86: introduce a state stack for .arch Jan Beulich
6 siblings, 1 reply; 17+ messages in thread
From: Jan Beulich @ 2022-06-30 12:55 UTC (permalink / raw)
To: Binutils
I never really understood upon what basis ".arch .no*" options were made
available. Let's not have any "criteria" at all, and simply allow
disabling of all of them. Then we also have all data for a sub-arch in
a single place, as we now only need a single table.
---
For now I've re-used all CPU_ANY_*_FLAGS there were, but I think we
might be better off introducing such only once dependent features
appear. Until then the base CPU_*_FLAGS can very well be used, just like
done here in all cases where there was no suitable CPU_ANY_*_FLAGS.
--- a/gas/config/tc-i386.c
+++ b/gas/config/tc-i386.c
@@ -129,19 +129,11 @@ typedef struct
unsigned int len:8; /* arch string length */
bool skip:1; /* show_arch should skip this. */
enum processor_type type; /* arch type */
- i386_cpu_flags flags; /* cpu feature flags */
+ i386_cpu_flags enable; /* cpu feature enable flags */
+ i386_cpu_flags disable; /* cpu feature disable flags */
}
arch_entry;
-/* Used to turn off indicated flags. */
-typedef struct
-{
- const char *name; /* arch name */
- unsigned int len; /* arch string length */
- i386_cpu_flags flags; /* cpu feature flags */
-}
-noarch_entry;
-
static void update_code_flag (int, int);
static void set_code_flag (int);
static void set_16bit_gcc_code_flag (int);
@@ -932,9 +924,11 @@ const relax_typeS md_relax_table[] =
};
#define ARCH(n, t, f, s) \
- { STRING_COMMA_LEN (#n), s, PROCESSOR_ ## t, CPU_ ## f ## _FLAGS }
-#define SUBARCH(n, e, s) \
- { STRING_COMMA_LEN ("." #n), s, PROCESSOR_NONE, CPU_ ## e ## _FLAGS }
+ { STRING_COMMA_LEN (#n), s, PROCESSOR_ ## t, CPU_ ## f ## _FLAGS, \
+ CPU_NONE_FLAGS }
+#define SUBARCH(n, e, d, s) \
+ { STRING_COMMA_LEN (#n), s, PROCESSOR_NONE, CPU_ ## e ## _FLAGS, \
+ CPU_ ## d ## _FLAGS }
static const arch_entry cpu_arch[] =
{
@@ -979,182 +973,128 @@ static const arch_entry cpu_arch[] =
ARCH (btver1, BT, BTVER1, false),
ARCH (btver2, BT, BTVER2, false),
- SUBARCH (8087, 8087, false),
- SUBARCH (287, 287, false),
- SUBARCH (387, 387, false),
- SUBARCH (687, 687, false),
- SUBARCH (cmov, CMOV, false),
- SUBARCH (fxsr, FXSR, false),
- SUBARCH (mmx, MMX, false),
- SUBARCH (sse, SSE, false),
- SUBARCH (sse2, SSE2, false),
- SUBARCH (sse3, SSE3, false),
- SUBARCH (sse4a, SSE4A, false),
- SUBARCH (ssse3, SSSE3, false),
- SUBARCH (sse4.1, SSE4_1, false),
- SUBARCH (sse4.2, SSE4_2, false),
- SUBARCH (sse4, SSE4_2, false),
- SUBARCH (avx, AVX, false),
- SUBARCH (avx2, AVX2, false),
- SUBARCH (avx512f, AVX512F, false),
- SUBARCH (avx512cd, AVX512CD, false),
- SUBARCH (avx512er, AVX512ER, false),
- SUBARCH (avx512pf, AVX512PF, false),
- SUBARCH (avx512dq, AVX512DQ, false),
- SUBARCH (avx512bw, AVX512BW, false),
- SUBARCH (avx512vl, AVX512VL, false),
- SUBARCH (vmx, VMX, false),
- SUBARCH (vmfunc, VMFUNC, false),
- SUBARCH (smx, SMX, false),
- SUBARCH (xsave, XSAVE, false),
- SUBARCH (xsaveopt, XSAVEOPT, false),
- SUBARCH (xsavec, XSAVEC, false),
- SUBARCH (xsaves, XSAVES, false),
- SUBARCH (aes, AES, false),
- SUBARCH (pclmul, PCLMUL, false),
- SUBARCH (clmul, PCLMUL, true),
- SUBARCH (fsgsbase, FSGSBASE, false),
- SUBARCH (rdrnd, RDRND, false),
- SUBARCH (f16c, F16C, false),
- SUBARCH (bmi2, BMI2, false),
- SUBARCH (fma, FMA, false),
- SUBARCH (fma4, FMA4, false),
- SUBARCH (xop, XOP, false),
- SUBARCH (lwp, LWP, false),
- SUBARCH (movbe, MOVBE, false),
- SUBARCH (cx16, CX16, false),
- SUBARCH (ept, EPT, false),
- SUBARCH (lzcnt, LZCNT, false),
- SUBARCH (popcnt, POPCNT, false),
- SUBARCH (hle, HLE, false),
- SUBARCH (rtm, RTM, false),
- SUBARCH (invpcid, INVPCID, false),
- SUBARCH (clflush, CLFLUSH, false),
- SUBARCH (nop, NOP, false),
- SUBARCH (syscall, SYSCALL, false),
- SUBARCH (rdtscp, RDTSCP, false),
- SUBARCH (3dnow, 3DNOW, false),
- SUBARCH (3dnowa, 3DNOWA, false),
- SUBARCH (padlock, PADLOCK, false),
- SUBARCH (pacifica, SVME, true),
- SUBARCH (svme, SVME, false),
- SUBARCH (sse4a, SSE4A, false),
- SUBARCH (abm, ABM, false),
- SUBARCH (bmi, BMI, false),
- SUBARCH (tbm, TBM, false),
- SUBARCH (adx, ADX, false),
- SUBARCH (rdseed, RDSEED, false),
- SUBARCH (prfchw, PRFCHW, false),
- SUBARCH (smap, SMAP, false),
- SUBARCH (mpx, MPX, false),
- SUBARCH (sha, SHA, false),
- SUBARCH (clflushopt, CLFLUSHOPT, false),
- SUBARCH (prefetchwt1, PREFETCHWT1, false),
- SUBARCH (se1, SE1, false),
- SUBARCH (clwb, CLWB, false),
- SUBARCH (avx512ifma, AVX512IFMA, false),
- SUBARCH (avx512vbmi, AVX512VBMI, false),
- SUBARCH (avx512_4fmaps, AVX512_4FMAPS, false),
- SUBARCH (avx512_4vnniw, AVX512_4VNNIW, false),
- SUBARCH (avx512_vpopcntdq, AVX512_VPOPCNTDQ, false),
- SUBARCH (avx512_vbmi2, AVX512_VBMI2, false),
- SUBARCH (avx512_vnni, AVX512_VNNI, false),
- SUBARCH (avx512_bitalg, AVX512_BITALG, false),
- SUBARCH (avx_vnni, AVX_VNNI, false),
- SUBARCH (clzero, CLZERO, false),
- SUBARCH (mwaitx, MWAITX, false),
- SUBARCH (ospke, OSPKE, false),
- SUBARCH (rdpid, RDPID, false),
- SUBARCH (ptwrite, PTWRITE, false),
- SUBARCH (ibt, IBT, false),
- SUBARCH (shstk, SHSTK, false),
- SUBARCH (gfni, GFNI, false),
- SUBARCH (vaes, VAES, false),
- SUBARCH (vpclmulqdq, VPCLMULQDQ, false),
- SUBARCH (wbnoinvd, WBNOINVD, false),
- SUBARCH (pconfig, PCONFIG, false),
- SUBARCH (waitpkg, WAITPKG, false),
- SUBARCH (cldemote, CLDEMOTE, false),
- SUBARCH (amx_int8, AMX_INT8, false),
- SUBARCH (amx_bf16, AMX_BF16, false),
- SUBARCH (amx_tile, AMX_TILE, false),
- SUBARCH (movdiri, MOVDIRI, false),
- SUBARCH (movdir64b, MOVDIR64B, false),
- SUBARCH (avx512_bf16, AVX512_BF16, false),
- SUBARCH (avx512_vp2intersect, AVX512_VP2INTERSECT, false),
- SUBARCH (tdx, TDX, false),
- SUBARCH (enqcmd, ENQCMD, false),
- SUBARCH (serialize, SERIALIZE, false),
- SUBARCH (rdpru, RDPRU, false),
- SUBARCH (mcommit, MCOMMIT, false),
- SUBARCH (sev_es, SEV_ES, false),
- SUBARCH (tsxldtrk, TSXLDTRK, false),
- SUBARCH (kl, KL, false),
- SUBARCH (widekl, WIDEKL, false),
- SUBARCH (uintr, UINTR, false),
- SUBARCH (hreset, HRESET, false),
- SUBARCH (avx512_fp16, AVX512_FP16, false),
+ SUBARCH (8087, 8087, ANY_X87, false),
+ SUBARCH (87, NONE, ANY_X87, false), /* Disable only! */
+ SUBARCH (287, 287, ANY_287, false),
+ SUBARCH (387, 387, ANY_387, false),
+ SUBARCH (687, 687, ANY_687, false),
+ SUBARCH (cmov, CMOV, ANY_CMOV, false),
+ SUBARCH (fxsr, FXSR, ANY_FXSR, false),
+ SUBARCH (mmx, MMX, ANY_MMX, false),
+ SUBARCH (sse, SSE, ANY_SSE, false),
+ SUBARCH (sse2, SSE2, ANY_SSE2, false),
+ SUBARCH (sse3, SSE3, ANY_SSE3, false),
+ SUBARCH (sse4a, SSE4A, ANY_SSE4A, false),
+ SUBARCH (ssse3, SSSE3, ANY_SSSE3, false),
+ SUBARCH (sse4.1, SSE4_1, ANY_SSE4_1, false),
+ SUBARCH (sse4.2, SSE4_2, ANY_SSE4_2, false),
+ SUBARCH (sse4, SSE4_2, ANY_SSE4_1, false),
+ SUBARCH (avx, AVX, ANY_AVX, false),
+ SUBARCH (avx2, AVX2, ANY_AVX2, false),
+ SUBARCH (avx512f, AVX512F, ANY_AVX512F, false),
+ SUBARCH (avx512cd, AVX512CD, ANY_AVX512CD, false),
+ SUBARCH (avx512er, AVX512ER, ANY_AVX512ER, false),
+ SUBARCH (avx512pf, AVX512PF, ANY_AVX512PF, false),
+ SUBARCH (avx512dq, AVX512DQ, ANY_AVX512DQ, false),
+ SUBARCH (avx512bw, AVX512BW, ANY_AVX512BW, false),
+ SUBARCH (avx512vl, AVX512VL, ANY_AVX512VL, false),
+ SUBARCH (vmx, VMX, VMX, false),
+ SUBARCH (vmfunc, VMFUNC, VMFUNC, false),
+ SUBARCH (smx, SMX, SMX, false),
+ SUBARCH (xsave, XSAVE, XSAVE, false),
+ SUBARCH (xsaveopt, XSAVEOPT, XSAVEOPT, false),
+ SUBARCH (xsavec, XSAVEC, XSAVEC, false),
+ SUBARCH (xsaves, XSAVES, XSAVES, false),
+ SUBARCH (aes, AES, AES, false),
+ SUBARCH (pclmul, PCLMUL, PCLMUL, false),
+ SUBARCH (clmul, PCLMUL, PCLMUL, true),
+ SUBARCH (fsgsbase, FSGSBASE, FSGSBASE, false),
+ SUBARCH (rdrnd, RDRND, RDRND, false),
+ SUBARCH (f16c, F16C, F16C, false),
+ SUBARCH (bmi2, BMI2, BMI2, false),
+ SUBARCH (fma, FMA, FMA, false),
+ SUBARCH (fma4, FMA4, FMA4, false),
+ SUBARCH (xop, XOP, XOP, false),
+ SUBARCH (lwp, LWP, LWP, false),
+ SUBARCH (movbe, MOVBE, MOVBE, false),
+ SUBARCH (cx16, CX16, CX16, false),
+ SUBARCH (ept, EPT, EPT, false),
+ SUBARCH (lzcnt, LZCNT, LZCNT, false),
+ SUBARCH (popcnt, POPCNT, POPCNT, false),
+ SUBARCH (hle, HLE, HLE, false),
+ SUBARCH (rtm, RTM, RTM, false),
+ SUBARCH (invpcid, INVPCID, INVPCID, false),
+ SUBARCH (clflush, CLFLUSH, CLFLUSH, false),
+ SUBARCH (nop, NOP, NOP, false),
+ SUBARCH (syscall, SYSCALL, SYSCALL, false),
+ SUBARCH (rdtscp, RDTSCP, RDTSCP, false),
+ SUBARCH (3dnow, 3DNOW, 3DNOW, false),
+ SUBARCH (3dnowa, 3DNOWA, 3DNOWA, false),
+ SUBARCH (padlock, PADLOCK, PADLOCK, false),
+ SUBARCH (pacifica, SVME, SVME, true),
+ SUBARCH (svme, SVME, SVME, false),
+ SUBARCH (sse4a, SSE4A, SSE4A, false),
+ SUBARCH (abm, ABM, ABM, false),
+ SUBARCH (bmi, BMI, BMI, false),
+ SUBARCH (tbm, TBM, TBM, false),
+ SUBARCH (adx, ADX, ADX, false),
+ SUBARCH (rdseed, RDSEED, RDSEED, false),
+ SUBARCH (prfchw, PRFCHW, PRFCHW, false),
+ SUBARCH (smap, SMAP, SMAP, false),
+ SUBARCH (mpx, MPX, MPX, false),
+ SUBARCH (sha, SHA, SHA, false),
+ SUBARCH (clflushopt, CLFLUSHOPT, CLFLUSHOPT, false),
+ SUBARCH (prefetchwt1, PREFETCHWT1, PREFETCHWT1, false),
+ SUBARCH (se1, SE1, SE1, false),
+ SUBARCH (clwb, CLWB, CLWB, false),
+ SUBARCH (avx512ifma, AVX512IFMA, ANY_AVX512IFMA, false),
+ SUBARCH (avx512vbmi, AVX512VBMI, ANY_AVX512VBMI, false),
+ SUBARCH (avx512_4fmaps, AVX512_4FMAPS, ANY_AVX512_4FMAPS, false),
+ SUBARCH (avx512_4vnniw, AVX512_4VNNIW, ANY_AVX512_4VNNIW, false),
+ SUBARCH (avx512_vpopcntdq, AVX512_VPOPCNTDQ, ANY_AVX512_VPOPCNTDQ, false),
+ SUBARCH (avx512_vbmi2, AVX512_VBMI2, ANY_AVX512_VBMI2, false),
+ SUBARCH (avx512_vnni, AVX512_VNNI, ANY_AVX512_VNNI, false),
+ SUBARCH (avx512_bitalg, AVX512_BITALG, ANY_AVX512_BITALG, false),
+ SUBARCH (avx_vnni, AVX_VNNI, ANY_AVX_VNNI, false),
+ SUBARCH (clzero, CLZERO, CLZERO, false),
+ SUBARCH (mwaitx, MWAITX, MWAITX, false),
+ SUBARCH (ospke, OSPKE, OSPKE, false),
+ SUBARCH (rdpid, RDPID, RDPID, false),
+ SUBARCH (ptwrite, PTWRITE, PTWRITE, false),
+ SUBARCH (ibt, IBT, ANY_IBT, false),
+ SUBARCH (shstk, SHSTK, ANY_SHSTK, false),
+ SUBARCH (gfni, GFNI, GFNI, false),
+ SUBARCH (vaes, VAES, VAES, false),
+ SUBARCH (vpclmulqdq, VPCLMULQDQ, VPCLMULQDQ, false),
+ SUBARCH (wbnoinvd, WBNOINVD, WBNOINVD, false),
+ SUBARCH (pconfig, PCONFIG, PCONFIG, false),
+ SUBARCH (waitpkg, WAITPKG, WAITPKG, false),
+ SUBARCH (cldemote, CLDEMOTE, CLDEMOTE, false),
+ SUBARCH (amx_int8, AMX_INT8, ANY_AMX_INT8, false),
+ SUBARCH (amx_bf16, AMX_BF16, ANY_AMX_BF16, false),
+ SUBARCH (amx_tile, AMX_TILE, ANY_AMX_TILE, false),
+ SUBARCH (movdiri, MOVDIRI, ANY_MOVDIRI, false),
+ SUBARCH (movdir64b, MOVDIR64B, ANY_MOVDIR64B, false),
+ SUBARCH (avx512_bf16, AVX512_BF16, ANY_AVX512_BF16, false),
+ SUBARCH (avx512_vp2intersect, AVX512_VP2INTERSECT,
+ ANY_AVX512_VP2INTERSECT, false),
+ SUBARCH (tdx, TDX, ANY_TDX, false),
+ SUBARCH (enqcmd, ENQCMD, ANY_ENQCMD, false),
+ SUBARCH (serialize, SERIALIZE, ANY_SERIALIZE, false),
+ SUBARCH (rdpru, RDPRU, RDPRU, false),
+ SUBARCH (mcommit, MCOMMIT, MCOMMIT, false),
+ SUBARCH (sev_es, SEV_ES, SEV_ES, false),
+ SUBARCH (tsxldtrk, TSXLDTRK, ANY_TSXLDTRK, false),
+ SUBARCH (kl, KL, ANY_KL, false),
+ SUBARCH (widekl, WIDEKL, ANY_WIDEKL, false),
+ SUBARCH (uintr, UINTR, ANY_UINTR, false),
+ SUBARCH (hreset, HRESET, ANY_HRESET, false),
+ SUBARCH (avx512_fp16, AVX512_FP16, ANY_AVX512_FP16, false),
};
#undef SUBARCH
#undef ARCH
-static const noarch_entry cpu_noarch[] =
-{
- { STRING_COMMA_LEN ("no87"), CPU_ANY_X87_FLAGS },
- { STRING_COMMA_LEN ("no287"), CPU_ANY_287_FLAGS },
- { STRING_COMMA_LEN ("no387"), CPU_ANY_387_FLAGS },
- { STRING_COMMA_LEN ("no687"), CPU_ANY_687_FLAGS },
- { STRING_COMMA_LEN ("nocmov"), CPU_ANY_CMOV_FLAGS },
- { STRING_COMMA_LEN ("nofxsr"), CPU_ANY_FXSR_FLAGS },
- { STRING_COMMA_LEN ("nommx"), CPU_ANY_MMX_FLAGS },
- { STRING_COMMA_LEN ("nosse"), CPU_ANY_SSE_FLAGS },
- { STRING_COMMA_LEN ("nosse2"), CPU_ANY_SSE2_FLAGS },
- { STRING_COMMA_LEN ("nosse3"), CPU_ANY_SSE3_FLAGS },
- { STRING_COMMA_LEN ("nosse4a"), CPU_ANY_SSE4A_FLAGS },
- { STRING_COMMA_LEN ("nossse3"), CPU_ANY_SSSE3_FLAGS },
- { STRING_COMMA_LEN ("nosse4.1"), CPU_ANY_SSE4_1_FLAGS },
- { STRING_COMMA_LEN ("nosse4.2"), CPU_ANY_SSE4_2_FLAGS },
- { STRING_COMMA_LEN ("nosse4"), CPU_ANY_SSE4_1_FLAGS },
- { STRING_COMMA_LEN ("noavx"), CPU_ANY_AVX_FLAGS },
- { STRING_COMMA_LEN ("noavx2"), CPU_ANY_AVX2_FLAGS },
- { STRING_COMMA_LEN ("noavx512f"), CPU_ANY_AVX512F_FLAGS },
- { STRING_COMMA_LEN ("noavx512cd"), CPU_ANY_AVX512CD_FLAGS },
- { STRING_COMMA_LEN ("noavx512er"), CPU_ANY_AVX512ER_FLAGS },
- { STRING_COMMA_LEN ("noavx512pf"), CPU_ANY_AVX512PF_FLAGS },
- { STRING_COMMA_LEN ("noavx512dq"), CPU_ANY_AVX512DQ_FLAGS },
- { STRING_COMMA_LEN ("noavx512bw"), CPU_ANY_AVX512BW_FLAGS },
- { STRING_COMMA_LEN ("noavx512vl"), CPU_ANY_AVX512VL_FLAGS },
- { STRING_COMMA_LEN ("noavx512ifma"), CPU_ANY_AVX512IFMA_FLAGS },
- { STRING_COMMA_LEN ("noavx512vbmi"), CPU_ANY_AVX512VBMI_FLAGS },
- { STRING_COMMA_LEN ("noavx512_4fmaps"), CPU_ANY_AVX512_4FMAPS_FLAGS },
- { STRING_COMMA_LEN ("noavx512_4vnniw"), CPU_ANY_AVX512_4VNNIW_FLAGS },
- { STRING_COMMA_LEN ("noavx512_vpopcntdq"), CPU_ANY_AVX512_VPOPCNTDQ_FLAGS },
- { STRING_COMMA_LEN ("noavx512_vbmi2"), CPU_ANY_AVX512_VBMI2_FLAGS },
- { STRING_COMMA_LEN ("noavx512_vnni"), CPU_ANY_AVX512_VNNI_FLAGS },
- { STRING_COMMA_LEN ("noavx512_bitalg"), CPU_ANY_AVX512_BITALG_FLAGS },
- { STRING_COMMA_LEN ("noavx_vnni"), CPU_ANY_AVX_VNNI_FLAGS },
- { STRING_COMMA_LEN ("noibt"), CPU_ANY_IBT_FLAGS },
- { STRING_COMMA_LEN ("noshstk"), CPU_ANY_SHSTK_FLAGS },
- { STRING_COMMA_LEN ("noamx_int8"), CPU_ANY_AMX_INT8_FLAGS },
- { STRING_COMMA_LEN ("noamx_bf16"), CPU_ANY_AMX_BF16_FLAGS },
- { STRING_COMMA_LEN ("noamx_tile"), CPU_ANY_AMX_TILE_FLAGS },
- { STRING_COMMA_LEN ("nomovdiri"), CPU_ANY_MOVDIRI_FLAGS },
- { STRING_COMMA_LEN ("nomovdir64b"), CPU_ANY_MOVDIR64B_FLAGS },
- { STRING_COMMA_LEN ("noavx512_bf16"), CPU_ANY_AVX512_BF16_FLAGS },
- { STRING_COMMA_LEN ("noavx512_vp2intersect"),
- CPU_ANY_AVX512_VP2INTERSECT_FLAGS },
- { STRING_COMMA_LEN ("notdx"), CPU_ANY_TDX_FLAGS },
- { STRING_COMMA_LEN ("noenqcmd"), CPU_ANY_ENQCMD_FLAGS },
- { STRING_COMMA_LEN ("noserialize"), CPU_ANY_SERIALIZE_FLAGS },
- { STRING_COMMA_LEN ("notsxldtrk"), CPU_ANY_TSXLDTRK_FLAGS },
- { STRING_COMMA_LEN ("nokl"), CPU_ANY_KL_FLAGS },
- { STRING_COMMA_LEN ("nowidekl"), CPU_ANY_WIDEKL_FLAGS },
- { STRING_COMMA_LEN ("nouintr"), CPU_ANY_UINTR_FLAGS },
- { STRING_COMMA_LEN ("nohreset"), CPU_ANY_HRESET_FLAGS },
- { STRING_COMMA_LEN ("noavx512_fp16"), CPU_ANY_AVX512_FP16_FLAGS },
-};
-
#ifdef I386COFF
/* Like s_lcomm_internal in gas/read.c but the alignment string
is allowed to be optional. */
@@ -2715,9 +2655,9 @@ extend_cpu_sub_arch_name (const char *na
{
if (cpu_sub_arch_name)
cpu_sub_arch_name = reconcat (cpu_sub_arch_name, cpu_sub_arch_name,
- name, (const char *) NULL);
+ ".", name, (const char *) NULL);
else
- cpu_sub_arch_name = xstrdup (name);
+ cpu_sub_arch_name = concat (".", name, (const char *) NULL);
}
static void
@@ -2756,7 +2696,7 @@ set_cpu_arch (int dummy ATTRIBUTE_UNUSED
cpu_arch_flags.bitfield.cpuno64 = 1;
}
cpu_arch_isa = PROCESSOR_UNKNOWN;
- cpu_arch_isa_flags = cpu_arch[flag_code == CODE_64BIT].flags;
+ cpu_arch_isa_flags = cpu_arch[flag_code == CODE_64BIT].enable;
if (!cpu_arch_tune_set)
{
cpu_arch_tune = cpu_arch_isa;
@@ -2769,16 +2709,17 @@ set_cpu_arch (int dummy ATTRIBUTE_UNUSED
for (; j < ARRAY_SIZE (cpu_arch); j++)
{
- if (strcmp (string, cpu_arch[j].name) == 0)
+ if (strcmp (string + (*string == '.'), cpu_arch[j].name) == 0
+ && (*string == '.') == (cpu_arch[j].type == PROCESSOR_NONE))
{
if (*string != '.')
{
- check_cpu_arch_compatible (string, cpu_arch[j].flags);
+ check_cpu_arch_compatible (string, cpu_arch[j].enable);
cpu_arch_name = cpu_arch[j].name;
xfree (cpu_sub_arch_name);
cpu_sub_arch_name = NULL;
- cpu_arch_flags = cpu_arch[j].flags;
+ cpu_arch_flags = cpu_arch[j].enable;
if (flag_code == CODE_64BIT)
{
cpu_arch_flags.bitfield.cpu64 = 1;
@@ -2790,7 +2731,7 @@ set_cpu_arch (int dummy ATTRIBUTE_UNUSED
cpu_arch_flags.bitfield.cpuno64 = 1;
}
cpu_arch_isa = cpu_arch[j].type;
- cpu_arch_isa_flags = cpu_arch[j].flags;
+ cpu_arch_isa_flags = cpu_arch[j].enable;
if (!cpu_arch_tune_set)
{
cpu_arch_tune = cpu_arch_isa;
@@ -2799,36 +2740,40 @@ set_cpu_arch (int dummy ATTRIBUTE_UNUSED
break;
}
+ if (cpu_flags_all_zero (&cpu_arch[j].enable))
+ continue;
+
flags = cpu_flags_or (cpu_arch_flags,
- cpu_arch[j].flags);
+ cpu_arch[j].enable);
if (!cpu_flags_equal (&flags, &cpu_arch_flags))
{
- extend_cpu_sub_arch_name (string);
+ extend_cpu_sub_arch_name (string + 1);
cpu_arch_flags = flags;
cpu_arch_isa_flags = flags;
}
else
cpu_arch_isa_flags
= cpu_flags_or (cpu_arch_isa_flags,
- cpu_arch[j].flags);
+ cpu_arch[j].enable);
(void) restore_line_pointer (e);
demand_empty_rest_of_line ();
return;
}
}
- if (*string == '.' && j >= ARRAY_SIZE (cpu_arch))
+ if (startswith (string, ".no") && j >= ARRAY_SIZE (cpu_arch))
{
/* Disable an ISA extension. */
- for (j = 0; j < ARRAY_SIZE (cpu_noarch); j++)
- if (strcmp (string + 1, cpu_noarch [j].name) == 0)
+ for (j = 0; j < ARRAY_SIZE (cpu_arch); j++)
+ if (cpu_arch[j].type == PROCESSOR_NONE
+ && strcmp (string + 3, cpu_arch[j].name) == 0)
{
flags = cpu_flags_and_not (cpu_arch_flags,
- cpu_noarch[j].flags);
+ cpu_arch[j].disable);
if (!cpu_flags_equal (&flags, &cpu_arch_flags))
{
- extend_cpu_sub_arch_name (string);
+ extend_cpu_sub_arch_name (string + 1);
cpu_arch_flags = flags;
cpu_arch_isa_flags = flags;
}
@@ -2836,8 +2781,6 @@ set_cpu_arch (int dummy ATTRIBUTE_UNUSED
demand_empty_rest_of_line ();
return;
}
-
- j = ARRAY_SIZE (cpu_arch);
}
if (j == ARRAY_SIZE (cpu_arch))
@@ -13288,18 +13231,19 @@ md_parse_option (int c, const char *arg)
*next++ = '\0';
for (j = 0; j < ARRAY_SIZE (cpu_arch); j++)
{
- if (arch == saved && strcmp (arch, cpu_arch [j].name) == 0)
+ if (arch == saved && cpu_arch[j].type != PROCESSOR_NONE
+ && strcmp (arch, cpu_arch[j].name) == 0)
{
/* Processor. */
- if (! cpu_arch[j].flags.bitfield.cpui386)
+ if (! cpu_arch[j].enable.bitfield.cpui386)
continue;
cpu_arch_name = cpu_arch[j].name;
xfree (cpu_sub_arch_name);
cpu_sub_arch_name = NULL;
- cpu_arch_flags = cpu_arch[j].flags;
+ cpu_arch_flags = cpu_arch[j].enable;
cpu_arch_isa = cpu_arch[j].type;
- cpu_arch_isa_flags = cpu_arch[j].flags;
+ cpu_arch_isa_flags = cpu_arch[j].enable;
if (!cpu_arch_tune_set)
{
cpu_arch_tune = cpu_arch_isa;
@@ -13307,39 +13251,41 @@ md_parse_option (int c, const char *arg)
}
break;
}
- else if (*cpu_arch [j].name == '.'
- && strcmp (arch, cpu_arch [j].name + 1) == 0)
+ else if (cpu_arch[j].type == PROCESSOR_NONE
+ && strcmp (arch, cpu_arch[j].name) == 0
+ && !cpu_flags_all_zero (&cpu_arch[j].enable))
{
/* ISA extension. */
i386_cpu_flags flags;
flags = cpu_flags_or (cpu_arch_flags,
- cpu_arch[j].flags);
+ cpu_arch[j].enable);
if (!cpu_flags_equal (&flags, &cpu_arch_flags))
{
- extend_cpu_sub_arch_name (cpu_arch[j].name);
+ extend_cpu_sub_arch_name (arch);
cpu_arch_flags = flags;
cpu_arch_isa_flags = flags;
}
else
cpu_arch_isa_flags
= cpu_flags_or (cpu_arch_isa_flags,
- cpu_arch[j].flags);
+ cpu_arch[j].enable);
break;
}
}
- if (j >= ARRAY_SIZE (cpu_arch))
+ if (j >= ARRAY_SIZE (cpu_arch) && startswith (arch, "no"))
{
/* Disable an ISA extension. */
- for (j = 0; j < ARRAY_SIZE (cpu_noarch); j++)
- if (strcmp (arch, cpu_noarch [j].name) == 0)
+ for (j = 0; j < ARRAY_SIZE (cpu_arch); j++)
+ if (cpu_arch[j].type == PROCESSOR_NONE
+ && strcmp (arch + 2, cpu_arch[j].name) == 0)
{
i386_cpu_flags flags;
flags = cpu_flags_and_not (cpu_arch_flags,
- cpu_noarch[j].flags);
+ cpu_arch[j].disable);
if (!cpu_flags_equal (&flags, &cpu_arch_flags))
{
extend_cpu_sub_arch_name (arch);
@@ -13348,9 +13294,6 @@ md_parse_option (int c, const char *arg)
}
break;
}
-
- if (j >= ARRAY_SIZE (cpu_noarch))
- j = ARRAY_SIZE (cpu_arch);
}
if (j >= ARRAY_SIZE (cpu_arch))
@@ -13367,11 +13310,12 @@ md_parse_option (int c, const char *arg)
as_fatal (_("invalid -mtune= option: `%s'"), arg);
for (j = 0; j < ARRAY_SIZE (cpu_arch); j++)
{
- if (strcmp (arg, cpu_arch [j].name) == 0)
+ if (cpu_arch[j].type != PROCESSOR_NONE
+ && strcmp (arg, cpu_arch[j].name) == 0)
{
cpu_arch_tune_set = 1;
cpu_arch_tune = cpu_arch [j].type;
- cpu_arch_tune_flags = cpu_arch[j].flags;
+ cpu_arch_tune_flags = cpu_arch[j].enable;
break;
}
}
@@ -13751,15 +13695,10 @@ show_arch (FILE *stream, int ext, int ch
name = cpu_arch [j].name;
len = cpu_arch [j].len;
- if (*name == '.')
+ if (cpu_arch[j].type == PROCESSOR_NONE)
{
/* It is an extension. Skip if we aren't asked to show it. */
- if (ext)
- {
- name++;
- len--;
- }
- else
+ if (!ext || cpu_flags_all_zero (&cpu_arch[j].enable))
continue;
}
else if (ext)
@@ -13767,7 +13706,7 @@ show_arch (FILE *stream, int ext, int ch
/* It is an processor. Skip if we show only extension. */
continue;
}
- else if (check && ! cpu_arch[j].flags.bitfield.cpui386)
+ else if (check && ! cpu_arch[j].enable.bitfield.cpui386)
{
/* It is an impossible processor - skip. */
continue;
@@ -13778,12 +13717,17 @@ show_arch (FILE *stream, int ext, int ch
/* Display disabled extensions. */
if (ext)
- for (j = 0; j < ARRAY_SIZE (cpu_noarch); j++)
+ for (j = 0; j < ARRAY_SIZE (cpu_arch); j++)
{
- name = cpu_noarch [j].name;
- len = cpu_noarch [j].len;
- p = output_message (stream, p, message, start, &left, name,
- len);
+ char *str;
+
+ if (cpu_arch[j].type != PROCESSOR_NONE
+ || !cpu_flags_all_zero (&cpu_arch[j].enable))
+ continue;
+ str = xasprintf ("no%s", cpu_arch[j].name);
+ p = output_message (stream, p, message, start, &left, str,
+ strlen (str));
+ xfree (str);
}
*p = '\0';
@@ -13827,7 +13771,7 @@ md_show_usage (FILE *stream)
generate code for CPU and EXTENSION, CPU is one of:\n"));
show_arch (stream, 0, 1);
fprintf (stream, _("\
- EXTENSION is combination of:\n"));
+ EXTENSION is combination of (possibly \"no\"-prefixed):\n"));
show_arch (stream, 1, 0);
fprintf (stream, _("\
-mtune=CPU optimize for CPU, CPU is one of:\n"));
@@ -13985,9 +13929,9 @@ i386_target_format (void)
as_fatal (_("unknown architecture"));
if (cpu_flags_all_zero (&cpu_arch_isa_flags))
- cpu_arch_isa_flags = cpu_arch[flag_code == CODE_64BIT].flags;
+ cpu_arch_isa_flags = cpu_arch[flag_code == CODE_64BIT].enable;
if (cpu_flags_all_zero (&cpu_arch_tune_flags))
- cpu_arch_tune_flags = cpu_arch[flag_code == CODE_64BIT].flags;
+ cpu_arch_tune_flags = cpu_arch[flag_code == CODE_64BIT].enable;
switch (OUTPUT_FLAVOR)
{
^ permalink raw reply [flat|nested] 17+ messages in thread* Re: [PATCH 6/7] x86: generalize disabling of sub-architectures
2022-06-30 12:55 ` [PATCH 6/7] x86: generalize disabling of sub-architectures Jan Beulich
@ 2022-06-30 23:11 ` H.J. Lu
0 siblings, 0 replies; 17+ messages in thread
From: H.J. Lu @ 2022-06-30 23:11 UTC (permalink / raw)
To: Jan Beulich; +Cc: Binutils
On Thu, Jun 30, 2022 at 5:55 AM Jan Beulich <jbeulich@suse.com> wrote:
>
> I never really understood upon what basis ".arch .no*" options were made
> available. Let's not have any "criteria" at all, and simply allow
> disabling of all of them. Then we also have all data for a sub-arch in
> a single place, as we now only need a single table.
> ---
> For now I've re-used all CPU_ANY_*_FLAGS there were, but I think we
> might be better off introducing such only once dependent features
> appear. Until then the base CPU_*_FLAGS can very well be used, just like
> done here in all cases where there was no suitable CPU_ANY_*_FLAGS.
>
> --- a/gas/config/tc-i386.c
> +++ b/gas/config/tc-i386.c
> @@ -129,19 +129,11 @@ typedef struct
> unsigned int len:8; /* arch string length */
> bool skip:1; /* show_arch should skip this. */
> enum processor_type type; /* arch type */
> - i386_cpu_flags flags; /* cpu feature flags */
> + i386_cpu_flags enable; /* cpu feature enable flags */
> + i386_cpu_flags disable; /* cpu feature disable flags */
> }
> arch_entry;
>
> -/* Used to turn off indicated flags. */
> -typedef struct
> -{
> - const char *name; /* arch name */
> - unsigned int len; /* arch string length */
> - i386_cpu_flags flags; /* cpu feature flags */
> -}
> -noarch_entry;
> -
> static void update_code_flag (int, int);
> static void set_code_flag (int);
> static void set_16bit_gcc_code_flag (int);
> @@ -932,9 +924,11 @@ const relax_typeS md_relax_table[] =
> };
>
> #define ARCH(n, t, f, s) \
> - { STRING_COMMA_LEN (#n), s, PROCESSOR_ ## t, CPU_ ## f ## _FLAGS }
> -#define SUBARCH(n, e, s) \
> - { STRING_COMMA_LEN ("." #n), s, PROCESSOR_NONE, CPU_ ## e ## _FLAGS }
> + { STRING_COMMA_LEN (#n), s, PROCESSOR_ ## t, CPU_ ## f ## _FLAGS, \
> + CPU_NONE_FLAGS }
> +#define SUBARCH(n, e, d, s) \
> + { STRING_COMMA_LEN (#n), s, PROCESSOR_NONE, CPU_ ## e ## _FLAGS, \
> + CPU_ ## d ## _FLAGS }
>
> static const arch_entry cpu_arch[] =
> {
> @@ -979,182 +973,128 @@ static const arch_entry cpu_arch[] =
> ARCH (btver1, BT, BTVER1, false),
> ARCH (btver2, BT, BTVER2, false),
>
> - SUBARCH (8087, 8087, false),
> - SUBARCH (287, 287, false),
> - SUBARCH (387, 387, false),
> - SUBARCH (687, 687, false),
> - SUBARCH (cmov, CMOV, false),
> - SUBARCH (fxsr, FXSR, false),
> - SUBARCH (mmx, MMX, false),
> - SUBARCH (sse, SSE, false),
> - SUBARCH (sse2, SSE2, false),
> - SUBARCH (sse3, SSE3, false),
> - SUBARCH (sse4a, SSE4A, false),
> - SUBARCH (ssse3, SSSE3, false),
> - SUBARCH (sse4.1, SSE4_1, false),
> - SUBARCH (sse4.2, SSE4_2, false),
> - SUBARCH (sse4, SSE4_2, false),
> - SUBARCH (avx, AVX, false),
> - SUBARCH (avx2, AVX2, false),
> - SUBARCH (avx512f, AVX512F, false),
> - SUBARCH (avx512cd, AVX512CD, false),
> - SUBARCH (avx512er, AVX512ER, false),
> - SUBARCH (avx512pf, AVX512PF, false),
> - SUBARCH (avx512dq, AVX512DQ, false),
> - SUBARCH (avx512bw, AVX512BW, false),
> - SUBARCH (avx512vl, AVX512VL, false),
> - SUBARCH (vmx, VMX, false),
> - SUBARCH (vmfunc, VMFUNC, false),
> - SUBARCH (smx, SMX, false),
> - SUBARCH (xsave, XSAVE, false),
> - SUBARCH (xsaveopt, XSAVEOPT, false),
> - SUBARCH (xsavec, XSAVEC, false),
> - SUBARCH (xsaves, XSAVES, false),
> - SUBARCH (aes, AES, false),
> - SUBARCH (pclmul, PCLMUL, false),
> - SUBARCH (clmul, PCLMUL, true),
> - SUBARCH (fsgsbase, FSGSBASE, false),
> - SUBARCH (rdrnd, RDRND, false),
> - SUBARCH (f16c, F16C, false),
> - SUBARCH (bmi2, BMI2, false),
> - SUBARCH (fma, FMA, false),
> - SUBARCH (fma4, FMA4, false),
> - SUBARCH (xop, XOP, false),
> - SUBARCH (lwp, LWP, false),
> - SUBARCH (movbe, MOVBE, false),
> - SUBARCH (cx16, CX16, false),
> - SUBARCH (ept, EPT, false),
> - SUBARCH (lzcnt, LZCNT, false),
> - SUBARCH (popcnt, POPCNT, false),
> - SUBARCH (hle, HLE, false),
> - SUBARCH (rtm, RTM, false),
> - SUBARCH (invpcid, INVPCID, false),
> - SUBARCH (clflush, CLFLUSH, false),
> - SUBARCH (nop, NOP, false),
> - SUBARCH (syscall, SYSCALL, false),
> - SUBARCH (rdtscp, RDTSCP, false),
> - SUBARCH (3dnow, 3DNOW, false),
> - SUBARCH (3dnowa, 3DNOWA, false),
> - SUBARCH (padlock, PADLOCK, false),
> - SUBARCH (pacifica, SVME, true),
> - SUBARCH (svme, SVME, false),
> - SUBARCH (sse4a, SSE4A, false),
> - SUBARCH (abm, ABM, false),
> - SUBARCH (bmi, BMI, false),
> - SUBARCH (tbm, TBM, false),
> - SUBARCH (adx, ADX, false),
> - SUBARCH (rdseed, RDSEED, false),
> - SUBARCH (prfchw, PRFCHW, false),
> - SUBARCH (smap, SMAP, false),
> - SUBARCH (mpx, MPX, false),
> - SUBARCH (sha, SHA, false),
> - SUBARCH (clflushopt, CLFLUSHOPT, false),
> - SUBARCH (prefetchwt1, PREFETCHWT1, false),
> - SUBARCH (se1, SE1, false),
> - SUBARCH (clwb, CLWB, false),
> - SUBARCH (avx512ifma, AVX512IFMA, false),
> - SUBARCH (avx512vbmi, AVX512VBMI, false),
> - SUBARCH (avx512_4fmaps, AVX512_4FMAPS, false),
> - SUBARCH (avx512_4vnniw, AVX512_4VNNIW, false),
> - SUBARCH (avx512_vpopcntdq, AVX512_VPOPCNTDQ, false),
> - SUBARCH (avx512_vbmi2, AVX512_VBMI2, false),
> - SUBARCH (avx512_vnni, AVX512_VNNI, false),
> - SUBARCH (avx512_bitalg, AVX512_BITALG, false),
> - SUBARCH (avx_vnni, AVX_VNNI, false),
> - SUBARCH (clzero, CLZERO, false),
> - SUBARCH (mwaitx, MWAITX, false),
> - SUBARCH (ospke, OSPKE, false),
> - SUBARCH (rdpid, RDPID, false),
> - SUBARCH (ptwrite, PTWRITE, false),
> - SUBARCH (ibt, IBT, false),
> - SUBARCH (shstk, SHSTK, false),
> - SUBARCH (gfni, GFNI, false),
> - SUBARCH (vaes, VAES, false),
> - SUBARCH (vpclmulqdq, VPCLMULQDQ, false),
> - SUBARCH (wbnoinvd, WBNOINVD, false),
> - SUBARCH (pconfig, PCONFIG, false),
> - SUBARCH (waitpkg, WAITPKG, false),
> - SUBARCH (cldemote, CLDEMOTE, false),
> - SUBARCH (amx_int8, AMX_INT8, false),
> - SUBARCH (amx_bf16, AMX_BF16, false),
> - SUBARCH (amx_tile, AMX_TILE, false),
> - SUBARCH (movdiri, MOVDIRI, false),
> - SUBARCH (movdir64b, MOVDIR64B, false),
> - SUBARCH (avx512_bf16, AVX512_BF16, false),
> - SUBARCH (avx512_vp2intersect, AVX512_VP2INTERSECT, false),
> - SUBARCH (tdx, TDX, false),
> - SUBARCH (enqcmd, ENQCMD, false),
> - SUBARCH (serialize, SERIALIZE, false),
> - SUBARCH (rdpru, RDPRU, false),
> - SUBARCH (mcommit, MCOMMIT, false),
> - SUBARCH (sev_es, SEV_ES, false),
> - SUBARCH (tsxldtrk, TSXLDTRK, false),
> - SUBARCH (kl, KL, false),
> - SUBARCH (widekl, WIDEKL, false),
> - SUBARCH (uintr, UINTR, false),
> - SUBARCH (hreset, HRESET, false),
> - SUBARCH (avx512_fp16, AVX512_FP16, false),
> + SUBARCH (8087, 8087, ANY_X87, false),
> + SUBARCH (87, NONE, ANY_X87, false), /* Disable only! */
> + SUBARCH (287, 287, ANY_287, false),
> + SUBARCH (387, 387, ANY_387, false),
> + SUBARCH (687, 687, ANY_687, false),
> + SUBARCH (cmov, CMOV, ANY_CMOV, false),
> + SUBARCH (fxsr, FXSR, ANY_FXSR, false),
> + SUBARCH (mmx, MMX, ANY_MMX, false),
> + SUBARCH (sse, SSE, ANY_SSE, false),
> + SUBARCH (sse2, SSE2, ANY_SSE2, false),
> + SUBARCH (sse3, SSE3, ANY_SSE3, false),
> + SUBARCH (sse4a, SSE4A, ANY_SSE4A, false),
> + SUBARCH (ssse3, SSSE3, ANY_SSSE3, false),
> + SUBARCH (sse4.1, SSE4_1, ANY_SSE4_1, false),
> + SUBARCH (sse4.2, SSE4_2, ANY_SSE4_2, false),
> + SUBARCH (sse4, SSE4_2, ANY_SSE4_1, false),
> + SUBARCH (avx, AVX, ANY_AVX, false),
> + SUBARCH (avx2, AVX2, ANY_AVX2, false),
> + SUBARCH (avx512f, AVX512F, ANY_AVX512F, false),
> + SUBARCH (avx512cd, AVX512CD, ANY_AVX512CD, false),
> + SUBARCH (avx512er, AVX512ER, ANY_AVX512ER, false),
> + SUBARCH (avx512pf, AVX512PF, ANY_AVX512PF, false),
> + SUBARCH (avx512dq, AVX512DQ, ANY_AVX512DQ, false),
> + SUBARCH (avx512bw, AVX512BW, ANY_AVX512BW, false),
> + SUBARCH (avx512vl, AVX512VL, ANY_AVX512VL, false),
> + SUBARCH (vmx, VMX, VMX, false),
> + SUBARCH (vmfunc, VMFUNC, VMFUNC, false),
> + SUBARCH (smx, SMX, SMX, false),
> + SUBARCH (xsave, XSAVE, XSAVE, false),
> + SUBARCH (xsaveopt, XSAVEOPT, XSAVEOPT, false),
> + SUBARCH (xsavec, XSAVEC, XSAVEC, false),
> + SUBARCH (xsaves, XSAVES, XSAVES, false),
> + SUBARCH (aes, AES, AES, false),
> + SUBARCH (pclmul, PCLMUL, PCLMUL, false),
> + SUBARCH (clmul, PCLMUL, PCLMUL, true),
> + SUBARCH (fsgsbase, FSGSBASE, FSGSBASE, false),
> + SUBARCH (rdrnd, RDRND, RDRND, false),
> + SUBARCH (f16c, F16C, F16C, false),
> + SUBARCH (bmi2, BMI2, BMI2, false),
> + SUBARCH (fma, FMA, FMA, false),
> + SUBARCH (fma4, FMA4, FMA4, false),
> + SUBARCH (xop, XOP, XOP, false),
> + SUBARCH (lwp, LWP, LWP, false),
> + SUBARCH (movbe, MOVBE, MOVBE, false),
> + SUBARCH (cx16, CX16, CX16, false),
> + SUBARCH (ept, EPT, EPT, false),
> + SUBARCH (lzcnt, LZCNT, LZCNT, false),
> + SUBARCH (popcnt, POPCNT, POPCNT, false),
> + SUBARCH (hle, HLE, HLE, false),
> + SUBARCH (rtm, RTM, RTM, false),
> + SUBARCH (invpcid, INVPCID, INVPCID, false),
> + SUBARCH (clflush, CLFLUSH, CLFLUSH, false),
> + SUBARCH (nop, NOP, NOP, false),
> + SUBARCH (syscall, SYSCALL, SYSCALL, false),
> + SUBARCH (rdtscp, RDTSCP, RDTSCP, false),
> + SUBARCH (3dnow, 3DNOW, 3DNOW, false),
> + SUBARCH (3dnowa, 3DNOWA, 3DNOWA, false),
> + SUBARCH (padlock, PADLOCK, PADLOCK, false),
> + SUBARCH (pacifica, SVME, SVME, true),
> + SUBARCH (svme, SVME, SVME, false),
> + SUBARCH (sse4a, SSE4A, SSE4A, false),
> + SUBARCH (abm, ABM, ABM, false),
> + SUBARCH (bmi, BMI, BMI, false),
> + SUBARCH (tbm, TBM, TBM, false),
> + SUBARCH (adx, ADX, ADX, false),
> + SUBARCH (rdseed, RDSEED, RDSEED, false),
> + SUBARCH (prfchw, PRFCHW, PRFCHW, false),
> + SUBARCH (smap, SMAP, SMAP, false),
> + SUBARCH (mpx, MPX, MPX, false),
> + SUBARCH (sha, SHA, SHA, false),
> + SUBARCH (clflushopt, CLFLUSHOPT, CLFLUSHOPT, false),
> + SUBARCH (prefetchwt1, PREFETCHWT1, PREFETCHWT1, false),
> + SUBARCH (se1, SE1, SE1, false),
> + SUBARCH (clwb, CLWB, CLWB, false),
> + SUBARCH (avx512ifma, AVX512IFMA, ANY_AVX512IFMA, false),
> + SUBARCH (avx512vbmi, AVX512VBMI, ANY_AVX512VBMI, false),
> + SUBARCH (avx512_4fmaps, AVX512_4FMAPS, ANY_AVX512_4FMAPS, false),
> + SUBARCH (avx512_4vnniw, AVX512_4VNNIW, ANY_AVX512_4VNNIW, false),
> + SUBARCH (avx512_vpopcntdq, AVX512_VPOPCNTDQ, ANY_AVX512_VPOPCNTDQ, false),
> + SUBARCH (avx512_vbmi2, AVX512_VBMI2, ANY_AVX512_VBMI2, false),
> + SUBARCH (avx512_vnni, AVX512_VNNI, ANY_AVX512_VNNI, false),
> + SUBARCH (avx512_bitalg, AVX512_BITALG, ANY_AVX512_BITALG, false),
> + SUBARCH (avx_vnni, AVX_VNNI, ANY_AVX_VNNI, false),
> + SUBARCH (clzero, CLZERO, CLZERO, false),
> + SUBARCH (mwaitx, MWAITX, MWAITX, false),
> + SUBARCH (ospke, OSPKE, OSPKE, false),
> + SUBARCH (rdpid, RDPID, RDPID, false),
> + SUBARCH (ptwrite, PTWRITE, PTWRITE, false),
> + SUBARCH (ibt, IBT, ANY_IBT, false),
> + SUBARCH (shstk, SHSTK, ANY_SHSTK, false),
> + SUBARCH (gfni, GFNI, GFNI, false),
> + SUBARCH (vaes, VAES, VAES, false),
> + SUBARCH (vpclmulqdq, VPCLMULQDQ, VPCLMULQDQ, false),
> + SUBARCH (wbnoinvd, WBNOINVD, WBNOINVD, false),
> + SUBARCH (pconfig, PCONFIG, PCONFIG, false),
> + SUBARCH (waitpkg, WAITPKG, WAITPKG, false),
> + SUBARCH (cldemote, CLDEMOTE, CLDEMOTE, false),
> + SUBARCH (amx_int8, AMX_INT8, ANY_AMX_INT8, false),
> + SUBARCH (amx_bf16, AMX_BF16, ANY_AMX_BF16, false),
> + SUBARCH (amx_tile, AMX_TILE, ANY_AMX_TILE, false),
> + SUBARCH (movdiri, MOVDIRI, ANY_MOVDIRI, false),
> + SUBARCH (movdir64b, MOVDIR64B, ANY_MOVDIR64B, false),
> + SUBARCH (avx512_bf16, AVX512_BF16, ANY_AVX512_BF16, false),
> + SUBARCH (avx512_vp2intersect, AVX512_VP2INTERSECT,
> + ANY_AVX512_VP2INTERSECT, false),
> + SUBARCH (tdx, TDX, ANY_TDX, false),
> + SUBARCH (enqcmd, ENQCMD, ANY_ENQCMD, false),
> + SUBARCH (serialize, SERIALIZE, ANY_SERIALIZE, false),
> + SUBARCH (rdpru, RDPRU, RDPRU, false),
> + SUBARCH (mcommit, MCOMMIT, MCOMMIT, false),
> + SUBARCH (sev_es, SEV_ES, SEV_ES, false),
> + SUBARCH (tsxldtrk, TSXLDTRK, ANY_TSXLDTRK, false),
> + SUBARCH (kl, KL, ANY_KL, false),
> + SUBARCH (widekl, WIDEKL, ANY_WIDEKL, false),
> + SUBARCH (uintr, UINTR, ANY_UINTR, false),
> + SUBARCH (hreset, HRESET, ANY_HRESET, false),
> + SUBARCH (avx512_fp16, AVX512_FP16, ANY_AVX512_FP16, false),
> };
>
> #undef SUBARCH
> #undef ARCH
>
> -static const noarch_entry cpu_noarch[] =
> -{
> - { STRING_COMMA_LEN ("no87"), CPU_ANY_X87_FLAGS },
> - { STRING_COMMA_LEN ("no287"), CPU_ANY_287_FLAGS },
> - { STRING_COMMA_LEN ("no387"), CPU_ANY_387_FLAGS },
> - { STRING_COMMA_LEN ("no687"), CPU_ANY_687_FLAGS },
> - { STRING_COMMA_LEN ("nocmov"), CPU_ANY_CMOV_FLAGS },
> - { STRING_COMMA_LEN ("nofxsr"), CPU_ANY_FXSR_FLAGS },
> - { STRING_COMMA_LEN ("nommx"), CPU_ANY_MMX_FLAGS },
> - { STRING_COMMA_LEN ("nosse"), CPU_ANY_SSE_FLAGS },
> - { STRING_COMMA_LEN ("nosse2"), CPU_ANY_SSE2_FLAGS },
> - { STRING_COMMA_LEN ("nosse3"), CPU_ANY_SSE3_FLAGS },
> - { STRING_COMMA_LEN ("nosse4a"), CPU_ANY_SSE4A_FLAGS },
> - { STRING_COMMA_LEN ("nossse3"), CPU_ANY_SSSE3_FLAGS },
> - { STRING_COMMA_LEN ("nosse4.1"), CPU_ANY_SSE4_1_FLAGS },
> - { STRING_COMMA_LEN ("nosse4.2"), CPU_ANY_SSE4_2_FLAGS },
> - { STRING_COMMA_LEN ("nosse4"), CPU_ANY_SSE4_1_FLAGS },
> - { STRING_COMMA_LEN ("noavx"), CPU_ANY_AVX_FLAGS },
> - { STRING_COMMA_LEN ("noavx2"), CPU_ANY_AVX2_FLAGS },
> - { STRING_COMMA_LEN ("noavx512f"), CPU_ANY_AVX512F_FLAGS },
> - { STRING_COMMA_LEN ("noavx512cd"), CPU_ANY_AVX512CD_FLAGS },
> - { STRING_COMMA_LEN ("noavx512er"), CPU_ANY_AVX512ER_FLAGS },
> - { STRING_COMMA_LEN ("noavx512pf"), CPU_ANY_AVX512PF_FLAGS },
> - { STRING_COMMA_LEN ("noavx512dq"), CPU_ANY_AVX512DQ_FLAGS },
> - { STRING_COMMA_LEN ("noavx512bw"), CPU_ANY_AVX512BW_FLAGS },
> - { STRING_COMMA_LEN ("noavx512vl"), CPU_ANY_AVX512VL_FLAGS },
> - { STRING_COMMA_LEN ("noavx512ifma"), CPU_ANY_AVX512IFMA_FLAGS },
> - { STRING_COMMA_LEN ("noavx512vbmi"), CPU_ANY_AVX512VBMI_FLAGS },
> - { STRING_COMMA_LEN ("noavx512_4fmaps"), CPU_ANY_AVX512_4FMAPS_FLAGS },
> - { STRING_COMMA_LEN ("noavx512_4vnniw"), CPU_ANY_AVX512_4VNNIW_FLAGS },
> - { STRING_COMMA_LEN ("noavx512_vpopcntdq"), CPU_ANY_AVX512_VPOPCNTDQ_FLAGS },
> - { STRING_COMMA_LEN ("noavx512_vbmi2"), CPU_ANY_AVX512_VBMI2_FLAGS },
> - { STRING_COMMA_LEN ("noavx512_vnni"), CPU_ANY_AVX512_VNNI_FLAGS },
> - { STRING_COMMA_LEN ("noavx512_bitalg"), CPU_ANY_AVX512_BITALG_FLAGS },
> - { STRING_COMMA_LEN ("noavx_vnni"), CPU_ANY_AVX_VNNI_FLAGS },
> - { STRING_COMMA_LEN ("noibt"), CPU_ANY_IBT_FLAGS },
> - { STRING_COMMA_LEN ("noshstk"), CPU_ANY_SHSTK_FLAGS },
> - { STRING_COMMA_LEN ("noamx_int8"), CPU_ANY_AMX_INT8_FLAGS },
> - { STRING_COMMA_LEN ("noamx_bf16"), CPU_ANY_AMX_BF16_FLAGS },
> - { STRING_COMMA_LEN ("noamx_tile"), CPU_ANY_AMX_TILE_FLAGS },
> - { STRING_COMMA_LEN ("nomovdiri"), CPU_ANY_MOVDIRI_FLAGS },
> - { STRING_COMMA_LEN ("nomovdir64b"), CPU_ANY_MOVDIR64B_FLAGS },
> - { STRING_COMMA_LEN ("noavx512_bf16"), CPU_ANY_AVX512_BF16_FLAGS },
> - { STRING_COMMA_LEN ("noavx512_vp2intersect"),
> - CPU_ANY_AVX512_VP2INTERSECT_FLAGS },
> - { STRING_COMMA_LEN ("notdx"), CPU_ANY_TDX_FLAGS },
> - { STRING_COMMA_LEN ("noenqcmd"), CPU_ANY_ENQCMD_FLAGS },
> - { STRING_COMMA_LEN ("noserialize"), CPU_ANY_SERIALIZE_FLAGS },
> - { STRING_COMMA_LEN ("notsxldtrk"), CPU_ANY_TSXLDTRK_FLAGS },
> - { STRING_COMMA_LEN ("nokl"), CPU_ANY_KL_FLAGS },
> - { STRING_COMMA_LEN ("nowidekl"), CPU_ANY_WIDEKL_FLAGS },
> - { STRING_COMMA_LEN ("nouintr"), CPU_ANY_UINTR_FLAGS },
> - { STRING_COMMA_LEN ("nohreset"), CPU_ANY_HRESET_FLAGS },
> - { STRING_COMMA_LEN ("noavx512_fp16"), CPU_ANY_AVX512_FP16_FLAGS },
> -};
> -
> #ifdef I386COFF
> /* Like s_lcomm_internal in gas/read.c but the alignment string
> is allowed to be optional. */
> @@ -2715,9 +2655,9 @@ extend_cpu_sub_arch_name (const char *na
> {
> if (cpu_sub_arch_name)
> cpu_sub_arch_name = reconcat (cpu_sub_arch_name, cpu_sub_arch_name,
> - name, (const char *) NULL);
> + ".", name, (const char *) NULL);
> else
> - cpu_sub_arch_name = xstrdup (name);
> + cpu_sub_arch_name = concat (".", name, (const char *) NULL);
> }
>
> static void
> @@ -2756,7 +2696,7 @@ set_cpu_arch (int dummy ATTRIBUTE_UNUSED
> cpu_arch_flags.bitfield.cpuno64 = 1;
> }
> cpu_arch_isa = PROCESSOR_UNKNOWN;
> - cpu_arch_isa_flags = cpu_arch[flag_code == CODE_64BIT].flags;
> + cpu_arch_isa_flags = cpu_arch[flag_code == CODE_64BIT].enable;
> if (!cpu_arch_tune_set)
> {
> cpu_arch_tune = cpu_arch_isa;
> @@ -2769,16 +2709,17 @@ set_cpu_arch (int dummy ATTRIBUTE_UNUSED
>
> for (; j < ARRAY_SIZE (cpu_arch); j++)
> {
> - if (strcmp (string, cpu_arch[j].name) == 0)
> + if (strcmp (string + (*string == '.'), cpu_arch[j].name) == 0
> + && (*string == '.') == (cpu_arch[j].type == PROCESSOR_NONE))
> {
> if (*string != '.')
> {
> - check_cpu_arch_compatible (string, cpu_arch[j].flags);
> + check_cpu_arch_compatible (string, cpu_arch[j].enable);
>
> cpu_arch_name = cpu_arch[j].name;
> xfree (cpu_sub_arch_name);
> cpu_sub_arch_name = NULL;
> - cpu_arch_flags = cpu_arch[j].flags;
> + cpu_arch_flags = cpu_arch[j].enable;
> if (flag_code == CODE_64BIT)
> {
> cpu_arch_flags.bitfield.cpu64 = 1;
> @@ -2790,7 +2731,7 @@ set_cpu_arch (int dummy ATTRIBUTE_UNUSED
> cpu_arch_flags.bitfield.cpuno64 = 1;
> }
> cpu_arch_isa = cpu_arch[j].type;
> - cpu_arch_isa_flags = cpu_arch[j].flags;
> + cpu_arch_isa_flags = cpu_arch[j].enable;
> if (!cpu_arch_tune_set)
> {
> cpu_arch_tune = cpu_arch_isa;
> @@ -2799,36 +2740,40 @@ set_cpu_arch (int dummy ATTRIBUTE_UNUSED
> break;
> }
>
> + if (cpu_flags_all_zero (&cpu_arch[j].enable))
> + continue;
> +
> flags = cpu_flags_or (cpu_arch_flags,
> - cpu_arch[j].flags);
> + cpu_arch[j].enable);
>
> if (!cpu_flags_equal (&flags, &cpu_arch_flags))
> {
> - extend_cpu_sub_arch_name (string);
> + extend_cpu_sub_arch_name (string + 1);
> cpu_arch_flags = flags;
> cpu_arch_isa_flags = flags;
> }
> else
> cpu_arch_isa_flags
> = cpu_flags_or (cpu_arch_isa_flags,
> - cpu_arch[j].flags);
> + cpu_arch[j].enable);
> (void) restore_line_pointer (e);
> demand_empty_rest_of_line ();
> return;
> }
> }
>
> - if (*string == '.' && j >= ARRAY_SIZE (cpu_arch))
> + if (startswith (string, ".no") && j >= ARRAY_SIZE (cpu_arch))
> {
> /* Disable an ISA extension. */
> - for (j = 0; j < ARRAY_SIZE (cpu_noarch); j++)
> - if (strcmp (string + 1, cpu_noarch [j].name) == 0)
> + for (j = 0; j < ARRAY_SIZE (cpu_arch); j++)
> + if (cpu_arch[j].type == PROCESSOR_NONE
> + && strcmp (string + 3, cpu_arch[j].name) == 0)
> {
> flags = cpu_flags_and_not (cpu_arch_flags,
> - cpu_noarch[j].flags);
> + cpu_arch[j].disable);
> if (!cpu_flags_equal (&flags, &cpu_arch_flags))
> {
> - extend_cpu_sub_arch_name (string);
> + extend_cpu_sub_arch_name (string + 1);
> cpu_arch_flags = flags;
> cpu_arch_isa_flags = flags;
> }
> @@ -2836,8 +2781,6 @@ set_cpu_arch (int dummy ATTRIBUTE_UNUSED
> demand_empty_rest_of_line ();
> return;
> }
> -
> - j = ARRAY_SIZE (cpu_arch);
> }
>
> if (j == ARRAY_SIZE (cpu_arch))
> @@ -13288,18 +13231,19 @@ md_parse_option (int c, const char *arg)
> *next++ = '\0';
> for (j = 0; j < ARRAY_SIZE (cpu_arch); j++)
> {
> - if (arch == saved && strcmp (arch, cpu_arch [j].name) == 0)
> + if (arch == saved && cpu_arch[j].type != PROCESSOR_NONE
> + && strcmp (arch, cpu_arch[j].name) == 0)
> {
> /* Processor. */
> - if (! cpu_arch[j].flags.bitfield.cpui386)
> + if (! cpu_arch[j].enable.bitfield.cpui386)
> continue;
>
> cpu_arch_name = cpu_arch[j].name;
> xfree (cpu_sub_arch_name);
> cpu_sub_arch_name = NULL;
> - cpu_arch_flags = cpu_arch[j].flags;
> + cpu_arch_flags = cpu_arch[j].enable;
> cpu_arch_isa = cpu_arch[j].type;
> - cpu_arch_isa_flags = cpu_arch[j].flags;
> + cpu_arch_isa_flags = cpu_arch[j].enable;
> if (!cpu_arch_tune_set)
> {
> cpu_arch_tune = cpu_arch_isa;
> @@ -13307,39 +13251,41 @@ md_parse_option (int c, const char *arg)
> }
> break;
> }
> - else if (*cpu_arch [j].name == '.'
> - && strcmp (arch, cpu_arch [j].name + 1) == 0)
> + else if (cpu_arch[j].type == PROCESSOR_NONE
> + && strcmp (arch, cpu_arch[j].name) == 0
> + && !cpu_flags_all_zero (&cpu_arch[j].enable))
> {
> /* ISA extension. */
> i386_cpu_flags flags;
>
> flags = cpu_flags_or (cpu_arch_flags,
> - cpu_arch[j].flags);
> + cpu_arch[j].enable);
>
> if (!cpu_flags_equal (&flags, &cpu_arch_flags))
> {
> - extend_cpu_sub_arch_name (cpu_arch[j].name);
> + extend_cpu_sub_arch_name (arch);
> cpu_arch_flags = flags;
> cpu_arch_isa_flags = flags;
> }
> else
> cpu_arch_isa_flags
> = cpu_flags_or (cpu_arch_isa_flags,
> - cpu_arch[j].flags);
> + cpu_arch[j].enable);
> break;
> }
> }
>
> - if (j >= ARRAY_SIZE (cpu_arch))
> + if (j >= ARRAY_SIZE (cpu_arch) && startswith (arch, "no"))
> {
> /* Disable an ISA extension. */
> - for (j = 0; j < ARRAY_SIZE (cpu_noarch); j++)
> - if (strcmp (arch, cpu_noarch [j].name) == 0)
> + for (j = 0; j < ARRAY_SIZE (cpu_arch); j++)
> + if (cpu_arch[j].type == PROCESSOR_NONE
> + && strcmp (arch + 2, cpu_arch[j].name) == 0)
> {
> i386_cpu_flags flags;
>
> flags = cpu_flags_and_not (cpu_arch_flags,
> - cpu_noarch[j].flags);
> + cpu_arch[j].disable);
> if (!cpu_flags_equal (&flags, &cpu_arch_flags))
> {
> extend_cpu_sub_arch_name (arch);
> @@ -13348,9 +13294,6 @@ md_parse_option (int c, const char *arg)
> }
> break;
> }
> -
> - if (j >= ARRAY_SIZE (cpu_noarch))
> - j = ARRAY_SIZE (cpu_arch);
> }
>
> if (j >= ARRAY_SIZE (cpu_arch))
> @@ -13367,11 +13310,12 @@ md_parse_option (int c, const char *arg)
> as_fatal (_("invalid -mtune= option: `%s'"), arg);
> for (j = 0; j < ARRAY_SIZE (cpu_arch); j++)
> {
> - if (strcmp (arg, cpu_arch [j].name) == 0)
> + if (cpu_arch[j].type != PROCESSOR_NONE
> + && strcmp (arg, cpu_arch[j].name) == 0)
> {
> cpu_arch_tune_set = 1;
> cpu_arch_tune = cpu_arch [j].type;
> - cpu_arch_tune_flags = cpu_arch[j].flags;
> + cpu_arch_tune_flags = cpu_arch[j].enable;
> break;
> }
> }
> @@ -13751,15 +13695,10 @@ show_arch (FILE *stream, int ext, int ch
>
> name = cpu_arch [j].name;
> len = cpu_arch [j].len;
> - if (*name == '.')
> + if (cpu_arch[j].type == PROCESSOR_NONE)
> {
> /* It is an extension. Skip if we aren't asked to show it. */
> - if (ext)
> - {
> - name++;
> - len--;
> - }
> - else
> + if (!ext || cpu_flags_all_zero (&cpu_arch[j].enable))
> continue;
> }
> else if (ext)
> @@ -13767,7 +13706,7 @@ show_arch (FILE *stream, int ext, int ch
> /* It is an processor. Skip if we show only extension. */
> continue;
> }
> - else if (check && ! cpu_arch[j].flags.bitfield.cpui386)
> + else if (check && ! cpu_arch[j].enable.bitfield.cpui386)
> {
> /* It is an impossible processor - skip. */
> continue;
> @@ -13778,12 +13717,17 @@ show_arch (FILE *stream, int ext, int ch
>
> /* Display disabled extensions. */
> if (ext)
> - for (j = 0; j < ARRAY_SIZE (cpu_noarch); j++)
> + for (j = 0; j < ARRAY_SIZE (cpu_arch); j++)
> {
> - name = cpu_noarch [j].name;
> - len = cpu_noarch [j].len;
> - p = output_message (stream, p, message, start, &left, name,
> - len);
> + char *str;
> +
> + if (cpu_arch[j].type != PROCESSOR_NONE
> + || !cpu_flags_all_zero (&cpu_arch[j].enable))
> + continue;
> + str = xasprintf ("no%s", cpu_arch[j].name);
> + p = output_message (stream, p, message, start, &left, str,
> + strlen (str));
> + xfree (str);
> }
>
> *p = '\0';
> @@ -13827,7 +13771,7 @@ md_show_usage (FILE *stream)
> generate code for CPU and EXTENSION, CPU is one of:\n"));
> show_arch (stream, 0, 1);
> fprintf (stream, _("\
> - EXTENSION is combination of:\n"));
> + EXTENSION is combination of (possibly \"no\"-prefixed):\n"));
> show_arch (stream, 1, 0);
> fprintf (stream, _("\
> -mtune=CPU optimize for CPU, CPU is one of:\n"));
> @@ -13985,9 +13929,9 @@ i386_target_format (void)
> as_fatal (_("unknown architecture"));
>
> if (cpu_flags_all_zero (&cpu_arch_isa_flags))
> - cpu_arch_isa_flags = cpu_arch[flag_code == CODE_64BIT].flags;
> + cpu_arch_isa_flags = cpu_arch[flag_code == CODE_64BIT].enable;
> if (cpu_flags_all_zero (&cpu_arch_tune_flags))
> - cpu_arch_tune_flags = cpu_arch[flag_code == CODE_64BIT].flags;
> + cpu_arch_tune_flags = cpu_arch[flag_code == CODE_64BIT].enable;
>
> switch (OUTPUT_FLAVOR)
> {
>
OK.
Thanks.
--
H.J.
^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH 7/7] x86: introduce a state stack for .arch
2022-06-30 12:52 [PATCH] gas/x86: .arch / -march= enhancements Jan Beulich
` (5 preceding siblings ...)
2022-06-30 12:55 ` [PATCH 6/7] x86: generalize disabling of sub-architectures Jan Beulich
@ 2022-06-30 12:55 ` Jan Beulich
2022-06-30 23:15 ` H.J. Lu
6 siblings, 1 reply; 17+ messages in thread
From: Jan Beulich @ 2022-06-30 12:55 UTC (permalink / raw)
To: Binutils
When using just slightly non-trivial combinations of .arch, it can be
quite useful to be able to go back to prior state without needing to
re-invoke perhaps many earlier directives and without needing to invoke
perhaps many "negative" ones. Like some other architectures allow
saving (pushing) and restoring (popping) present/prior state.
For now require the same .code<N> to be in effect for ".arch pop" that
was in effect for the corresponding ".arch push".
Also change the global "no_cond_jump_promotion" to be bool, to match the
new struct field.
--- a/gas/config/tc-i386.c
+++ b/gas/config/tc-i386.c
@@ -788,7 +788,7 @@ i386_cpu_flags cpu_arch_isa_flags;
/* If set, conditional jumps are not automatically promoted to handle
larger than a byte offset. */
-static unsigned int no_cond_jump_promotion = 0;
+static bool no_cond_jump_promotion = false;
/* Encode SSE instructions with VEX prefix. */
static unsigned int sse2avx;
@@ -2663,6 +2663,20 @@ extend_cpu_sub_arch_name (const char *na
static void
set_cpu_arch (int dummy ATTRIBUTE_UNUSED)
{
+ typedef struct arch_stack_entry
+ {
+ const struct arch_stack_entry *prev;
+ const char *name;
+ char *sub_name;
+ i386_cpu_flags flags;
+ i386_cpu_flags isa_flags;
+ enum processor_type isa;
+ enum flag_code flag_code;
+ char stackop_size;
+ bool no_cond_jump_promotion;
+ } arch_stack_entry;
+ static const arch_stack_entry *arch_stack_top;
+
SKIP_WHITESPACE ();
if (!is_end_of_line[(unsigned char) *input_line_pointer])
@@ -2706,6 +2720,67 @@ set_cpu_arch (int dummy ATTRIBUTE_UNUSED
j = ARRAY_SIZE (cpu_arch) + 1;
}
}
+ else if (strcmp (string, "push") == 0)
+ {
+ arch_stack_entry *top = XNEW (arch_stack_entry);
+
+ top->name = cpu_arch_name;
+ if (cpu_sub_arch_name)
+ top->sub_name = xstrdup (cpu_sub_arch_name);
+ else
+ top->sub_name = NULL;
+ top->flags = cpu_arch_flags;
+ top->isa = cpu_arch_isa;
+ top->isa_flags = cpu_arch_isa_flags;
+ top->flag_code = flag_code;
+ top->stackop_size = stackop_size;
+ top->no_cond_jump_promotion = no_cond_jump_promotion;
+
+ top->prev = arch_stack_top;
+ arch_stack_top = top;
+
+ (void) restore_line_pointer (e);
+ demand_empty_rest_of_line ();
+ return;
+ }
+ else if (strcmp (string, "pop") == 0)
+ {
+ const arch_stack_entry *top = arch_stack_top;
+
+ if (!top)
+ as_bad (_(".arch stack is empty"));
+ else if (top->flag_code != flag_code
+ || top->stackop_size != stackop_size)
+ {
+ static const unsigned int bits[] = {
+ [CODE_16BIT] = 16,
+ [CODE_32BIT] = 32,
+ [CODE_64BIT] = 64,
+ };
+
+ as_bad (_("this `.arch pop' requires `.code%u%s' to be in effect"),
+ bits[top->flag_code],
+ top->stackop_size == LONG_MNEM_SUFFIX ? "gcc" : "");
+ }
+ else
+ {
+ arch_stack_top = top->prev;
+
+ cpu_arch_name = top->name;
+ xfree (cpu_sub_arch_name);
+ cpu_sub_arch_name = top->sub_name;
+ cpu_arch_flags = top->flags;
+ cpu_arch_isa = top->isa;
+ cpu_arch_isa_flags = top->isa_flags;
+ no_cond_jump_promotion = top->no_cond_jump_promotion;
+
+ XDELETE (top);
+ }
+
+ (void) restore_line_pointer (e);
+ demand_empty_rest_of_line ();
+ return;
+ }
for (; j < ARRAY_SIZE (cpu_arch); j++)
{
@@ -13685,6 +13760,10 @@ show_arch (FILE *stream, int ext, int ch
{
p = output_message (stream, p, message, start, &left,
STRING_COMMA_LEN ("default"));
+ p = output_message (stream, p, message, start, &left,
+ STRING_COMMA_LEN ("push"));
+ p = output_message (stream, p, message, start, &left,
+ STRING_COMMA_LEN ("pop"));
}
for (j = 0; j < ARRAY_SIZE (cpu_arch); j++)
--- a/gas/doc/c-i386.texi
+++ b/gas/doc/c-i386.texi
@@ -1504,7 +1504,7 @@ directive enables a warning when gas det
supported on the CPU specified. The choices for @var{cpu_type} are:
@multitable @columnfractions .20 .20 .20 .20
-@item @samp{default}
+@item @samp{default} @tab @samp{push} @tab @samp{pop}
@item @samp{i8086} @tab @samp{i186} @tab @samp{i286} @tab @samp{i386}
@item @samp{i486} @tab @samp{i586} @tab @samp{i686} @tab @samp{pentium}
@item @samp{pentiumpro} @tab @samp{pentiumii} @tab @samp{pentiumiii} @tab @samp{pentium4}
--- /dev/null
+++ b/gas/testsuite/gas/i386/arch-stk.l
@@ -0,0 +1,43 @@
+.*: Assembler messages:
+.*:3: Error:.*`cmovl'.*
+.*:10: Error:.*`cmovg'.*
+.*:17: Error:.*`cmovz'.*
+.*:21: Error:.*`\.arch pop'.*`\.code32'.*
+.*:28: Error:.*`\.arch pop'.*`\.code16gcc'.*
+.*:32: Error:.*\.arch.*empty.*
+GAS LISTING .*
+
+
+[ ]*[0-9]*[ ]+\.text
+[ ]*[0-9]*[ ]+start:
+[ ]*[0-9]*[ ]+cmovl %eax, %ecx
+[ ]*[0-9]*[ ]*
+[ ]*[0-9]*[ ]+\.arch push
+[ ]*[0-9]*[ ]+\.arch default
+[ ]*[0-9]*[ ]+\?\?\?\? 0F4DC8[ ]+cmovnl %eax, %ecx
+[ ]*[0-9]*[ ]*
+[ ]*[0-9]*[ ]+\.arch pop
+[ ]*[0-9]*[ ]+cmovg %eax, %ecx
+[ ]*[0-9]*[ ]*
+[ ]*[0-9]*[ ]+\.arch push
+[ ]*[0-9]*[ ]+\.arch \.cmov
+[ ]*[0-9]*[ ]+\?\?\?\? 0F4EC8[ ]+cmovng %eax, %ecx
+[ ]*[0-9]*[ ]*
+[ ]*[0-9]*[ ]+\.arch pop
+[ ]*[0-9]*[ ]+cmovz %eax, %ecx
+[ ]*[0-9]*[ ]*
+[ ]*[0-9]*[ ]+\.arch push
+[ ]*[0-9]*[ ]+\.code16
+[ ]*[0-9]*[ ]+\.arch pop
+[ ]*[0-9]*[ ]+\.code32
+[ ]*[0-9]*[ ]+\.arch pop
+[ ]*[0-9]*[ ]*
+[ ]*[0-9]*[ ]+\.code16gcc
+[ ]*[0-9]*[ ]+\.arch push
+[ ]*[0-9]*[ ]+\.code32
+[ ]*[0-9]*[ ]+\.arch pop
+[ ]*[0-9]*[ ]+\.code16gcc
+[ ]*[0-9]*[ ]+\.arch pop
+[ ]*[0-9]*[ ]*
+[ ]*[0-9]*[ ]+\.arch pop
+#pass
--- /dev/null
+++ b/gas/testsuite/gas/i386/arch-stk.s
@@ -0,0 +1,34 @@
+ .text
+start:
+ cmovl %eax, %ecx
+
+ .arch push
+ .arch default
+ cmovnl %eax, %ecx
+
+ .arch pop
+ cmovg %eax, %ecx
+
+ .arch push
+ .arch .cmov
+ cmovng %eax, %ecx
+
+ .arch pop
+ cmovz %eax, %ecx
+
+ .arch push
+ .code16
+ .arch pop
+ .code32
+ .arch pop
+
+ .code16gcc
+ .arch push
+ .code32
+ .arch pop
+ .code16gcc
+ .arch pop
+
+ .arch pop
+
+ .end
--- a/gas/testsuite/gas/i386/i386.exp
+++ b/gas/testsuite/gas/i386/i386.exp
@@ -206,6 +206,7 @@ if [gas_32_check] then {
run_dump_test "arch-13"
run_dump_test "arch-14"
run_list_test "arch-dflt" "-march=generic32 -al"
+ run_list_test "arch-stk" "-march=generic32 -al"
run_dump_test "8087"
run_dump_test "287"
run_dump_test "387"
^ permalink raw reply [flat|nested] 17+ messages in thread* Re: [PATCH 7/7] x86: introduce a state stack for .arch
2022-06-30 12:55 ` [PATCH 7/7] x86: introduce a state stack for .arch Jan Beulich
@ 2022-06-30 23:15 ` H.J. Lu
0 siblings, 0 replies; 17+ messages in thread
From: H.J. Lu @ 2022-06-30 23:15 UTC (permalink / raw)
To: Jan Beulich; +Cc: Binutils
On Thu, Jun 30, 2022 at 5:55 AM Jan Beulich <jbeulich@suse.com> wrote:
>
> When using just slightly non-trivial combinations of .arch, it can be
> quite useful to be able to go back to prior state without needing to
> re-invoke perhaps many earlier directives and without needing to invoke
> perhaps many "negative" ones. Like some other architectures allow
> saving (pushing) and restoring (popping) present/prior state.
>
> For now require the same .code<N> to be in effect for ".arch pop" that
> was in effect for the corresponding ".arch push".
>
> Also change the global "no_cond_jump_promotion" to be bool, to match the
> new struct field.
>
> --- a/gas/config/tc-i386.c
> +++ b/gas/config/tc-i386.c
> @@ -788,7 +788,7 @@ i386_cpu_flags cpu_arch_isa_flags;
>
> /* If set, conditional jumps are not automatically promoted to handle
> larger than a byte offset. */
> -static unsigned int no_cond_jump_promotion = 0;
> +static bool no_cond_jump_promotion = false;
>
> /* Encode SSE instructions with VEX prefix. */
> static unsigned int sse2avx;
> @@ -2663,6 +2663,20 @@ extend_cpu_sub_arch_name (const char *na
> static void
> set_cpu_arch (int dummy ATTRIBUTE_UNUSED)
> {
> + typedef struct arch_stack_entry
> + {
> + const struct arch_stack_entry *prev;
> + const char *name;
> + char *sub_name;
> + i386_cpu_flags flags;
> + i386_cpu_flags isa_flags;
> + enum processor_type isa;
> + enum flag_code flag_code;
> + char stackop_size;
> + bool no_cond_jump_promotion;
> + } arch_stack_entry;
> + static const arch_stack_entry *arch_stack_top;
> +
> SKIP_WHITESPACE ();
>
> if (!is_end_of_line[(unsigned char) *input_line_pointer])
> @@ -2706,6 +2720,67 @@ set_cpu_arch (int dummy ATTRIBUTE_UNUSED
> j = ARRAY_SIZE (cpu_arch) + 1;
> }
> }
> + else if (strcmp (string, "push") == 0)
> + {
> + arch_stack_entry *top = XNEW (arch_stack_entry);
> +
> + top->name = cpu_arch_name;
> + if (cpu_sub_arch_name)
> + top->sub_name = xstrdup (cpu_sub_arch_name);
> + else
> + top->sub_name = NULL;
> + top->flags = cpu_arch_flags;
> + top->isa = cpu_arch_isa;
> + top->isa_flags = cpu_arch_isa_flags;
> + top->flag_code = flag_code;
> + top->stackop_size = stackop_size;
> + top->no_cond_jump_promotion = no_cond_jump_promotion;
> +
> + top->prev = arch_stack_top;
> + arch_stack_top = top;
> +
> + (void) restore_line_pointer (e);
> + demand_empty_rest_of_line ();
> + return;
> + }
> + else if (strcmp (string, "pop") == 0)
> + {
> + const arch_stack_entry *top = arch_stack_top;
> +
> + if (!top)
> + as_bad (_(".arch stack is empty"));
> + else if (top->flag_code != flag_code
> + || top->stackop_size != stackop_size)
> + {
> + static const unsigned int bits[] = {
> + [CODE_16BIT] = 16,
> + [CODE_32BIT] = 32,
> + [CODE_64BIT] = 64,
> + };
> +
> + as_bad (_("this `.arch pop' requires `.code%u%s' to be in effect"),
> + bits[top->flag_code],
> + top->stackop_size == LONG_MNEM_SUFFIX ? "gcc" : "");
> + }
> + else
> + {
> + arch_stack_top = top->prev;
> +
> + cpu_arch_name = top->name;
> + xfree (cpu_sub_arch_name);
Can we just use free?
> + cpu_sub_arch_name = top->sub_name;
> + cpu_arch_flags = top->flags;
> + cpu_arch_isa = top->isa;
> + cpu_arch_isa_flags = top->isa_flags;
> + no_cond_jump_promotion = top->no_cond_jump_promotion;
> +
> + XDELETE (top);
> + }
> +
> + (void) restore_line_pointer (e);
> + demand_empty_rest_of_line ();
> + return;
> + }
>
> for (; j < ARRAY_SIZE (cpu_arch); j++)
> {
> @@ -13685,6 +13760,10 @@ show_arch (FILE *stream, int ext, int ch
> {
> p = output_message (stream, p, message, start, &left,
> STRING_COMMA_LEN ("default"));
> + p = output_message (stream, p, message, start, &left,
> + STRING_COMMA_LEN ("push"));
> + p = output_message (stream, p, message, start, &left,
> + STRING_COMMA_LEN ("pop"));
> }
>
> for (j = 0; j < ARRAY_SIZE (cpu_arch); j++)
> --- a/gas/doc/c-i386.texi
> +++ b/gas/doc/c-i386.texi
> @@ -1504,7 +1504,7 @@ directive enables a warning when gas det
> supported on the CPU specified. The choices for @var{cpu_type} are:
>
> @multitable @columnfractions .20 .20 .20 .20
> -@item @samp{default}
> +@item @samp{default} @tab @samp{push} @tab @samp{pop}
> @item @samp{i8086} @tab @samp{i186} @tab @samp{i286} @tab @samp{i386}
> @item @samp{i486} @tab @samp{i586} @tab @samp{i686} @tab @samp{pentium}
> @item @samp{pentiumpro} @tab @samp{pentiumii} @tab @samp{pentiumiii} @tab @samp{pentium4}
> --- /dev/null
> +++ b/gas/testsuite/gas/i386/arch-stk.l
> @@ -0,0 +1,43 @@
> +.*: Assembler messages:
> +.*:3: Error:.*`cmovl'.*
> +.*:10: Error:.*`cmovg'.*
> +.*:17: Error:.*`cmovz'.*
> +.*:21: Error:.*`\.arch pop'.*`\.code32'.*
> +.*:28: Error:.*`\.arch pop'.*`\.code16gcc'.*
> +.*:32: Error:.*\.arch.*empty.*
> +GAS LISTING .*
> +
> +
> +[ ]*[0-9]*[ ]+\.text
> +[ ]*[0-9]*[ ]+start:
> +[ ]*[0-9]*[ ]+cmovl %eax, %ecx
> +[ ]*[0-9]*[ ]*
> +[ ]*[0-9]*[ ]+\.arch push
> +[ ]*[0-9]*[ ]+\.arch default
> +[ ]*[0-9]*[ ]+\?\?\?\? 0F4DC8[ ]+cmovnl %eax, %ecx
> +[ ]*[0-9]*[ ]*
> +[ ]*[0-9]*[ ]+\.arch pop
> +[ ]*[0-9]*[ ]+cmovg %eax, %ecx
> +[ ]*[0-9]*[ ]*
> +[ ]*[0-9]*[ ]+\.arch push
> +[ ]*[0-9]*[ ]+\.arch \.cmov
> +[ ]*[0-9]*[ ]+\?\?\?\? 0F4EC8[ ]+cmovng %eax, %ecx
> +[ ]*[0-9]*[ ]*
> +[ ]*[0-9]*[ ]+\.arch pop
> +[ ]*[0-9]*[ ]+cmovz %eax, %ecx
> +[ ]*[0-9]*[ ]*
> +[ ]*[0-9]*[ ]+\.arch push
> +[ ]*[0-9]*[ ]+\.code16
> +[ ]*[0-9]*[ ]+\.arch pop
> +[ ]*[0-9]*[ ]+\.code32
> +[ ]*[0-9]*[ ]+\.arch pop
> +[ ]*[0-9]*[ ]*
> +[ ]*[0-9]*[ ]+\.code16gcc
> +[ ]*[0-9]*[ ]+\.arch push
> +[ ]*[0-9]*[ ]+\.code32
> +[ ]*[0-9]*[ ]+\.arch pop
> +[ ]*[0-9]*[ ]+\.code16gcc
> +[ ]*[0-9]*[ ]+\.arch pop
> +[ ]*[0-9]*[ ]*
> +[ ]*[0-9]*[ ]+\.arch pop
> +#pass
> --- /dev/null
> +++ b/gas/testsuite/gas/i386/arch-stk.s
> @@ -0,0 +1,34 @@
> + .text
> +start:
> + cmovl %eax, %ecx
> +
> + .arch push
> + .arch default
> + cmovnl %eax, %ecx
> +
> + .arch pop
> + cmovg %eax, %ecx
> +
> + .arch push
> + .arch .cmov
> + cmovng %eax, %ecx
> +
> + .arch pop
> + cmovz %eax, %ecx
> +
> + .arch push
> + .code16
> + .arch pop
> + .code32
> + .arch pop
> +
> + .code16gcc
> + .arch push
> + .code32
> + .arch pop
> + .code16gcc
> + .arch pop
> +
> + .arch pop
> +
> + .end
> --- a/gas/testsuite/gas/i386/i386.exp
> +++ b/gas/testsuite/gas/i386/i386.exp
> @@ -206,6 +206,7 @@ if [gas_32_check] then {
> run_dump_test "arch-13"
> run_dump_test "arch-14"
> run_list_test "arch-dflt" "-march=generic32 -al"
> + run_list_test "arch-stk" "-march=generic32 -al"
> run_dump_test "8087"
> run_dump_test "287"
> run_dump_test "387"
>
--
H.J.
^ permalink raw reply [flat|nested] 17+ messages in thread