combine: Do not throw away unneeded arms of parallels (PR83156)
authorSegher Boessenkool <segher@kernel.crashing.org>
Wed, 29 Nov 2017 22:42:37 +0000 (23:42 +0100)
committerSegher Boessenkool <segher@gcc.gnu.org>
Wed, 29 Nov 2017 22:42:37 +0000 (23:42 +0100)
commit4a01617886c1833f456948048e5167dac784ef19
tree84cbcb2424e0e5745b60c58f0ffc57a5dfe877f9
parent18b776e936e15310907e014e4e460d45d148bbf6
combine: Do not throw away unneeded arms of parallels (PR83156)

The fix for PR82621 makes us not split an I2 if one of the results of
those SETs is unused, since combine does not handle that properly.  But
this results in degradation for i386 (or more in general, for any
target that does not have patterns for parallels with an unused result
as a CLOBBER instead of a SET for that result).

This patch instead makes us not split only if one of the results is set
again before I3.  That fixes PR83156 and also fixes PR82621.

Unfortunately it undoes the nice optimisations that the previous patch
did on powerpc.

PR rtl-optimization/83156
PR rtl-optimization/82621
* combine.c (try_combine): Don't split an I2 if one of the dests is
set again before I3.  Allow unused dests.

From-SVN: r255260
gcc/ChangeLog
gcc/combine.c