[PATCH] [RTL] Relax CSE check to set REG_EQUAL notes.

Jeff Law law@redhat.com
Mon Mar 9 17:40:00 GMT 2015


On 03/09/15 03:53, Steven Bosscher wrote:
> On Wed, Mar 4, 2015 at 12:09 PM, Alex Velenko wrote:
>> For example, in arm testcase pr43920-2.c, CSE previously decided not to put
>> an "obvious" note on insn 9, as set value was the same as note value.
>> At the same time, other insns set up as -1 were set up through a register
>> and did get a note:
>
> ...which is the point of the REG_EQUAL notes. In insn 8 there is a
> REG_EQUAL note to show that the value of r111 is known. In insn 9 the
> known value is, well, known from SET_SRC so there is no need for a
> REG_EQUAL note. Adding REG_EQUAL notes in such cases is just wasteful.
RIght.  I'd rather look into why later passes aren't discovering 
whatever equivalences are important rather than adding the redundant notes.

Regardless, I think this is a gcc-6 issue, so I'm not likely to look at 
it in the immediate future.

jeff



More information about the Gcc-patches mailing list