On 12/09/2016 04:34 PM, Andre Vieira (lists) wrote:

Regardless, the other testcases I add in this patch show a sub-optimal
transformation done by postreload, turning direct calls into indirect
calls, for targets which have specifically pointed out that no CSE
should be done on functions through 'NO_FUNCTION_CSE'.

What I'm wondering about is whether the patch wouldn't also prevent the opposite transformation. Is there a reason not to do that one? Can the problem be modeled by tweaking costs?

Would you prefer I create a new PR for the problem this is actually
fixing and refile this PATCH under that PR?

Well, as long as you're working on fixing it I see no reason to clutter the bug database for the function cse issue, but do keep the existing PR open if there also ought to be register class changes.


Bernd

Reply via email to