This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] combine: Allow substituting the target reg of a clobber


On 10/01/14 16:36, Segher Boessenkool wrote:
On Sat, Sep 27, 2014 at 05:03:26PM -0500, Segher Boessenkool wrote:
I've tried to make a stable future-proof testcase that does such a three-insn
combination.  Not easy at all.

Turns out it is quite easy (when you've seen the solution, anyway :-P )

Tested on powerpc64-linux as before, and bootstrapped + regression checked
on x86_64-linux.  Also checked the testcase fails before on i386 and x86_64,
and works afterwards.

Does this look good?


Segher



2014-10-02  Segher Boessenkool  <segher@kernel.crashing.org>

gcc/
	PR rtl-optimization/62151
	* combine.c (can_combine_p): Allow the destination register of INSN
	to be clobbered in I3.
	(subst): Do not substitute into clobbers of registers.

gcc/testsuite/
	* gcc.dg/combine-clobber.c: New.
Excellent. Thanks. Even more so for making the test for x86 since that's the platform that gets tested the most often.

Ok for the trunk.

jeff


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]