On Fri, Nov 18, 2016 at 03:38:38PM -0500, Michael Meissner wrote: > This patch tweaks the movdi constraints for the PowerPC to use "^" or "$" > constraints instead of "?*". This allows the register allocator to more often > allocate DImode to a floating point/vector register when it is desirable to do > so.
It also changes some plain "?" to "^" or "$" or even "*" (which cannot work for multi-character constraints, it skips one character, not one constraint). Wrong version of the patch? > I built bootstrap compilers and did make check with no regressions on: > 1) Little endian power8, --with-cpu=power8 > 2) Big endian power8, --with-cpu=power8 (no 32-bit support) > 3) Big endian power7, --with-cpu=power7 (both 32/64-bit support) Could you also test with reload please? Just LE is enough I guess. We'd like to keep reload working for GCC 7 at least, and these cost prefixes tend to break mov patterns :-/ Segher
