[llvm-commits] [llvm-gcc-4.2] r51608 - /llvm-gcc-4.2/trunk/gcc/llvm-backend.cpp

[Duncan Sands]

Hi Evan,

> Really honor -fno-inline-functions.

> -    if (flag_inline_trees)                      // respect -fno-inline-functions
> +    if (flag_inline_trees > 1)                  // respect -fno-inline-functions

the gcc middle- and back-ends do not make this distinction.
All the places that use flag_inline_trees only check whether
it is non-zero or not!

However language front-ends do play with it in order to set
DECL_INLINE, for example in c-decl.c there is:

        else if (flag_inline_trees == 2 && initialized)
          DECL_INLINE (decl) = 1;

While the Ada front-end does:

  DECL_INLINE (fndecl)
    = DECL_DECLARED_INLINE_P (fndecl) || flag_inline_trees == 2;

On the other hand the Fortran front-end doesn't do anything.

I think the right thing to look at is DECL_INLINE, and only
inline functions with the DECL_INLINE flag set.  What do you
think?

Ciao,

Duncan.