[testsuite] Modify some PRE tests so that they don't fail on pretty-ipa

Richard Guenther richard.guenther@gmail.com
Fri Mar 26 11:43:00 GMT 2010


On Thu, Mar 25, 2010 at 6:04 PM, Janis Johnson
<janis.marie.johnson@gmail.com> wrote:
> On Thu, Mar 25, 2010 at 9:59 AM, Jan Hubicka <hubicka@ucw.cz> wrote:
>>> Hi,
>>>
>>> as in the case with the IPA testcases, some of the PRE tests fail
>>> because of changes in predict.c.  The code is deemed to be cold there
>>> because main is assumed to be running just once and PRE then avoids
>>> some transformations we check for.  Because all of them are compile
>>> only tests, I have changed the name of the function to be foo.
>>>
>>> All the affected tests pass on both pretty-ipa and trunk.  Again, I'd
>>> probably prefer to commit this to trunk get it to the branch through
>>> merge so that I don't have to care about this patch anymore.  So is it
>>> OK for trunk?
>>
>> This is fine with me for pretty-ipa, I can't approve these for trunk,
>> but also would preffer them to be updated tehre.
>
> OK for trunk as well.

Would be nice if you removed the argc/argv arguments if they
are not used.

Richard.

> Janis
>



More information about the Gcc-patches mailing list