[PATCH] Reassociate X == CST1 || X == CST2 if popcount (CST2 - CST1) == 1 into ((X - CST1) & ~(CST2 - CST1)) == 0

Richard Biener richard.guenther@gmail.com
Wed Oct 16 09:02:00 GMT 2013


On Tue, Oct 15, 2013 at 6:55 PM, Jeff Law <law@redhat.com> wrote:
> On 10/15/13 10:53, Jakub Jelinek wrote:
>>
>> On Tue, Oct 15, 2013 at 10:50:39AM -0600, Jeff Law wrote:
>>>
>>> I noticed that we're now including rtl.h and tm_p.h in
>>> tree-ssa-reassoc.c, which seems wrong.
>>
>>
>> Isn't that required for BRANCH_COST use?
>> Other option would be to add some dummy wrapper around
>> BRANCH_COST, put that wrapper into some file that already includes rtl.h
>> and
>> tm_p.h and just call it from there.
>
> Yea, looking at it now, I'm a bit surprised this hasn't been converted to a
> target hook which would avoid this problem entirely.

You got the job!

;)

Richard.

> jeff



More information about the Gcc-patches mailing list