[Bug target/105339] New: [x86] missing AVX-512F scalef functions when optimization is disabled

[Bug target/105339] New: [x86] missing AVX-512F scalef functions when optimization is disabled

[evan@coeus-group.com]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=105339

            Bug ID: 105339
           Summary: [x86] missing AVX-512F scalef functions when
                    optimization is disabled
           Product: gcc
           Version: 12.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: target
          Assignee: unassigned at gcc dot gnu.org
          Reporter: evan@coeus-group.com
  Target Milestone: ---

Several AVX-512F functions scalef functions are only declared when __OPTIMIZE__
is defined:

 * _mm_maskz_scalef_ss
 * _mm_mask_scalef_sd
 * _mm_maskz_scalef_sd

There may be others; I haven't done an exhaustive check.

[Bug target/105339] [x86] missing AVX-512F scalef functions when optimization is disabled

[crazylht at gmail dot com]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=105339

Hongtao.liu <crazylht at gmail dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |crazylht at gmail dot com

--- Comment #1 from Hongtao.liu <crazylht at gmail dot com> ---
Hmm

 3316#define _mm_mask_scalef_sd(W, U, A, B) \
 3317    _mm_mask_scalef_round_sd ((W), (U), (A), (B),
_MM_FROUND_CUR_DIRECTION)
 3318
 3319#define _mm_maskz_scalef_sd(U, A, B) \
 3320    _mm_maskz_scalef_round_sd ((U), (A), (B), _MM_FROUND_CUR_DIRECTION)
 3321
 3322#define _mm_mask_scalef_ss(W, U, A, B) \
 3323    _mm_mask_scalef_round_ss ((W), (U), (A), (B),
_MM_FROUND_CUR_DIRECTION)
 3324
 3325#define _mm_maskz_scalef_ss(U, A, B) \
 3326    _mm_maskz_scalef_round_ss ((U), (A), (B), _MM_FROUND_CUR_DIRECTION)

they're defined as macros, but _mm_maskz_scalef_round_ss is only available
under __OPTIMIZE__.

[Bug target/105339] [x86] missing AVX-512F scalef functions when optimization is disabled

[crazylht at gmail dot com]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=105339

--- Comment #2 from Hongtao.liu <crazylht at gmail dot com> ---
We need to add macro for _mm_{mask,maskz}_scalef_round_{sd,ss} intriniscs for
gcc-9/10/11/12

[Bug target/105339] [x86] missing AVX-512F scalef functions when optimization is disabled

