This is the mail archive of the
gcc-bugs@gcc.gnu.org
mailing list for the GCC project.
[Bug rtl-optimization/69891] wrong code with -mstringop-strategy=libcall @ i686
- From: "ebotcazou at gcc dot gnu.org" <gcc-bugzilla at gcc dot gnu dot org>
- To: gcc-bugs at gcc dot gnu dot org
- Date: Fri, 26 Feb 2016 11:06:12 +0000
- Subject: [Bug rtl-optimization/69891] wrong code with -mstringop-strategy=libcall @ i686
- Auto-submitted: auto-generated
- References: <bug-69891-4 at http dot gcc dot gnu dot org/bugzilla/>
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69891
Eric Botcazou <ebotcazou at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Assignee|ebotcazou at gcc dot gnu.org |jakub at gcc dot gnu.org
--- Comment #10 from Eric Botcazou <ebotcazou at gcc dot gnu.org> ---
> I think the right fix is just:
> --- gcc/dse.c.jj 2016-01-19 13:32:12.000000000 +0100
> +++ gcc/dse.c 2016-02-26 11:03:36.694206088 +0100
> @@ -2556,6 +2556,8 @@ scan_insn (bb_info_t bb_info, rtx_insn *
> active_local_stores = insn_info;
> }
> }
> + else
> + clear_rhs_from_active_local_stores ();
> }
> }
> else if (SIBLING_CALL_P (insn) && reload_completed)
>
> memset, even if we can't figure out its arguments, is never a read of
> anything (except for the arguments, but those are already handled similarly
> to const calls). clear_rhs_from_active_local_stores () is the punt action
> in record_store, we store something, but we don't know where exactly.
Yes, this looks more precise than reset_active_stores indeed. Assign back if
you want me to do the testing.