On Mon, Aug 2, 2021 at 4:57 PM Richard Earnshaw <rearn...@arm.com> wrote:
> This patch series addresses an issue that has come to light due to a > change in the way GAS handles .fpu directives in the assembler. A fix > to the assembler made in binutils 2.34 to clear out all features > realated to the FPU when .fpu is emitted has started causing problems > for GCC because of the order in which we emit .fpu and .arch_extension > directives. To fully address this we need to re-organize the way in > which the compiler does this. > > I'll hold of pushing the patches for a couple of days. Although I've > gone through the testsuite quite carefully and run this through > several configurations, it's possible that this may have some impact > on the testsuite that I've missed. Christophe, is the any chance you > can run this through your test environment before I commit this? > > Sorry for the delay, still unpacking emails after hollidays. Yes I can run the validation for these patches. I think you mean with all 3 patches combined, not 3 validations (patch 1, patches 1+2, patches 1-3) ? Thanks, Christophe > R. > > Richard Earnshaw (3): > arm: ensure the arch_name is always set for the build target > arm: Don't reconfigure globals in arm_configure_build_target > arm: reorder assembler architecture directives [PR101723] > > gcc/config/arm/arm-c.c | 1 + > gcc/config/arm/arm-cpus.in | 1 + > gcc/config/arm/arm.c | 190 ++++++++---------- > gcc/testsuite/gcc.target/arm/attr-neon.c | 9 +- > gcc/testsuite/gcc.target/arm/attr-neon2.c | 35 +++- > gcc/testsuite/gcc.target/arm/attr-neon3.c | 43 +++- > .../arm/cortex-m55-nofp-flag-hard.c | 2 +- > .../arm/cortex-m55-nofp-flag-softfp.c | 2 +- > .../arm/cortex-m55-nofp-nomve-flag-softfp.c | 2 +- > .../gcc.target/arm/mve/intrinsics/mve_fpu1.c | 5 +- > .../gcc.target/arm/mve/intrinsics/mve_fpu2.c | 5 +- > gcc/testsuite/gcc.target/arm/pr98636.c | 3 +- > 12 files changed, 153 insertions(+), 145 deletions(-) > > -- > 2.25.1 > >
