On 11/09/2015 07:42 PM, Jeff Law wrote:
On 11/09/2015 11:27 AM, Bernd Schmidt wrote:
On 11/09/2015 05:47 PM, tbsaunde+...@tbsaunde.org wrote:
From: Trevor Saunders <tbsaunde+...@tbsaunde.org>

gcc/ChangeLog:

2015-11-09  Trevor Saunders  <tbsaunde+...@tbsaunde.org>

    * defaults.h (EH_RETURN_HANDLER_RTX): New default definition.
    * df-scan.c (df_get_exit_block_use_set): Adjust.
    * except.c (expand_eh_return): Likewise.

As I said for a previous patch series, if we go to the trouble of fixing
up stuff like this, we might as well do it properly and turn things like
this into a target hook.
I agree that pushing hookization further is good as well.  I still think
the patch in and of itself is a step forward, even if it doesn't hookize
EH_RETURN_HANDLER_RTX.

Well, I was hoping that, by pointing out the issue for the last patch set, the next set of patches would get things right. We really shouldn't make sideways steps when there's a simple way to go forward.


Bernd

Reply via email to