| Message ID | CAFULd4aCe6sPUOeUbmypN9B1RYy4FqOnMTrWNoO5oqRghuy79A@mail.gmail.com |
|---|---|
| State | New |
| Headers | show |
On Mon, Jan 27, 2014 at 10:15 AM, Uros Bizjak <ubizjak@gmail.com> wrote: > +2013-12-29 Allan Sandfeld Jensen <sandfeld@kde.org> > > Missing space in ChangeLog entry. > > + * config/i386/i386.c (get_builtin_code_for_version): Separate > + Westmere from Nehalem, Ivy Bridge from Sandy Bridge and > + Broadwell from Haswell. > > --- a/gcc/config/i386/i386.c > +++ b/gcc/config/i386/i386.c > @@ -31298,18 +31298,27 @@ get_builtin_code_for_version (tree decl, > tree *predicate_list) > priority = P_PROC_SSSE3; > break; > case PROCESSOR_NEHALEM: > - /* We translate "arch=corei7" and "arch=nehelam" to > - "corei7" so that it will be mapped to M_INTEL_COREI7 > - as cpu type to cover all M_INTEL_COREI7_XXXs. */ > - arg_str = "corei7"; > + if (new_target->x_ix86_isa_flags & OPTION_MASK_ISA_AES) > + arg_str = "westmere"; > + else > + /* We translate "arch=corei7" and "arch=nehelam" to > > Trivial typo above: "arch=nehalem". > > OK for mainline with these changes. I have committed slightly reformated patches with following ChangeLog to mainline SVN. 2014-01-27 Allan Sandfeld Jensen <sandfeld@kde.org> * config/i386/i386.c (get_builtin_code_for_version): Separate Westmere from Nehalem, Ivy Bridge from Sandy Bridge and Broadwell from Haswell. testsuite/ChangeLog: 2014-01-27 Allan Sandfeld Jensen <sandfeld@kde.org> * g++.dg/ext/mv16.C: New tests. Uros.
--- a/gcc/config/i386/i386.c +++ b/gcc/config/i386/i386.c @@ -31298,18 +31298,27 @@ get_builtin_code_for_version (tree decl, tree *predicate_list) priority = P_PROC_SSSE3; break; case PROCESSOR_NEHALEM: - /* We translate "arch=corei7" and "arch=nehelam" to - "corei7" so that it will be mapped to M_INTEL_COREI7 - as cpu type to cover all M_INTEL_COREI7_XXXs. */ - arg_str = "corei7"; + if (new_target->x_ix86_isa_flags & OPTION_MASK_ISA_AES) + arg_str = "westmere"; + else + /* We translate "arch=corei7" and "arch=nehelam" to Trivial typo above: "arch=nehalem". OK for mainline with these changes. Thanks, Uros.