[PATCH] D37877: Update getMergedLocation to check the instruction type and merge properly.

Dehao Chen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 14 17:00:29 PDT 2017


danielcdh created this revision.
Herald added a subscriber: sanjoy.

If the merged instruction is call instruction, we need to set the scope to the closes common scope between 2 locations, otherwise it will cause trouble when the call is getting inlined.


https://reviews.llvm.org/D37877

Files:
  include/llvm/IR/DebugInfoMetadata.h
  lib/IR/DebugInfo.cpp
  lib/Transforms/Utils/SimplifyCFG.cpp
  test/Transforms/SimplifyCFG/remove-debug.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D37877.115320.patch
Type: text/x-patch
Size: 4566 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170915/233b34c6/attachment.bin>


More information about the llvm-commits mailing list