Hi!

On Thu, Mar 09, 2023 at 08:40:36PM -0500, Michael Meissner wrote:
>       PR target/109067
>       * config/rs6000/rs6000.cc (create_complex_muldiv): Delete.
>       (init_float128_ieee): Delete code to switch complex multiply and divide
>       for long double.
>       (complex_multiply_builtin_code): New helper function.
>       (complex_divide_builtin_code): Likewise.
>       (rs6000_mangle_decl_assembler_name): Add support for mangling the name
>       of complex 128-bit multiply and divide built-in functions.

> --- /dev/null
> +++ b/gcc/testsuite/gcc.target/powerpc/divic3-1.c
> +/* { dg-final { scan-assembler "__divtc3" } } */

/* { dg-final { scan-assembler {\m__divtc3\M} } } */

It might well be that we can use a sloppier regexp here, but why would
we do that?  It is a good thing to use the \m and \M constraint escapes
pretty much always.

Similar for the other three testcases of course.

This patch is okay for trunk, if you have tested it on all
configurations (powerpc-linux, powerpc64-linux, powerpc64le-linux with
and without default IEEE128 long double at least).  Thank you!

Does this need backports?


Segher

Reply via email to