[PATCH] D44312: Retain both sets of debug intrinsics in HoistThenElseCodeToIf (fixes PR 36410)

Mon Mar 12 06:53:52 PDT 2018

uweigand updated this revision to Diff 137996.
uweigand added a comment.

> Would it be an option to only hoist debug info instructions where both variable and value match (just like the existing code), but then not attempt to merge the !dbg locations, but instead just hoist both copies of the original instructions with both the original locations (like in this patch)?

Updated the patch to implement the above option, just to have a better basis for discussion ...

https://reviews.llvm.org/D44312

Files:
  lib/Transforms/Utils/SimplifyCFG.cpp
  test/Transforms/SimplifyCFG/hoist-dbgvalue-inlined.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D44312.137996.patch
Type: text/x-patch
Size: 6233 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180312/41dadd1b/attachment.bin>