From 792a073a0217b190cb15ffbaeaf48d6d66e5d7c2 Mon Sep 17 00:00:00 2001 From: ctice Date: Mon, 16 May 2005 21:37:01 +0000 Subject: [PATCH] 2005-05-16 Caroline Tice * bb-reorder.c (verify_hot_cold_block_grouping): Replace internal_error with gcc_assert. * dwarf2out.c (dwarf2out_switch_text_section): Likewise. * function.h (struct function): Remove space between "*" and struct field names. * varasm.c (initialize_cold_section_name): Add gcc_assert; modify to use alloca. (unlikely_text_section): Add check for flag_function_sections. (assemble_start_function): Move call to resolve_section, so it comes before first call to unlikely_text_section. (assemble_end_function): Add test and call to function_section if appropriate. git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@99796 138bc75d-0d04-0410-961f-82ee72b054a4 --- gcc/ChangeLog | 15 +++++++++++++++ gcc/bb-reorder.c | 3 +-- gcc/dwarf2out.c | 3 +-- gcc/function.h | 8 ++++---- gcc/varasm.c | 53 ++++++++++++++++++++++++----------------------------- 5 files changed, 45 insertions(+), 37 deletions(-) diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 5605cc2..2ac24ad 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,18 @@ +2005-05-16 Caroline Tice + + * bb-reorder.c (verify_hot_cold_block_grouping): Replace + internal_error with gcc_assert. + * dwarf2out.c (dwarf2out_switch_text_section): Likewise. + * function.h (struct function): Remove space between "*" and + struct field names. + * varasm.c (initialize_cold_section_name): Add gcc_assert; + modify to use alloca. + (unlikely_text_section): Add check for flag_function_sections. + (assemble_start_function): Move call to resolve_section, so it + comes before first call to unlikely_text_section. + (assemble_end_function): Add test and call to function_section + if appropriate. + 2005-05-16 Roger Sayle * c-common.c (c_common_truthvalue_conversion): Use fold_build2 diff --git a/gcc/bb-reorder.c b/gcc/bb-reorder.c index cbb4932..5472967 100644 --- a/gcc/bb-reorder.c +++ b/gcc/bb-reorder.c @@ -1874,8 +1874,7 @@ verify_hot_cold_block_grouping (void) } } - if (err) - internal_error ("verify_hot_cold_block_grouping failed"); + gcc_assert(!err); } /* Reorder basic blocks. The main entry point to this file. FLAGS is diff --git a/gcc/dwarf2out.c b/gcc/dwarf2out.c index aee1c2f..32711c7 100644 --- a/gcc/dwarf2out.c +++ b/gcc/dwarf2out.c @@ -6814,8 +6814,7 @@ dwarf2out_switch_text_section (void) { dw_fde_ref fde; - if (!cfun) - internal_error ("Attempt to switch text sections without any code."); + gcc_assert (cfun); fde = &fde_table[fde_table_in_use - 1]; fde->dw_fde_switched_sections = true; diff --git a/gcc/function.h b/gcc/function.h index 0720c12..f062233 100644 --- a/gcc/function.h +++ b/gcc/function.h @@ -357,10 +357,10 @@ struct function GTY(()) be used by debugger functions for determining the size of text sections. */ - const char * hot_section_label; - const char * cold_section_label; - const char * hot_section_end_label; - const char * cold_section_end_label; + const char *hot_section_label; + const char *cold_section_label; + const char *hot_section_end_label; + const char *cold_section_end_label; /* String to be used for name of cold text sections, via targetm.asm_out.named_section. */ diff --git a/gcc/varasm.c b/gcc/varasm.c index bc0597e..1f51e2e 100644 --- a/gcc/varasm.c +++ b/gcc/varasm.c @@ -179,34 +179,23 @@ initialize_cold_section_name (void) const char *name; const char *stripped_name; char *buffer; - int len; - if (cfun - && current_function_decl) + gcc_assert (cfun && current_function_decl); + if (cfun->unlikely_text_section_name) + return; + + if (flag_function_sections && DECL_SECTION_NAME (current_function_decl)) { - if (!cfun->unlikely_text_section_name) - { - if (flag_function_sections - && DECL_SECTION_NAME (current_function_decl)) - { - name = xstrdup (TREE_STRING_POINTER (DECL_SECTION_NAME - (current_function_decl))); - stripped_name = targetm.strip_name_encoding (name); - len = strlen (stripped_name); - buffer = (char *) xmalloc (len + 10); - sprintf (buffer, "%s%s", stripped_name, "_unlikely"); - cfun->unlikely_text_section_name = ggc_strdup (buffer); - free (buffer); - free ((char *) name); - } - else - cfun->unlikely_text_section_name = - UNLIKELY_EXECUTED_TEXT_SECTION_NAME; - } + name = alloca (TREE_STRING_LENGTH (DECL_SECTION_NAME + (current_function_decl))); + strcpy ((char *) name, TREE_STRING_POINTER (DECL_SECTION_NAME + (current_function_decl))); + stripped_name = targetm.strip_name_encoding (name); + buffer = ACONCAT ((stripped_name, "_unlikely", NULL)); + cfun->unlikely_text_section_name = ggc_strdup (buffer); } else - internal_error - ("initialize_cold_section_name called without valid current_function_decl."); + cfun->unlikely_text_section_name = UNLIKELY_EXECUTED_TEXT_SECTION_NAME; } /* Tell assembler to switch to text section. */ @@ -232,9 +221,11 @@ unlikely_text_section (void) if (!cfun->unlikely_text_section_name) initialize_cold_section_name (); - if ((in_section != in_unlikely_executed_text) - && (in_section != in_named - || strcmp (in_named_name, cfun->unlikely_text_section_name) != 0)) + if (flag_function_sections + || ((in_section != in_unlikely_executed_text) + && (in_section != in_named + || (strcmp (in_named_name, cfun->unlikely_text_section_name) + != 0)))) { named_section (NULL_TREE, cfun->unlikely_text_section_name, 0); in_section = in_unlikely_executed_text; @@ -1267,6 +1258,8 @@ assemble_start_function (tree decl, const char *fnname) if (CONSTANT_POOL_BEFORE_FUNCTION) output_constant_pool (fnname, decl); + resolve_unique_section (decl, 0, flag_function_sections); + /* Make sure the not and cold text (code) sections are properly aligned. This is necessary here in the case where the function has both hot and cold sections, because we don't want to re-set @@ -1320,7 +1313,6 @@ assemble_start_function (tree decl, const char *fnname) } last_text_section = no_section; - resolve_unique_section (decl, 0, flag_function_sections); /* Switch to the correct text section for the start of the function. */ @@ -1406,7 +1398,10 @@ assemble_end_function (tree decl, const char *fnname) save_text_section = in_section; unlikely_text_section (); ASM_OUTPUT_LABEL (asm_out_file, cfun->cold_section_end_label); - text_section (); + if (first_function_block_is_cold) + text_section (); + else + function_section (decl); ASM_OUTPUT_LABEL (asm_out_file, cfun->hot_section_end_label); if (save_text_section == in_unlikely_executed_text) unlikely_text_section (); -- 2.7.4