Re: [PATCH] AArch64: Cleanup CPU option processing code

2022-05-12 Thread Richard Sandiford via Gcc-patches
Wilco Dijkstra writes: > Hi Richard, > >> Looks like you might have attached the old patch. The aarch64_option_restore >> change is mentioned in the changelog but doesn't appear in the patch itself. > > Indeed, not sure how that happened. Here is the correct v2 anyway. > > Wilco > > > The --with-

Re: [PATCH] AArch64: Cleanup CPU option processing code

2022-05-12 Thread Wilco Dijkstra via Gcc-patches
Hi Richard, > Looks like you might have attached the old patch.  The aarch64_option_restore > change is mentioned in the changelog but doesn't appear in the patch itself. Indeed, not sure how that happened. Here is the correct v2 anyway. Wilco The --with-cpu/--with-arch configure option proces

Re: [PATCH] AArch64: Cleanup CPU option processing code

2022-05-12 Thread Richard Sandiford via Gcc-patches
Wilco Dijkstra via Gcc-patches writes: > Hi Richard, > >> Although invoking ./cc1 directly only half-works with --with-arch, >> it half-works well-enough that I'd still like to keep it working. >> But I agree we should apply your change first, then I can follow up >> with a patch to make --with-*

Re: [PATCH] AArch64: Cleanup CPU option processing code

2022-05-11 Thread Wilco Dijkstra via Gcc-patches
Hi Richard, > Although invoking ./cc1 directly only half-works with --with-arch, > it half-works well-enough that I'd still like to keep it working. > But I agree we should apply your change first, then I can follow up > with a patch to make --with-* work with ./cc1 later.  (I have a version > loc

Re: [PATCH] AArch64: Cleanup CPU option processing code

2022-05-09 Thread Richard Sandiford via Gcc-patches
Wilco Dijkstra via Gcc-patches writes: > The --with-cpu/--with-arch configure option processing not only checks valid > arguments > but also sets TARGET_CPU_DEFAULT with a CPU and extension bitmask. This > isn't used > however since a --with-cpu is translated into a -mcpu option which is > pro

[PATCH] AArch64: Cleanup CPU option processing code

2022-05-09 Thread Wilco Dijkstra via Gcc-patches
The --with-cpu/--with-arch configure option processing not only checks valid arguments but also sets TARGET_CPU_DEFAULT with a CPU and extension bitmask. This isn't used however since a --with-cpu is translated into a -mcpu option which is processed as if written on the command-line (so TARGET