This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Fix profile updating in ifcombine
- From: Jiong Wang <jiong dot wang at foss dot arm dot com>
- To: Jan Hubicka <hubicka at ucw dot cz>, Christophe Lyon <christophe dot lyon at linaro dot org>
- Cc: Jeff Law <law at redhat dot com>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>, Richard Biener <rguenther at suse dot de>, jiong dot wang at arm dot com
- Date: Mon, 6 Feb 2017 15:54:46 +0000
- Subject: Re: Fix profile updating in ifcombine
- Authentication-results: sourceware.org; auth=none
- References: <20170202202923.GC91024@kam.mff.cuni.cz> <2f15e536-416a-42f5-8bce-28e4ec8d076f@redhat.com> <CAKdteOaNd8y-DtzPhH1VN3iL=g_sBD0Yw_0CB5W=SnjtJm+QOA@mail.gmail.com> <20170206152643.GA58190@kam.mff.cuni.cz>
On 06/02/17 15:26, Jan Hubicka wrote:
I think it is not a regression, just the testcase if fragile and depends on outcome
of ifcombine. It seems it was updated several time in the past. I am not quite
sure what the test is testing.
They are tring to make sure optimal stack adjustment decisions are made.
Fix the testcases by disabling relevant transformation passes looks one way to
me. The other way, might be more reliable, is we dump the decisions made during
aarch64 frame layout if dump_file be true, and prefix the dump entry by function
name to make it easier caught by dejagnu. We then scan rtl dump instead of
instructions.