On 11/12/14 09:23, Kyrill Tkachov wrote:
Hi all,
I noticed that the check for modified_in_p in sched-deps is not needed
and needlessly restricts the insn pairs that we can check for macro
fusion in the backend hooks.
This patch removes the check. Currently that execution path is only used
in arm and aarch64.
This enables the backend hooks to catch more cases that I'm working on.
Bootstrapped on aarch64, x86, arm. Tested on aarch64.
Ok for trunk?
Thanks,
Kyrill
2014-11-12 Kyrylo Tkachov <kyrylo.tkac...@arm.com>
* sched-deps.c (sched_macro_fuse_insns): Do not check modified_in_p
in the not condtional jump case.
Typo in ChangeLog "condtional"
Can you please include a testcase which shows fusion occurring after
this patch where it does not occur before this patch.
I think you probably need to include some kind of doc change around this
change since you're effectively pushing responsibility for checking this
into the ports.
With the testcase and doc change, this will probably be OK. Please take
care of those, repost for a review of the test & doc changes.
jeff
