Re: [PATCH, SMS] Minor misc. fixes

2011-09-12 Thread Revital Eres
Hello, > 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 c

Re: [PATCH, SMS] Minor misc. fixes

2011-09-11 Thread Ayal Zaks
Copying the lists.. ‎-- Forwarded message --‎ From: Ayal Zaks ‎‎ Date: 2011/9/11 Subject: Re: [PATCH, SMS] Minor misc. fixes To: Revital Eres ‫revital.e...@linaro.org‬ 2011/9/8 Revital Eres > > Hello, > > The attached patch contains minor fixes. > > Cur

[PATCH, SMS] Minor misc. fixes

2011-09-08 Thread Revital Eres
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? Thanks, Revital Changelog * modulo-sched.c (optimize_sc): Call remove_node_from_ps outside of gcc_ass