> > +/* Nonzero if inlining should prefer inlining this function.
> > +   Shorthand for DECL_DECLARED_INLINE_P && !DECL_FEEBLE_INLINE_P.  */
> > +#define DECL_OPTIMIZABLE_INLINE_P(NODE) \
> > +  (DECL_DECLARED_INLINE_P (NODE) && !DECL_FEEBLE_INLINE_P (NODE))
> 
> We have 10 bits left, but it would also make sense to put the flag into
> inline summary and use explicit lookup_attribute elsewhere.  Inliner is
> only place that cares about this in hot code.
> 
> IPA parts of the patch looks OK however I wonder if simply clearing
> DECL_DECLARED_INLINE_P would have same effect?

Also DECL_OPTIMIZABLE_INLINE_P sounds bit odd.  Perhaps
DECL_AGGRESSIVE_INLINING_P?

Honza
> 
> Honza

Reply via email to