https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65862
Wilco <wilco at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |wilco at gcc dot gnu.org
--- Comment #16 from Wilco <wilco at gcc dot gnu.org> ---
(In reply to niva from comment #15)
> (In reply to Vladimir Makarov from comment #14)
> > Author: vmakarov
> > Date: Thu May 14 20:40:44 2015
> > New Revision: 223202
> >
> > URL: https://gcc.gnu.org/viewcvs?rev=223202&root=gcc&view=rev
> > Log:
> > 2015-05-14 Vladimir Makarov <vmaka...@redhat.com>
> >
> > PR rtl-optimization/65862
> > * target.def (ira_change_pseudo_allocno_class): New hook.
> > * targhooks.c (default_ira_change_pseudo_allocno_class): Default
> > value of the hook.
> > * targhooks.h (default_ira_change_pseudo_allocno_class): New
> > extern
> > * doc/tm.texi.in (TARGET_IRA_CHANGE_PSEUDO_ALLOCNO_CLASS): Add the
> > hook.
> > * ira-costs.c (find_costs_and_classes): Call the hook and change
> > classes when it is necessary.
> > * doc/tm.texi: Update.
> >
> >
> > Modified:
> > trunk/gcc/ChangeLog
> > trunk/gcc/doc/tm.texi
> > trunk/gcc/doc/tm.texi.in
> > trunk/gcc/ira-costs.c
> > trunk/gcc/target.def
> > trunk/gcc/targhooks.c
> > trunk/gcc/targhooks.h
>
> :
> Is it true that this patch fixes the problem?
> If so then why this bug is not marked as resolved?
> (I'm asking because I am greatly interested in this fix.)
If you implement the hook like I did on AArch64 then the problem is worked
around reasonably. Note there were several follow-on patches, and the issue
still exists on almost all targets. So in that sense it is not fixed until it
works better by default (I don't mind if this bug gets closed but we will need
another to keep track of this issue until GCC does the right thing by default).
