This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Re: patch: subreg of mem when converting vector types


> 
> > Wouldn't it create more efficient code if instead simplify_subreg was
> > always called afterwards?  It would save a (perhaps unnecessary) copy
> > to a register.
> 
> i was just following the precedent in store_bit_field() which does
> exactly what my code does (copy to a reg).
It is not so long since i've implemented the simplify_subreg stuff.
In general I would love to see the compiler use it whenever possible instead
of doing even trivial transforamtions by hand.  It brings more flexibility
into the nasty subreg issues.

I've searched and fixed number of places, but some are still pending.
Perhaps, can you just fix the store_bit_field to use the simplify_subreg
too?

Honza
> 
> i think this is not a common case, so we could afford to just copy to a
> register.
> 
> it's your call.  imo, either way is fine.
> 
> lemme know so i can come up with a patch.
> 
> aldy


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]