Re: [PATCH] Remove unused DELAY_SLOTS_FOR_EPILOGUE target macro

2012-11-26 Thread Jeff Law

On 11/24/2012 03:08 PM, Steven Bosscher wrote:

Hello,

The DELAY_SLOTS_FOR_EPILOGUE target macro, and the related
ELIGIBLE_FOR_EPILOGUE_DELAY macro, are unused. The attached patch
removes them.

OK for trunk?
OK, though this arguably shouldn't be going in during stage3. But given 
those are totally unused, OK.


jeff


Re: [PATCH] Remove unused DELAY_SLOTS_FOR_EPILOGUE target macro

2012-11-24 Thread Hans-Peter Nilsson
On Sat, 24 Nov 2012, Steven Bosscher wrote:
 Hello,

 The DELAY_SLOTS_FOR_EPILOGUE target macro, and the related
 ELIGIBLE_FOR_EPILOGUE_DELAY macro, are unused. The attached patch
 removes them.

 OK for trunk?

Obviously; Jeff Law preapproved their removal long ago.
(I forgot about it.)  But please poison them too!

brgds, H-P


Re: [PATCH] Remove unused DELAY_SLOTS_FOR_EPILOGUE target macro

2012-11-24 Thread Steven Bosscher
On Sun, Nov 25, 2012 at 12:54 AM, Hans-Peter Nilsson wrote:
 On Sat, 24 Nov 2012, Steven Bosscher wrote:
 Hello,

 The DELAY_SLOTS_FOR_EPILOGUE target macro, and the related
 ELIGIBLE_FOR_EPILOGUE_DELAY macro, are unused. The attached patch
 removes them.

 OK for trunk?

 Obviously; Jeff Law preapproved their removal long ago.
 (I forgot about it.)  But please poison them too!

Right, committed with this additional hunk:

Index: system.h
===
--- system.h(revision 193766)
+++ system.h(working copy)
@@ -902,7 +902,8 @@ extern void fancy_abort (const char *, i
UNALIGNED_LONG_ASM_OP UNALIGNED_DOUBLE_INT_ASM_OP  \
USE_COMMON_FOR_ONE_ONLY IFCVT_EXTRA_FIELDS IFCVT_INIT_EXTRA_FIELDS \
CASE_USE_BIT_TESTS FIXUNS_TRUNC_LIKE_FIX_TRUNC \
-GO_IF_MODE_DEPENDENT_ADDRESS
+GO_IF_MODE_DEPENDENT_ADDRESS DELAY_SLOTS_FOR_EPILOGUE  \
+ELIGIBLE_FOR_EPILOGUE_DELAY

 /* Hooks that are no longer used.  */
  #pragma GCC poison LANG_HOOKS_FUNCTION_MARK LANG_HOOKS_FUNCTION_FREE  \


Ciao!
Steven