On 10/3/19 1:46 PM, Jan Hubicka wrote: >> >> >>> Tests that now fail, but worked before (15 tests): >>> >>> g++.dg/tree-ssa/pr61034.C -std=gnu++14 scan-tree-dump-times fre3 ";; >>> Function" 1 >>> g++.dg/tree-ssa/pr61034.C -std=gnu++14 scan-tree-dump-times fre3 "free" 14 >>> g++.dg/tree-ssa/pr61034.C -std=gnu++14 scan-tree-dump-times optimized >>> "free" 0 >>> g++.dg/tree-ssa/pr61034.C -std=gnu++17 scan-tree-dump-times fre3 ";; >>> Function" 1 >>> g++.dg/tree-ssa/pr61034.C -std=gnu++17 scan-tree-dump-times fre3 "free" 14 >>> g++.dg/tree-ssa/pr61034.C -std=gnu++17 scan-tree-dump-times optimized >>> "free" 0 >>> g++.dg/tree-ssa/pr61034.C -std=gnu++98 scan-tree-dump-times fre3 ";; >>> Function" 1 >>> g++.dg/tree-ssa/pr61034.C -std=gnu++98 scan-tree-dump-times fre3 "free" 14 >>> g++.dg/tree-ssa/pr61034.C -std=gnu++98 scan-tree-dump-times optimized >>> "free" 0 >> >> There were others, but starting with these seems as good as any. > > Sorry for that. Seems I managed to leave this one out of the final > patch. It is very sensitive to how things are inlined. If it breaks > again I guess I will try to work out explicit noinline/always_inline > attributes to make it to work. > > I have commited the following which returns the inliner to previous > behaviour. > > Honza > > 2019-10-03 Jan Hubicka <hubi...@ucw.cz> > > * g++.dg/tree-ssa/pr61034.C: Add --param max-inline-insns-single-O2=200. Looks like these are also triggered by the same inlining change:
> gcc.dg/guality/example.c -O2 -DPREVENT_OPTIMIZATION execution test > gcc.dg/guality/example.c -O2 -flto -fno-use-linker-plugin > -flto-partition=none -DPREVENT_OPTIMIZATION execution test > gcc.target/powerpc/pr79439-1.c scan-assembler-times \\mbl f\\M 1 > gcc.target/powerpc/pr79439-1.c scan-assembler-times \\mbl g\\M 1 > gcc.target/powerpc/pr79439-1.c scan-assembler-times \\mnop\\M 2 > gcc.target/powerpc/vsx-builtin-7.c scan-assembler-times \\mrldic\\M 64 Jeff
