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]
Other format: [Raw text]

Re: [PATCH RFC]Pair load store instructions using a generic scheduling fusion pass


On 11/05/14 02:30, Bin.Cheng wrote:
[ ... ]

rfs_result's signature has changed, I think you need to pass in tmp & tmp2.
You'll need to make that trivial update for all the callers.


Can you make those changes and repost so that I can look at the docs (I
think the implementation is fine and won't need further review).

jeff

Hi,

Thanks very much for reviewing.  I refined the patch according to your
comments.  Also made two small changes: a)  skip breaking dependency
between memory access and the corresponding base-reg modifying
instruction.  This feature doesn't help load/store pair that much and
only increases compilation time.  b) a minor bug fix in arm backend
hook when calculating priority for memory accesses with minus offset.

I am running bootstrap/test against latest trunk, and will adapt
ChangeLog once get approved generally.  So how about this one?
OK for the trunk.  Thanks for your patience.

Jeff


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