[PATCH] D28075: MergeFunctions: Preserve debug info in thunks, under option -mergefunc-preserve-debug-info

[Anmol P. Paralkar via Phabricator via llvm-commits]

appcs updated this revision to Diff 83501.
appcs added a comment.

Under -mergefunc-preserve-debug-info

- A thunk's call site is preserved to point to the thunk when both occur within the same translation unit.

A thunk makes a tail call to the shared implementation (i.e. keep -mergefunc defined base behaviour).

Review comments have been incorporated; in particular, the test case has been reworked to:

- Use smaller & simpler functions that illustrate the problem.
- The debug intrinsics are minimal.
- Unnecessary debug metadata has been removed.
- Attributes have been removed.


https://reviews.llvm.org/D28075

Files:
  lib/Transforms/IPO/MergeFunctions.cpp
  test/Transforms/MergeFunc/mergefunc-preserve-debug-info.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D28075.83501.patch
Type: text/x-patch
Size: 22556 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170107/501e59f1/attachment.bin>