Re: [PATCH v2] aarch64: Add 64 bits fpcr fpsr getter/setter builtins

[Andrea Corallo]

Richard Sandiford <richard.sandif...@arm.com> writes:

> OK with those changes, thanks.
>
> Richard

Hi Richard,

thanks for the quick review.

I've installed the patch with the suggested changes as:

0d7e5fa655e aarch64: Add 64 bit setter getter fpsr fpcr

Regards

  Andrea

gcc/ChangeLog

        * config/aarch64/aarch64-builtins.c (aarch64_builtins): Add enums
        for 64bits fpsr/fpcr getter setters builtin variants.
        (aarch64_init_fpsr_fpcr_builtins): New function.
        (aarch64_general_init_builtins): Modify to make use of the later.
        (aarch64_expand_fpsr_fpcr_setter): New function.
        (aarch64_general_expand_builtin): Modify to make use of the later.
        * config/aarch64/aarch64.md (@aarch64_set_<fpscr_name><GPI:mode>)
        (@aarch64_get_<fpscr_name><GPI:mode>): New patterns replacing and
        generalizing 'get_fpcr', 'set_fpsr'.
        * config/aarch64/iterators.md (GET_FPSCR, SET_FPSCR): New int
        iterators.
        (fpscr_name): New int attribute.
        * doc/extend.texi (__builtin_aarch64_get_fpcr64)
        (__builtin_aarch64_set_fpcr64, __builtin_aarch64_get_fpsr64)
        (__builtin_aarch64_set_fpsr64): Add into AArch64 Built-in
        Functions.

gcc/testsuite/ChangeLog

        * gcc.target/aarch64/get_fpcr64_1.c: New test.
        * gcc.target/aarch64/set_fpcr64_1.c: New test.
        * gcc.target/aarch64/get_fpsr64_1.c: New test.
        * gcc.target/aarch64/set_fpsr64_1.c: New test.