[PATCH] D22666: Frontend: Fix mcount inlining bug

Honggyu Kim via cfe-commits cfe-commits at lists.llvm.org
Thu Jul 28 20:58:19 PDT 2016


honggyu.kim added a comment.

In https://reviews.llvm.org/D22666#500329, @hfinkel wrote:

> In this case, I think that making a simple test (changing the current test to be) like test/CodeGen/stackrealign.c would be fine. If you have any questions, please feel free to ask.


Thanks. Please correct me if I'm wrong.

  diff --git a/test/CodeGen/mcount.c b/test/CodeGen/mcount.c
  index 99dc907..31b5288 100644
  --- a/test/CodeGen/mcount.c
  +++ b/test/CodeGen/mcount.c
  @@ -13,6 +13,7 @@
   // RUN: %clang_cc1 -pg -triple sparc-netbsd -emit-llvm -o - %s | FileCheck -check-prefix=CHECK-PREFIXED %s
   // RUN: %clang_cc1 -pg -triple sparc64-netbsd -emit-llvm -o - %s | FileCheck -check-prefix=CHECK-PREFIXED %s
   void foo(void) {
  -// CHECK: call void @mcount()
  -// CHECK-PREFIXED: call void @_mcount()
   }
  +
  +// CHECK: attributes #{{[0-9]+}} = { "counting-function"="mcount" {{.*}} }
  +// CHECK-PREFIXED: attributes #{{[0-9]+}} = { "counting-function"="_mcount" {{.*}} }


https://reviews.llvm.org/D22666





More information about the cfe-commits mailing list