Ping. https://gcc.gnu.org/ml/gcc-patches/2016-10/msg02525.html
On Mon, Oct 31, 2016 at 08:56:10PM +0100, Dominik Vogt wrote: > The attached patch does a little change in > combine.c:combine_simplify_rtx() to prevent a "simplification" > where the rtl code gets more complex in reality. The complete > description of the change can be found in the commit comment in > the attached patch. > > The patch reduces the number of patterns in the s390 backend and > slightly reduces the size of the compiled SPEC2006 code. (Code > size or runtime only tested on s390x with -m64.) It is > theoretically possible that this patch leads to somewhat worse > code on some target if that only has a pattern for the formerly replaced > rtl expression but not for the original one. > > The patch has passed the testsuite on s390, s390x biarch, x86_64 > and Power biarch. > > -- > > (I'm not sure whether the const_int expression can appear in both > operands or only as the second. If the latter is the case, the > conditions can be simplified a bit.) > > What do you think about this patch? Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany
