On Fri, 2009-03-20 at 00:24 +0100, Steven Bosscher wrote:
> On Sun, Mar 15, 2009 at 9:39 PM, Gerald Pfeifer <ger...@pfeifer.com> wrote:
> >> I can't find any test results in gcc-testresults reported with
> >> -mtune=itanium1 [1].
> >
> > ...especially if theye do not even contribute test results or
> > feedback when things are broken (as in this case). Deprecating
> > Itanium 1 with GCC 4.4 sounds very reasonable.
>
> Very well. Like so?
> I'll propose something for gcc-4.4/changes.html too, after this is
> approved in some form.
>
> Ciao!
> Steven
Sorry for not chiming in sooner but I have been on Vacation. I think
depreciating Itanium1 tuning for 4.4 and removing it in 4.5 is
reasonable. Code generated and tuned for Itanium2 should run fine on
Itanium1 (Merced). It won't be scheduled optimally of course, but it
should run correctly.
>
>
>
> * config/ia64/ia64.c (ia64_handle_option): Inform user that Itanium1
> support is deprecated if the -mtune value is set to an Itanium1
> variant.
>
> Index: config/ia64/ia64.c
> ===================================================================
> --- config/ia64/ia64.c (revision 144970)
> +++ config/ia64/ia64.c (working copy)
> @@ -5212,6 +5212,8 @@ fix_range (const char *const_str)
> static bool
> ia64_handle_option (size_t code, const char *arg, int value)
> {
> + static bool warned_merced_deprecated = false;
> +
> switch (code)
> {
> case OPT_mfixed_range_:
> @@ -5245,6 +5247,13 @@ ia64_handle_option (size_t code, const char *arg,
> if (!strcmp (arg, processor_alias_table[i].name))
> {
> ia64_tune = processor_alias_table[i].processor;
> + if (ia64_tune == PROCESSOR_ITANIUM
> + && ! warned_merced_deprecated)
> + {
> + inform ("value %<%s%> for -mtune= switch is deprecated", arg);
> + inform ("GCC 4.4 is the last release with Itanium1 support");
> + warned_merced_deprecated = true;
> + }
> break;
> }
> if (i == pta_size)
I will approve this patch, but it should say "Itanium1 tuning support"
or something like that. The code will run on Itanium1, just not
optimally.
Steve Ellcey
s...@cup.hp.com
