On 11-05-23 5:45 AM, Bernd Schmidt wrote:
Ping: C6X scheduler changes.

http://gcc.gnu.org/ml/gcc-patches/2011-05/msg00747.html
I think it is ok. I spent a lot of time looking at the change. It looks ok to me although it is hard to be sure because the code is too complicated. So it is good thing that you are trying to simplify it.
http://gcc.gnu.org/ml/gcc-patches/2011-05/msg00749.html
Ok.
http://gcc.gnu.org/ml/gcc-patches/2011-05/msg00750.html
Ok.  But changelog entry for sched_change_pattern is absent.
http://gcc.gnu.org/ml/gcc-patches/2011-05/msg00753.html
It is a really big patch.  I need more time to look at it.
http://gcc.gnu.org/ml/gcc-patches/2011-05/msg00755.html

Ok. Only changelog entry for deps_analyze_insn is missed.

So I need to finish review of the 4th patch (I hope to finish it tomorrow). All other patches is ok.
Thanks for the patches, Bernd.

Reply via email to