> On 10/4/18 11:23 AM, Jan Hubicka wrote:
> >> Hi.
> >>
> >> That's new warnings that warns about ifunc having a function body.
> >> It provides following warning:
> >>
> >> /home/marxin/Programming/gcc/gcc/testsuite/gcc.dg/pr87483.c: In function
> >> ‘g’:
> >> /home/marxin/Programming/gcc/gcc/testsuite/gcc.dg/pr87483.c:5:35: warning:
> >> ‘alias’ attribute ignored because function is defined [-Wattributes]
> >> 5 | __attribute__ ((alias ("f"))) int g () { return 1; } /* { dg-warning
> >> "attribute ignored because function is defined" } */
> >> | ^
> >>
> >> Hope I found proper place in cgraphunit.c for the warning.
> >> Patch survives bootstrap and tests on x86_64-linux-gnu.
> >>
> >> Ready for trunk?
> >
> > I think one can first define the function and later declare it again
> > with additional attribute. I think it is better to take care of this
> > in process_function_and_variable_attributes
>
> I've done that.
> Patch survives regression tests.
>
> Is it ok now?
OK,
thanks!
Honza
