[PATCH] D40413: [CodeExtractor] Add debug locations for new call and branch instrs.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 8 04:17:38 PST 2017


fhahn updated this revision to Diff 126122.
fhahn added a comment.

Fix code to use the first debug location found, not the last! Updated the test case with multiple debug locations, to check we actually pick the first one, if there are multiple blocks with debug info.

It would be great if someone could have another quick look at this version of the patch, as it changed quite a bit since being accepted.


https://reviews.llvm.org/D40413

Files:
  lib/Transforms/Utils/CodeExtractor.cpp
  test/Transforms/CodeExtractor/PartialInlineDebug.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D40413.126122.patch
Type: text/x-patch
Size: 6427 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171208/7a15e3dc/attachment.bin>


More information about the llvm-commits mailing list