[cvs-commit at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=105339

--- Comment #3 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Hongyu Wang <hongyuw@gcc.gnu.org>:

https://gcc.gnu.org/g:3c940d42701707559fabe49be99296f60fbc43e7

commit r12-8238-g3c940d42701707559fabe49be99296f60fbc43e7
Author: Hongyu Wang <hongyu.wang@intel.com>
Date:   Fri Apr 22 14:42:30 2022 +0800

    AVX512F: Add missing macro for mask(z?)_scalf_s[sd] [PR 105339]

    Add missing macro under O0 and adjust macro format for scalf
    intrinsics.

    gcc/ChangeLog:

            PR target/105339
            * config/i386/avx512fintrin.h (_mm512_scalef_round_pd):
            Add parentheses for parameters and djust format.
            (_mm512_mask_scalef_round_pd): Ditto.
            (_mm512_maskz_scalef_round_pd): Ditto.
            (_mm512_scalef_round_ps): Ditto.
            (_mm512_mask_scalef_round_ps): Ditto.
            (_mm512_maskz_scalef_round_ps): Ditto.
            (_mm_scalef_round_sd): Use _mm_undefined_pd.
            (_mm_scalef_round_ss): Use _mm_undefined_ps.
            (_mm_mask_scalef_round_sd): New macro.
            (_mm_mask_scalef_round_ss): Ditto.
            (_mm_maskz_scalef_round_sd): Ditto.
            (_mm_maskz_scalef_round_ss): Ditto.

    gcc/testsuite/ChangeLog:

            PR target/105339
            * gcc.target/i386/sse-14.c: Add tests for new macro.

[Bug target/105339] [x86] missing AVX-512F scalef functions when optimization is disabled

[cvs-commit at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=105339

--- Comment #4 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-11 branch has been updated by Hongyu Wang
<hongyuw@gcc.gnu.org>:

https://gcc.gnu.org/g:8a02dab2aac640186acca8b265a790153632786d

commit r11-9940-g8a02dab2aac640186acca8b265a790153632786d
Author: Hongyu Wang <hongyu.wang@intel.com>
Date:   Fri Apr 22 14:42:30 2022 +0800

    AVX512F: Add missing macro for mask(z?)_scalf_s[sd] [PR 105339]

    Add missing macro under O0 and adjust macro format for scalf
    intrinsics.

    gcc/ChangeLog:

            PR target/105339
            * config/i386/avx512fintrin.h (_mm512_scalef_round_pd):
            Add parentheses for parameters and djust format.
            (_mm512_mask_scalef_round_pd): Ditto.
            (_mm512_maskz_scalef_round_pd): Ditto.
            (_mm512_scalef_round_ps): Ditto.
            (_mm512_mask_scalef_round_ps): Ditto.
            (_mm512_maskz_scalef_round_ps): Ditto.
            (_mm_scalef_round_sd): Use _mm_undefined_pd.
            (_mm_scalef_round_ss): Use _mm_undefined_ps.
            (_mm_mask_scalef_round_sd): New macro.
            (_mm_mask_scalef_round_ss): Ditto.
            (_mm_maskz_scalef_round_sd): Ditto.
            (_mm_maskz_scalef_round_ss): Ditto.

    gcc/testsuite/ChangeLog:

            PR target/105339
            * gcc.target/i386/sse-14.c: Add tests for new macro.

    (cherry picked from commit 3c940d42701707559fabe49be99296f60fbc43e7)

[Bug target/105339] [x86] missing AVX-512F scalef functions when optimization is disabled

[cvs-commit at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=105339

--- Comment #5 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-10 branch has been updated by Hongyu Wang
<hongyuw@gcc.gnu.org>:

https://gcc.gnu.org/g:5e53c8a04d182ead8b59682802478de3dfc0361b

commit r10-10584-g5e53c8a04d182ead8b59682802478de3dfc0361b
Author: Hongyu Wang <hongyu.wang@intel.com>
Date:   Fri Apr 22 14:42:30 2022 +0800

    AVX512F: Add missing macro for mask(z?)_scalf_s[sd] [PR 105339]

    Add missing macro under O0 and adjust macro format for scalf
    intrinsics.

    gcc/ChangeLog:

            PR target/105339
            * config/i386/avx512fintrin.h (_mm512_scalef_round_pd):
            Add parentheses for parameters and djust format.
            (_mm512_mask_scalef_round_pd): Ditto.
            (_mm512_maskz_scalef_round_pd): Ditto.
            (_mm512_scalef_round_ps): Ditto.
            (_mm512_mask_scalef_round_ps): Ditto.
            (_mm512_maskz_scalef_round_ps): Ditto.
            (_mm_scalef_round_sd): Use _mm_undefined_pd.
            (_mm_scalef_round_ss): Use _mm_undefined_ps.
            (_mm_mask_scalef_round_sd): New macro.
            (_mm_mask_scalef_round_ss): Ditto.
            (_mm_maskz_scalef_round_sd): Ditto.
            (_mm_maskz_scalef_round_ss): Ditto.

    gcc/testsuite/ChangeLog:

            PR target/105339
            * gcc.target/i386/sse-14.c: Add tests for new macro.

    (cherry picked from commit 3c940d42701707559fabe49be99296f60fbc43e7)

[Bug target/105339] [x86] missing AVX-512F scalef functions when optimization is disabled

[cvs-commit at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=105339

--- Comment #6 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-9 branch has been updated by Hongyu Wang
<hongyuw@gcc.gnu.org>:

https://gcc.gnu.org/g:18bee83150e235ebd596443a0909a72d0c4d077f

commit r9-10031-g18bee83150e235ebd596443a0909a72d0c4d077f
Author: Hongyu Wang <hongyu.wang@intel.com>
Date:   Fri Apr 22 14:42:30 2022 +0800

    AVX512F: Add missing macro for mask(z?)_scalf_s[sd] [PR 105339]

    Add missing macro under O0 and adjust macro format for scalf
    intrinsics.

    gcc/ChangeLog:

            PR target/105339
            * config/i386/avx512fintrin.h (_mm512_scalef_round_pd):
            Add parentheses for parameters and djust format.
            (_mm512_mask_scalef_round_pd): Ditto.
            (_mm512_maskz_scalef_round_pd): Ditto.
            (_mm512_scalef_round_ps): Ditto.
            (_mm512_mask_scalef_round_ps): Ditto.
            (_mm512_maskz_scalef_round_ps): Ditto.
            (_mm_scalef_round_sd): Use _mm_undefined_pd.
            (_mm_scalef_round_ss): Use _mm_undefined_ps.
            (_mm_mask_scalef_round_sd): New macro.
            (_mm_mask_scalef_round_ss): Ditto.
            (_mm_maskz_scalef_round_sd): Ditto.
            (_mm_maskz_scalef_round_ss): Ditto.

    gcc/testsuite/ChangeLog:

            PR target/105339
            * gcc.target/i386/sse-14.c: Add tests for new macro.

    (cherry picked from commit 3c940d42701707559fabe49be99296f60fbc43e7)

[Bug target/105339] [x86] missing AVX-512F scalef functions when optimization is disabled

[wwwhhhyyy333 at gmail dot com]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=105339

--- Comment #7 from Hongyu Wang <wwwhhhyyy333 at gmail dot com> ---
Fixed for gcc-9/10/11/12.

[Bug target/105339] [x86] missing AVX-512F scalef functions when optimization is disabled

[pinskia at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=105339

Andrew Pinski <pinskia at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
      Known to work|                            |10.4.0, 11.3.1, 12.1.0,
                   |                            |9.5.0
         Resolution|---                         |FIXED
   Target Milestone|---                         |9.5
      Known to fail|                            |11.3.0

--- Comment #8 from Andrew Pinski <pinskia at gcc dot gnu.org> ---
.