> The tests tonight shows quite good results - there are quite consistent > code size reductions on C++ code w/o slowdowns so I've comitted the patch. > Sadly tramp3d does not build and the heuristics was implemented with > tramp3d in mind. Important behaviour change is that the heuristic now > matches only for parameters passed by value not for parameters passed by > reference. I however have the patch for tracking the second more > consistently with aid of the new predicate infrastructure so I guess I will > fix the possible regression on tramp3d other way.
Thanks for the explanation and the fix. > I've comitted the patch. I never added Ada testcase, but it would make > sense if you commit it if that ICE is not covered by some other Ada test > already. There is apparently an ACATS failure on x86-64/Darwin, but I've installed the testcase as gnat.dg/opt19.adb in the tree. -- Eric Botcazou
