On Thu, Sep 8, 2011 at 9:38 PM, Iyer, Balaji V <balaji.v.i...@intel.com> wrote:
> Hello Everyone,
>        In several places, I found that finish_function was using an absolute 
> integer as input parameter instead of these #defines
>
> #define SF_DEFAULT           0  /* No flags.  */
> #define SF_PRE_PARSED        1  /* The function declaration has
>                                   already been parsed.  */
> #define SF_INCLASS_INLINE    2  /* The function is an inline, defined
>                                   in the class body.  */
>
> I have attached a patch that should fix this (for example, use SF_DEFAULT 
> instead of '0'). It should fix the bug #47791 
> (http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47791)
>

A few comments:

1.  Remove [Master] in your email subject.
2.  Put ChangeLog entries like:

http://gcc.gnu.org/ml/gcc-patches/2011-09/msg00615.html

since you are not the only one who is changing trunk. Your
inlined ChangeLog won't apply when ChangeLogs are
updated by other people.


-- 
H.J.

Reply via email to