This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, i386] Enable x86_ext_80387_constants for m_K8, m_CORE2 and m_GENERIC64
- From: Roger Sayle <roger at eyesopen dot com>
- To: Uros Bizjak <ubizjak at gmail dot com>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Sun, 26 Nov 2006 14:57:29 -0700 (MST)
- Subject: Re: [PATCH, i386] Enable x86_ext_80387_constants for m_K8, m_CORE2 and m_GENERIC64
Hi Uros,
On Sun, 26 Nov 2006, Uros Bizjak wrote:
> 2006-11-25 Uros Bizjak <ubizjak@gmail.com>
>
> * config/i386/i386.c (x86_ext_80387_constants): Add m_K8, m_CORE2
> and m_GENERIC64.
Assuming that Jan and H.J. have no objections, this is OK for mainline.
For CPU tuning patches, I wonder if you could also post timing on some
trivial microbenchmark, when possible, to confirm that the published cycle
counts are representative and not dominated by other timing issues (decode
paths, pipeline stalls, memory latency, etc...) so that we can confirm at
least some piece of code will be faster on some CPU in practice.
Compiler tuning is often more of an art than a science :-)
Thanks,
Roger
--