public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug target/99941] New: m_ALDERLAKE is missing from m_CORE_AVX2
@ 2021-04-06 16:19 hjl.tools at gmail dot com
  2021-04-07  2:23 ` [Bug target/99941] " crazylht at gmail dot com
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: hjl.tools at gmail dot com @ 2021-04-06 16:19 UTC (permalink / raw)
  To: gcc-bugs

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

            Bug ID: 99941
           Summary: m_ALDERLAKE is missing from m_CORE_AVX2
           Product: gcc
           Version: 11.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: target
          Assignee: unassigned at gcc dot gnu.org
          Reporter: hjl.tools at gmail dot com
                CC: crazylht at gmail dot com, lili.cui at intel dot com
  Target Milestone: ---
            Target: x86-64

i386-options.c has

#define m_ALDERLAKE (HOST_WIDE_INT_1U<<PROCESSOR_ALDERLAKE)
#define m_CORE_AVX512 (m_SKYLAKE_AVX512 | m_CANNONLAKE \
                       | m_ICELAKE_CLIENT | m_ICELAKE_SERVER | m_CASCADELAKE \
                       | m_TIGERLAKE | m_COOPERLAKE | m_SAPPHIRERAPIDS)
#define m_CORE_AVX2 (m_HASWELL | m_SKYLAKE | m_CORE_AVX512)

Should m_CORE_AVX2 include m_ALDERLAKE?

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [Bug target/99941] m_ALDERLAKE is missing from m_CORE_AVX2
  2021-04-06 16:19 [Bug target/99941] New: m_ALDERLAKE is missing from m_CORE_AVX2 hjl.tools at gmail dot com
@ 2021-04-07  2:23 ` crazylht at gmail dot com
  2021-04-07  3:17 ` crazylht at gmail dot com
  2021-07-31 13:09 ` hjl.tools at gmail dot com
  2 siblings, 0 replies; 4+ messages in thread
From: crazylht at gmail dot com @ 2021-04-07  2:23 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #1 from Hongtao.liu <crazylht at gmail dot com> ---
If we were more concerned about the performance of the big core, the answer
would be yes.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [Bug target/99941] m_ALDERLAKE is missing from m_CORE_AVX2
  2021-04-06 16:19 [Bug target/99941] New: m_ALDERLAKE is missing from m_CORE_AVX2 hjl.tools at gmail dot com
  2021-04-07  2:23 ` [Bug target/99941] " crazylht at gmail dot com
@ 2021-04-07  3:17 ` crazylht at gmail dot com
  2021-07-31 13:09 ` hjl.tools at gmail dot com
  2 siblings, 0 replies; 4+ messages in thread
From: crazylht at gmail dot com @ 2021-04-07  3:17 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #2 from Hongtao.liu <crazylht at gmail dot com> ---
(In reply to H.J. Lu from comment #0)
> i386-options.c has
> 
> #define m_ALDERLAKE (HOST_WIDE_INT_1U<<PROCESSOR_ALDERLAKE)
> #define m_CORE_AVX512 (m_SKYLAKE_AVX512 | m_CANNONLAKE \
>                        | m_ICELAKE_CLIENT | m_ICELAKE_SERVER | m_CASCADELAKE
> \
>                        | m_TIGERLAKE | m_COOPERLAKE | m_SAPPHIRERAPIDS)
> #define m_CORE_AVX2 (m_HASWELL | m_SKYLAKE | m_CORE_AVX512)
> 
> Should m_CORE_AVX2 include m_ALDERLAKE?

Let's adding m_ALDERLAKE to m_CORE_AVX2 first, if we want to do some balance
for small core and remove some tunes, we just need to mask off m_ALDERLAKE for
that specific tune.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [Bug target/99941] m_ALDERLAKE is missing from m_CORE_AVX2
  2021-04-06 16:19 [Bug target/99941] New: m_ALDERLAKE is missing from m_CORE_AVX2 hjl.tools at gmail dot com
  2021-04-07  2:23 ` [Bug target/99941] " crazylht at gmail dot com
  2021-04-07  3:17 ` crazylht at gmail dot com
@ 2021-07-31 13:09 ` hjl.tools at gmail dot com
  2 siblings, 0 replies; 4+ messages in thread
From: hjl.tools at gmail dot com @ 2021-07-31 13:09 UTC (permalink / raw)
  To: gcc-bugs

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

H.J. Lu <hjl.tools at gmail dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |11.0
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|---                         |FIXED

--- Comment #3 from H.J. Lu <hjl.tools at gmail dot com> ---
Fixed by r11-8125:

commit f2be08339b77d3495e210d6b5d9cea927f437720
Author: Cui,Lili <lili.cui@intel.com>
Date:   Mon Apr 12 09:59:25 2021 +0800

    Change march=alderlake ISA list and add m_ALDERLAKE to m_CORE_AVX2

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2021-07-31 13:09 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-04-06 16:19 [Bug target/99941] New: m_ALDERLAKE is missing from m_CORE_AVX2 hjl.tools at gmail dot com
2021-04-07  2:23 ` [Bug target/99941] " crazylht at gmail dot com
2021-04-07  3:17 ` crazylht at gmail dot com
2021-07-31 13:09 ` hjl.tools at gmail dot com

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).