Re: [PATCH, SMS] Minor misc. fixes

[Revital Eres <revital dot eres at linaro dot org>]

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 check its other invocation too.

OK, that's indeed seems reasonable.
The attached patch implements it.
Will commit it after re-testing completes if there is not objection.

Thanks,
Revital

Changelog:

       modulo-sched.c (remove_node_from_ps): Return void instead of bool.
        (optimize_sc): Adjust call to remove_node_from_ps.
        (sms_schedule): Add print info.

Attachment: patch_new_fix_12.txt
Description: Text document