[PATCH] D22666: Frontend: Fix mcount inlining bug

Honggyu Kim via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 22 21:15:42 PDT 2016


honggyu.kim added a reviewer: compnerd.
honggyu.kim removed a subscriber: compnerd.
honggyu.kim updated this revision to Diff 68954.
honggyu.kim added a comment.

I have fixed SOH character problem by https://reviews.llvm.org/D23792.  I've verified that mcount.c and gnu-mcount.c tests are passed now.

  $ llvm-lit tools/clang/test/CodeGen/mcount.c
  -- Testing: 1 tests, 1 threads --
  PASS: Clang :: CodeGen/mcount.c (1 of 1)
  Testing Time: 0.81s
    Expected Passes    : 1
  
  $ llvm-lit tools/clang/test/Frontend/gnu-mcount.c
  -- Testing: 1 tests, 1 threads --
  PASS: Clang :: Frontend/gnu-mcount.c (1 of 1)
  Testing Time: 10.36s
    Expected Passes    : 1

I think it's almost ready to be commited now.  So please have a look at it with https://reviews.llvm.org/D23792.


https://reviews.llvm.org/D22666

Files:
  lib/CodeGen/CodeGenFunction.cpp
  test/CodeGen/mcount.c
  test/Frontend/gnu-mcount.c

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D22666.68954.patch
Type: text/x-patch
Size: 8179 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20160823/6b961219/attachment.bin>


More information about the cfe-commits mailing list