public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
From: "pluto at agmk dot net" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug target/30961] [4.1/4.2/4.3 regression] redundant reg/mem stores/moves
Date: Wed, 08 Aug 2007 21:50:00 -0000	[thread overview]
Message-ID: <20070808215009.23027.qmail@sourceware.org> (raw)
In-Reply-To: <bug-30961-7667@http.gcc.gnu.org/bugzilla/>



------- Comment #31 from pluto at agmk dot net  2007-08-08 21:50 -------
r84072 introduces an extra movq regression:

+2004-07-04  Richard Henderson  <rth@redhat.com>
+
+       * rtl.def (ADDRESSOF): Remove.
+       * alias.c (rtx_equal_for_memref_p): Don't handle addressof.
+       (find_base_term, memrefs_conflict_p): Likewise.
+       * cse.c (fixed_base_plus_p, find_best_addr: Likewise.
+       (fold_rtx, cse_insn, set_live_p): Likewise.
+       * dwarf2out.c (mem_loc_descriptor): Likewise.
+       (add_location_or_const_value_attribute): Likewise.
+       * emit-rtl.c (copy_insn_1): Likewise.
+       * explow.c (memory_address): Likewise.
+       * expmed.c (store_split_bit_field): Likewise.
+       * expr.c (expand_expr_real_1): Likewise.
+       * function.c (instantiate_decl
+       (instantiate_virtual_regs_1, fix_lexical_addr): Likewise.
+       * genrecog.c (preds, validate_pattern): Likewise.
+       * integrate.c (copy_rtx_and_substitute): Likewise.
+       * recog.c (general_operand, register_operand): Likewise.
+       (memory_address_p): Likwise.
+       * reload1.c (eliminate_regs, elimination_effects): Likewise.
+       * rtl.c (copy_rtx): Likewise.
+       * rtlanal.c (rtx_unstable_p, rtx_varies_p): Likewise.
+       (rtx_addr_can_trap_p, nonzero_address_p, address_cost): Likewise.
+       * rtlhooks.c (gen_lowpart_general): Likewise.
+       * stmt.c (expand_asm_operands): Likewise.
+       * web.c (entry_register, replace_ref, web_main): Likewise.
+       * config/alpha/alpha.c (input_operand, alpha_legitimate_address_p,
+       alpha_expand_block_move, alpha_expand_block_clear): Likewise.
+       * config/arm/arm.c (thumb_rtx_costs): Likewise.
+       * config/c4x/c4x.c (c4x_valid_operands): Likewise.
+       * config/frv/frv.c (move_destination_operand, move_source_operand,
+       condexec_dest_operand, condexec_source_operand,
+       condexec_memory_operand): Likewise.
+       * config/h8300/h8300.h (PREDICATE_CODES): Likewise.
+       * config/ia64/ia64.c (general_xfmode_operand): Likewise.
+       (destination_xfmode_operand): Likewise.
+       * config/mips/mips.h (PREDICATE_CODES): Likewise.
+       * config/mn10300/mn10300.c (mn10300_address_cost_1): Likewise.
+       * config/s390/s390.c (general_s_operand): Likewise.
+       * config/s390/s390.md (mov*): Likewise.
+       * config/sparc/sparc.h (PREDICATE_CODES): Likewise.
+
+       * c-typeck.c (c_mark_addressable): Don't put_var_into_stack.
+       * expr.c (expand_expr_real_1): Likewise.
+       * stmt.c (expand_decl): Likewise.
+       * config/c4x/c4x.c (c4x_expand_builtin): Likewise.
+
+       * function.c (struct fixup_replacement, struct insns_for_mem_entry,
+       postponed_insns, put_var_into_stack, put_reg_into_stack,
+       schedule_fixup_var_refs, fixup_var_refs, find_fixup_replacement,
+       fixup_var_refs_insns, fixup_var_refs_insns_with_hash,
+       fixup_var_refs_insn, fixup_var_refs_1, fixup_memory_subreg,
+       walk_fixup_memory_subreg, fixup_stack_1, optimize_bit_field,
+       gen_mem_addressof, flush_addressof, put_addressof_into_stack,
+       purge_bitfield_addressof_replacements, purge_addressof_replacements,
+       purge_addressof_1, insns_for_mem_hash, insns_for_mem_comp,
+       struct insns_for_mem_walk_info, insns_for_mem_walk,
+       compute_insns_for_mem, is_addressof, purge_addressof, setjmp_protect,
+       setjmp_protect_args): Remove.
+       (push_function_context_to): Don't handle var_refs_queue.
+       (pop_function_context_from, free_after_compilation): Likewise.
+       (instantiate_virtual_regs): Don't handle parm_reg_stack_loc.
+       (assign_parms, allocate_struct_function): Likewise.
+       (use_register_for_decl): New.
+       (expand_function_end): Don't setjmp_protect.
+       * function.h (struct emit_status): Update commentary.
+       (struct function): Remove x_max_parm_reg, x_parm_reg_stack_loc.
+       (max_parm_reg, parm_reg_stack_loc): Remove.
+       * passes.c (DFI_addressof): Remove.
+       (dump_file_info): Remove addressof.
+       (rest_of_handle_addressof): Remove.
+       (rest_of_compilation): Don't call it.
+       * rtl.h (ADDRESSOF_REGNO, ADDRESSOF_DECL): Remove.
+       * stmt.c (expand_decl): Use use_register_for_decl.
+       * tree.h: Update decls.
+       * web.c (mark_addressof): Remove.
+       * doc/invoke.texi (-dF): Remove.
+


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=30961


  parent reply	other threads:[~2007-08-08 21:50 UTC|newest]

Thread overview: 41+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-02-25 18:26 [Bug target/30961] New: " pluto at agmk dot net
2007-03-05 11:11 ` [Bug target/30961] [4.2 regression] " pluto at agmk dot net
2007-04-11 19:00 ` [Bug target/30961] [4.2/4.3 " pluto at agmk dot net
2007-04-11 21:40 ` steven at gcc dot gnu dot org
2007-05-11 17:22 ` steven at gcc dot gnu dot org
2007-05-11 17:26 ` steven at gcc dot gnu dot org
2007-05-11 17:28 ` steven at gcc dot gnu dot org
2007-05-11 20:42 ` steven at gcc dot gnu dot org
2007-05-12 17:24 ` hjl at lucon dot org
2007-05-12 17:47 ` steven at gcc dot gnu dot org
2007-05-12 18:03 ` hjl at lucon dot org
2007-05-12 21:27 ` hjl at lucon dot org
2007-05-13  5:36 ` hjl at lucon dot org
2007-05-13 19:08 ` hjl at lucon dot org
2007-06-30  9:11 ` [Bug target/30961] [4.1/4.2/4.3 " pinskia at gcc dot gnu dot org
2007-07-04  3:05 ` mmitchel at gcc dot gnu dot org
2007-07-31 18:29 ` pluto at agmk dot net
2007-07-31 19:36 ` hjl at lucon dot org
2007-08-01 11:30 ` [Bug target/30961] [4.1/4.2 " pluto at agmk dot net
2007-08-02 14:21 ` hjl at lucon dot org
2007-08-06 17:49 ` pluto at agmk dot net
2007-08-06 18:02 ` [Bug target/30961] [4.1/4.2/4.3 " hjl at lucon dot org
2007-08-06 18:28 ` pluto at agmk dot net
2007-08-06 21:39 ` pluto at agmk dot net
2007-08-07  0:22 ` hjl at lucon dot org
2007-08-07  7:22 ` pluto at agmk dot net
2007-08-07 12:57 ` hjl at lucon dot org
2007-08-07 17:52 ` pluto at agmk dot net
2007-08-07 17:53 ` pluto at agmk dot net
2007-08-07 18:11 ` hjl at lucon dot org
2007-08-07 21:20 ` pluto at agmk dot net
2007-08-08  0:33 ` hjl at lucon dot org
2007-08-08  3:01 ` hjl at lucon dot org
2007-08-08 21:50 ` pluto at agmk dot net [this message]
2007-08-27 14:50 ` jakub at gcc dot gnu dot org
2007-08-27 18:38 ` hjl at lucon dot org
2007-11-06 17:30 ` hjl at lucon dot org
2007-11-06 19:19 ` hjl at gcc dot gnu dot org
2008-07-04 22:00 ` [Bug target/30961] [4.2 " jsm28 at gcc dot gnu dot org
2009-02-13 16:00 ` hjl dot tools at gmail dot com
2009-03-30 21:40 ` jsm28 at gcc dot gnu dot org

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20070808215009.23027.qmail@sourceware.org \
    --to=gcc-bugzilla@gcc.gnu.org \
    --cc=gcc-bugs@gcc.gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).