[PATCH, testsuite] fix ggcplug.c test-case

[Richard Biener]

On Mon, 12 Jan 2015, Prathamesh Kulkarni wrote:

> On 12 January 2015 at 14:19, Richard Biener <rguenther@suse.de> wrote:
> > On Sun, 11 Jan 2015, Prathamesh Kulkarni wrote:
> >
> >> Hi,
> >> The test-case plugin/ggcplug.c was failing due to flattening of tree.h
> >> and tree-core.h.
> >> Test-case was incorrect because it included gcc-plugin.h after tree.h whereas
> >> gcc-plugin.h should be the first header to be included by plugins.
> >
> > No, it should be definitely included _after_ config.h, system.h
> > and coretypes.h.
> gcc-plugin.h already includes these files. Shall I remove config.h,
> system.h and coretypes.h
> from ggcplug.c instead ?

No, keep the patch simple for now - we are inconsitent in all the
testsuite plugins it seems and wasn't the idea that plugins _only_
need to include gcc-plugin.h now?  Thus I'd rather cleanup all
plugin testcases at once, with a separate patch.

Thanks,
Richard.

> >
> > Ok with moving it after coretypes.h.
> >
> > Thanks,
> > Richard.
> 
> 

-- 
Richard Biener <rguenther@suse.de>
SUSE LINUX GmbH, GF: Felix Imendoerffer, Jane Smithard, Jennifer Guild,
Dilip Upmanyu, Graham Norton HRB 21284 (AG Nuernberg)