[PATCH] D22666: Frontend: Fix mcount inlining bug

Honggyu Kim via cfe-commits cfe-commits at lists.llvm.org
Tue Jul 26 06:48:54 PDT 2016


honggyu.kim added a comment.

One more problem is that -finstrument-functions has the same inlining problem.  The code in `llvm/tools/clang/lib/CodeGen/CodeGenFunction.cpp` shows that.

  void CodeGenFunction::StartFunction(GlobalDecl GD,
                                      QualType RetTy,
                                      llvm::Function *Fn,
                                      const CGFunctionInfo &FnInfo,
                                      const FunctionArgList &Args,
                                      SourceLocation Loc,
                                      SourceLocation StartLoc) {
      ...
    if (ShouldInstrumentFunction())
      EmitFunctionInstrumentation("__cyg_profile_func_enter");
  
    if (CGM.getCodeGenOpts().InstrumentForProfiling)
      EmitMCountInstrumentation();
      ...
  }


https://reviews.llvm.org/D22666





More information about the cfe-commits mailing list