[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