[PATCH] D29062: [InstCombine] Merge DebugLoc when speculatively hoisting store instruction

Taewook Oh via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 23 17:38:18 PST 2017


twoh created this revision.

Along with https://reviews.llvm.org/D27804, debug locations need to be merged when hoisting store instructions as well. Not sure if just dropping debug locations would make more sense for this case, but as the branch instruction will have at least different discriminator with the hoisted store instruction, I think there will be no difference in practice.


https://reviews.llvm.org/D29062

Files:
  lib/Transforms/Utils/SimplifyCFG.cpp
  test/Transforms/SimplifyCFG/remove-debug-2.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D29062.85505.patch
Type: text/x-patch
Size: 4941 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170124/62ce1c87/attachment.bin>


More information about the llvm-commits mailing list