[Frontend] Fix mcount inlining bug
authorHonggyu Kim <hong.gyu.kim@lge.com>
Thu, 1 Sep 2016 11:29:21 +0000 (11:29 +0000)
committerHonggyu Kim <hong.gyu.kim@lge.com>
Thu, 1 Sep 2016 11:29:21 +0000 (11:29 +0000)
commit2b0e424b2f7131d7f099708c9ddc208f2ba5a751
treea5c546fcd6961ce4c9c918936760fe53c743bea9
parent689457659b2499c6d35a899a31eafc2535bd82e2
[Frontend] Fix mcount inlining bug

Since some profiling tools, such as gprof, ftrace, and uftrace, use
-pg option to generate a mcount function call at the entry of each
function. Function invocation can be detected by this hook function.

But mcount insertion is done before function inlining phase in clang,
sometime a function that already has a mcount call can be inlined in the
middle of another function.

This patch adds an attribute "counting-function" to each function
rather than emitting the mcount call directly in frontend so that this
attribute can be processed in backend. Then the mcount calls can be
properly inserted in backend after all the other optimizations are
completed.

Link: https://llvm.org/bugs/show_bug.cgi?id=28660
Reviewers: hans, rjmccall, hfinkel, rengolin, compnerd

Subscribers: shenhan, cfe-commits

Differential Revision: https://reviews.llvm.org/D22666

llvm-svn: 280355
clang/lib/CodeGen/CodeGenFunction.cpp
clang/test/CodeGen/mcount.c
clang/test/Frontend/gnu-mcount.c