[PATCH] D29596: Make MachineBasicBlock::updateTerminator to update DebugLoc as well

Taewook Oh via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 7 09:21:07 PST 2017


twoh updated this revision to Diff 87470.
twoh added a comment.

Implemented a separate 'findBranchDebugLoc'. This functions does essentially the same job as 'getBranchDebugLoc' function in BranchFolding.cpp. Unlike getBranchDebugLoc, the new findBranchDebugLoc finds all branch instructions inside the function and returns the merged DebugLoc of them. I think it is more conservative, Though I can't imagine the case of two branch instructions in the same machine basic block have different DebugLoc.


https://reviews.llvm.org/D29596

Files:
  include/llvm/CodeGen/MachineBasicBlock.h
  lib/CodeGen/MachineBasicBlock.cpp
  test/CodeGen/X86/update-terminator-debugloc.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D29596.87470.patch
Type: text/x-patch
Size: 6470 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170207/13e839fd/attachment.bin>


More information about the llvm-commits mailing list