summaryrefslogtreecommitdiff
path: root/gcc/ipa-structure-reorg.c
diff options
context:
space:
mode:
Diffstat (limited to 'gcc/ipa-structure-reorg.c')
-rw-r--r--gcc/ipa-structure-reorg.c18
1 files changed, 5 insertions, 13 deletions
diff --git a/gcc/ipa-structure-reorg.c b/gcc/ipa-structure-reorg.c
index ea0623168e9..d4f53f4521c 100644
--- a/gcc/ipa-structure-reorg.c
+++ b/gcc/ipa-structure-reorg.c
@@ -1,7 +1,7 @@
-/* Interprocedural scalar replacement of aggregates
+/* Interprocedural structure reorganization
Copyright (C) 2019-2020 Free Software Foundation, Inc.
- Contributed by Gary Oblock <goblock@marvell.com>
+ Contributed by Gary Oblock <gary@amperecomputing.com>
This file is part of GCC.
@@ -342,7 +342,7 @@ reorg_analysis ( Info *info)
//DEBUG_L( "possible deletes:\n");
//INDENT(2);
- // It's LOT clear to use an iterator here TBD
+ // It's LOT more clear to use an iterator here TBD
for ( int i = 0; i < info->reorg_type->size (); i++ )
{
int n = (*(info->reorg_type))[i].instance_interleave.numbOfGlobalArrays
@@ -435,8 +435,6 @@ static bool find_decls_and_types ( Info *info)
// NOTE, the scheme above leaves out local variables so
// I'll repeat the for the local variable of functions.
- // TBD Am I doing something actually, after this, that
- // is actually done here or above... it seems likely.
cgraph_node* node;
FOR_EACH_FUNCTION_WITH_GIMPLE_BODY ( node)
@@ -966,8 +964,6 @@ modify_declarations ( Info *info)
// Note, do not relayout functions decls....
}
-// TBD Reading this I don't see how it actually works
-// correctly
// Returns true if a modification occurred
bool
modify_decl_core ( tree *location, Info *info)
@@ -1158,8 +1154,6 @@ reorg_recognize ( gimple *stmt, cgraph_node* node, Info_t *info )
default:
return Not_Supported;
}
- // TBD Might a case for ReorgOpT_Temp
- // other places too
default:
//INDENT(-4);
return Not_Supported;
@@ -1466,7 +1460,7 @@ recognize_op ( tree op, Info *info)
if ( is_reorg_type ( base_type_of ( inner_type), info) )
{
//DEBUG_L(" returns: ReorgOpT_Indirect\n");
- return ReorgOpT_Indirect; // TBD
+ return ReorgOpT_Indirect;
}
// Just normal field reference otherwise...
//DEBUG_L(" returns: ReorgOpT_Scalar\n");
@@ -1691,7 +1685,6 @@ detect_reorg ( tree *tp, int *dummy, void *data)
{
struct walk_stmt_info *walk_data = ( struct walk_stmt_info *)data;
hidden_info_t *hi = ( hidden_info_t *)walk_data->info;
- //ReorgType_t *ri = get_reorgtype_info ( *tp, hi->info); // TBD I suspect this needs base_type_of( TREE_TYPE(*tp))
//DEBUG_L( "*tp = ");
//DEBUG_F( print_generic_expr, stderr, *tp, (dump_flags_t)-1);
//DEBUG("\n");
@@ -1946,7 +1939,7 @@ print_function ( FILE *file, int leading_space, struct function *func)
}
fprintf ( file, ")\n");
- // TBD Print Local Decls
+ // Print Local Decls
tree decl;
unsigned i;
FOR_EACH_LOCAL_DECL ( func, i, decl)
@@ -2223,7 +2216,6 @@ const pass_data pass_data_ipa_structure_reorg =
IPA_PASS, /* type */
"structure-reorg", /* name */
OPTGROUP_NONE, /* optinfo_flags */
- //TV_IPA_SRA, /* tv_id */ // TBD
TV_IPA_STRUCTURE_REORG, /* tv_id */
0, /* properties_required */
0, /* properties_provided */