public inbox for gcc-cvs@sourceware.org
help / color / mirror / Atom feed
* [gcc r14-8466] lower-bitint: Add debugging dump of SSA_NAME -> decl mappings
@ 2024-01-27 12:07 Jakub Jelinek
  0 siblings, 0 replies; only message in thread
From: Jakub Jelinek @ 2024-01-27 12:07 UTC (permalink / raw)
  To: gcc-cvs

https://gcc.gnu.org/g:675e522903825d41230fcbcafaf7d640205179f4

commit r14-8466-g675e522903825d41230fcbcafaf7d640205179f4
Author: Jakub Jelinek <jakub@redhat.com>
Date:   Sat Jan 27 13:05:30 2024 +0100

    lower-bitint: Add debugging dump of SSA_NAME -> decl mappings
    
    While the SSA coalescing performed by lower bitint prints some information
    if -fdump-tree-bitintlower-details, it is really hard to read and doesn't
    contain the most important information which one looks for when debugging
    bitint lowering issues, namely what VAR_DECLs (or PARM_DECLs/RESULT_DECLs)
    each SSA_NAME in large_huge.m_names bitmap maps to.
    
    So, the following patch adds dumping of that, so that we know that say
    _3 -> bitint.3
    _8 -> bitint.7
    _16 -> bitint.7
    etc.
    
    2024-01-27  Jakub Jelinek  <jakub@redhat.com>
    
            * gimple-lower-bitint.cc (gimple_lower_bitint): For
            TDF_DETAILS dump mapping of SSA_NAMEs to decls.

Diff:
---
 gcc/gimple-lower-bitint.cc | 31 +++++++++++++++++++++----------
 1 file changed, 21 insertions(+), 10 deletions(-)

diff --git a/gcc/gimple-lower-bitint.cc b/gcc/gimple-lower-bitint.cc
index 298f16ef8da1..79e3b55a55b5 100644
--- a/gcc/gimple-lower-bitint.cc
+++ b/gcc/gimple-lower-bitint.cc
@@ -6344,22 +6344,33 @@ gimple_lower_bitint (void)
 	      }
 	  }
       tree atype = NULL_TREE;
+      if (dump_file && (dump_flags & TDF_DETAILS))
+	fprintf (dump_file, "Mapping SSA_NAMEs to decls:\n");
       EXECUTE_IF_SET_IN_BITMAP (large_huge.m_names, 0, i, bi)
 	{
 	  tree s = ssa_name (i);
 	  int p = var_to_partition (large_huge.m_map, s);
-	  if (large_huge.m_vars[p] != NULL_TREE)
-	    continue;
-	  if (atype == NULL_TREE
-	      || !tree_int_cst_equal (TYPE_SIZE (atype),
-				      TYPE_SIZE (TREE_TYPE (s))))
+	  if (large_huge.m_vars[p] == NULL_TREE)
+	    {
+	      if (atype == NULL_TREE
+		  || !tree_int_cst_equal (TYPE_SIZE (atype),
+					  TYPE_SIZE (TREE_TYPE (s))))
+		{
+		  unsigned HOST_WIDE_INT nelts
+		    = tree_to_uhwi (TYPE_SIZE (TREE_TYPE (s))) / limb_prec;
+		  atype = build_array_type_nelts (large_huge.m_limb_type,
+						  nelts);
+		}
+	      large_huge.m_vars[p] = create_tmp_var (atype, "bitint");
+	      mark_addressable (large_huge.m_vars[p]);
+	    }
+	  if (dump_file && (dump_flags & TDF_DETAILS))
 	    {
-	      unsigned HOST_WIDE_INT nelts
-		= tree_to_uhwi (TYPE_SIZE (TREE_TYPE (s))) / limb_prec;
-	      atype = build_array_type_nelts (large_huge.m_limb_type, nelts);
+	      print_generic_expr (dump_file, s, TDF_SLIM);
+	      fprintf (dump_file, " -> ");
+	      print_generic_expr (dump_file, large_huge.m_vars[p], TDF_SLIM);
+	      fprintf (dump_file, "\n");
 	    }
-	  large_huge.m_vars[p] = create_tmp_var (atype, "bitint");
-	  mark_addressable (large_huge.m_vars[p]);
 	}
     }

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2024-01-27 12:07 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-01-27 12:07 [gcc r14-8466] lower-bitint: Add debugging dump of SSA_NAME -> decl mappings Jakub Jelinek

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).