On 06/02/17 15:54, Jiong Wang wrote: > 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. > >
We only care that the epilogue instructions appear at least once. So for the epilogue we should probably just adjust the tests to use scan-assembler, rather than scan-assembler-times. R.
