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