Hi,
Gentle ping: https://gcc.gnu.org/pipermail/gcc-patches/2022-March/591147.html
BR,
Kewen
on 2022/3/3 10:11 AM, Kewen.Lin via Gcc-patches wrote:
> Hi,
>
> As PR103623 shows, it's a regression failure due to new built-in
> function framework, previously we guard __builtin_{un,}pack_{longdouble,
> ibm128} built-in functions under hard float, so they are unavailable
> with the given configuration. While with new bif infrastructure, it
> becomes available and gets ICE due to incomplete supports.
>
> Segher and Peter pointed out that we should make them available with
> soft float, I agree we can extend them to cover both soft and hard
> float. But considering it's stage 4 now and this regression is
> classified as P1, also the previous behavior requiring hard float
> aligns with what document [1] says, I guess it may be a good idea to
> fix it with the attached small patch to be consistent with the previous
> behavior. Then we can extend the functionality in upcoming stage 1.
>
> Bootstrapped and regtested on powerpc64-linux-gnu P8 and
> powerpc64le-linux-gnu P9 and P10.
>
> Any thoughts?
>
> [1]
> https://gcc.gnu.org/onlinedocs/gcc/Basic-PowerPC-Built-in-Functions-Available-on-ISA-2_002e05.html#Basic-PowerPC-Built-in-Functions-Available-on-ISA-2_002e05
>
> BR,
> Kewen
> ------
> PR target/103623
>
> gcc/ChangeLog:
>
> * config/rs6000/rs6000-builtins.def (__builtin_pack_longdouble): Add
> nosoft attribute.
> (__builtin_unpack_longdouble): Likewise.
> (__builtin_pack_ibm128): Likewise.
> (__builtin_unpack_ibm128): Likewise.
>
