[gcc r14-3858] Allow targets to check shrink-wrap-separate enabled or not

Kito Cheng kito@gcc.gnu.org
Tue Sep 12 01:57:41 GMT 2023


https://gcc.gnu.org/g:66d89a43a7b6bafca1d4675744808be53ef2736f

commit r14-3858-g66d89a43a7b6bafca1d4675744808be53ef2736f
Author: Fei Gao <gaofei@eswincomputing.com>
Date:   Wed Sep 6 09:39:08 2023 +0000

    Allow targets to check shrink-wrap-separate enabled or not
    
    No functional changes but restructure and expose use_shrink_wrapping_separate
    to the TARGETs.
    
    gcc/ChangeLog:
    
            * shrink-wrap.cc (try_shrink_wrapping_separate):call
            use_shrink_wrapping_separate.
            (use_shrink_wrapping_separate): wrap the condition
            check in use_shrink_wrapping_separate.
            * shrink-wrap.h (use_shrink_wrapping_separate): add to extern

Diff:
---
 gcc/shrink-wrap.cc | 22 +++++++++++++++-------
 gcc/shrink-wrap.h  |  1 +
 2 files changed, 16 insertions(+), 7 deletions(-)

diff --git a/gcc/shrink-wrap.cc b/gcc/shrink-wrap.cc
index b8d7b5571309..28301f04f89a 100644
--- a/gcc/shrink-wrap.cc
+++ b/gcc/shrink-wrap.cc
@@ -1776,16 +1776,13 @@ insert_prologue_epilogue_for_components (sbitmap components)
   commit_edge_insertions ();
 }
 
-/* The main entry point to this subpass.  FIRST_BB is where the prologue
-   would be normally put.  */
-void
-try_shrink_wrapping_separate (basic_block first_bb)
+bool
+use_shrink_wrapping_separate (void)
 {
-  if (!(SHRINK_WRAPPING_ENABLED
-	&& flag_shrink_wrap_separate
+  if (!(SHRINK_WRAPPING_ENABLED && flag_shrink_wrap_separate
 	&& optimize_function_for_speed_p (cfun)
 	&& targetm.shrink_wrap.get_separate_components))
-    return;
+    return false;
 
   /* We don't handle "strange" functions.  */
   if (cfun->calls_alloca
@@ -1794,6 +1791,17 @@ try_shrink_wrapping_separate (basic_block first_bb)
       || crtl->calls_eh_return
       || crtl->has_nonlocal_goto
       || crtl->saves_all_registers)
+    return false;
+
+  return true;
+}
+
+/* The main entry point to this subpass.  FIRST_BB is where the prologue
+   would be normally put.  */
+void
+try_shrink_wrapping_separate (basic_block first_bb)
+{
+  if (!use_shrink_wrapping_separate ())
     return;
 
   /* Ask the target what components there are.  If it returns NULL, don't
diff --git a/gcc/shrink-wrap.h b/gcc/shrink-wrap.h
index 161647711a3b..82386c2b7124 100644
--- a/gcc/shrink-wrap.h
+++ b/gcc/shrink-wrap.h
@@ -26,6 +26,7 @@ along with GCC; see the file COPYING3.  If not see
 extern bool requires_stack_frame_p (rtx_insn *, HARD_REG_SET, HARD_REG_SET);
 extern void try_shrink_wrapping (edge *entry_edge, rtx_insn *prologue_seq);
 extern void try_shrink_wrapping_separate (basic_block first_bb);
+extern bool use_shrink_wrapping_separate (void);
 #define SHRINK_WRAPPING_ENABLED \
   (flag_shrink_wrap && targetm.have_simple_return ())


More information about the Gcc-cvs mailing list