Copying the lists.. ---------- Forwarded message ---------- From: Ayal Zaks <ayal.z...@gmail.com> Date: 2011/9/11 Subject: Re: [PATCH, SMS] Minor misc. fixes To: Revital Eres revital.e...@linaro.org
2011/9/8 Revital Eres <revital.e...@linaro.org> > > Hello, > > The attached patch contains minor fixes. > > Currently testing and bootstrap on ppc64-redhat-linux enabling SMS on > loops with SC 1. > > OK for mainline once testing completes? OK. While we're at it, an alternative would be to have remove_node_from_ps() assert its own (parameters and) return value. That is, replace "if (c) return false" by "assert (!c)" and have it return void if successful. There's not much you can do if it returns false. That would check its other invocation too. Ayal. > > Thanks, > Revital > > > Changelog > > * modulo-sched.c (optimize_sc): Call remove_node_from_ps outside > of gcc_assert. > (sms_schedule): Add print info.
