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

[Duncan Sands]

Hi Dale,

> I don't think that's a good idea; the hint should be added to the IR.
> Compilers are not at the point where they can make better decisions
> than an intelligent user about which things are good to inline, and  
> IMO they
> never will be; for example, a user can know that certain calls are or  
> aren't
> executed frequently.

the linux kernel people removed "inline" almost everywhere because
(1) people put it on functions that were too big (eg: that had more
than one line in it), and (2) the gcc automatic inliner did a great
job, so it wasn't worth spending time thinking where to put "inline".

Ciao,

Duncan.