Patchwork PATCH: PR target/46768: [4.6 Regression] FAIL: gcc.target/i386/pr37434-[24].c

login
register
mail settings
Submitter H.J. Lu
Date Dec. 2, 2010, 6:11 p.m.
Message ID <AANLkTimk=W2GO0XRK_OWB5jxGqHsNWjT-Hb7CmVWjg9O@mail.gmail.com>
Download mbox | patch
Permalink /patch/74009/
State New
Headers show

Comments

H.J. Lu - Dec. 2, 2010, 6:11 p.m.
On Thu, Dec 2, 2010 at 10:05 AM, H.J. Lu <hjl.tools@gmail.com> wrote:
> On Thu, Dec 2, 2010 at 7:50 AM, Maxim Kuvyrkov <maxim@codesourcery.com> wrote:
>> On Dec 2, 2010, at 6:42 PM, H.J. Lu wrote:
>>>
>>> Here is the first one.  OK for trunk?
>>>
>>> Thanks.
>>>
>>>
>>> --
>>> H.J.
>>> --
>>> 2010-12-02  H.J. Lu  <hongjiu.lu@intel.com>
>>>
>>>       * config/i386/i386.c (ix86_option_override_internal): Properly
>>>       schedule 32bit Core 2 and Core i7.
>>
>> H.J.,
>>
>> Removal of the scheduling override is included in the 0006-Core-2-i7-DFA patch I posted and, now, checked in.
>>
>> Thanks,
>>
>
> A regression:
>
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46768
>

Here is a patch.  OK for trunk?

Thanks.
Uros Bizjak - Dec. 2, 2010, 10:46 p.m.
On Thu, Dec 2, 2010 at 7:11 PM, H.J. Lu <hjl.tools@gmail.com> wrote:

>>> Removal of the scheduling override is included in the 0006-Core-2-i7-DFA patch I posted and, now, checked in.
>>>
>>> Thanks,
>>>
>>
>> A regression:
>>
>> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46768
>>
>
> Here is a patch.  OK for trunk?
>
> Thanks.
>
>
> --
> H.J.
> ---
> 2010-12-02  H.J. Lu  <hongjiu.lu@intel.com>
>
>        PR target/46768
>        * config/i386/i386.c (initial_ix86_tune_features): Turn on
>        X86_TUNE_INTER_UNIT_MOVES for Core 2 and Core i7.

OK under trivial rule.

Thanks,
Uros.

Patch

diff --git a/gcc/config/i386/i386.c b/gcc/config/i386/i386.c
index f5fc2b6..fafa299 100644
--- a/gcc/config/i386/i386.c
+++ b/gcc/config/i386/i386.c
@@ -1846,7 +1846,7 @@  static unsigned int initial_ix86_tune_features[X86_TUNE_LAST] = {
   m_AMD_MULTIPLE,
 
   /* X86_TUNE_INTER_UNIT_MOVES */
-  ~(m_AMD_MULTIPLE | m_CORE2I7 | m_GENERIC),
+  ~(m_AMD_MULTIPLE | m_GENERIC),
 
   /* X86_TUNE_INTER_UNIT_CONVERSIONS */
   ~(m_AMDFAM10 | m_BDVER1),