[PATCH] D30226: [BranchFolding] Merge debug locations from common tail instead of removing

Taewook Oh via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 21 14:39:35 PST 2017


twoh created this revision.

https://reviews.llvm.org/D25742 improved the precision of debug locations for PGO by removing debug locations from common tail when tail-merging. However, if identical insturctions that are merged into a common tail have the same debug locations, there's no need to remove them. This patch creates a merged debug location of identical instructions across SameTails and assign it to the instruction in the common tail, so that the debug locations are maintained if they are same across identical instructions.


https://reviews.llvm.org/D30226

Files:
  lib/CodeGen/BranchFolding.cpp
  lib/CodeGen/BranchFolding.h
  test/CodeGen/X86/tail-merge-debugloc.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D30226.89283.patch
Type: text/x-patch
Size: 4729 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170221/cbb3f290/attachment.bin>


More information about the llvm-commits mailing list