[llvm-bugs] [Bug 39176] New: [SimplifyCFG] Merge DebugLoc when speculatively hoisting store instruction

[via llvm-bugs]

https://bugs.llvm.org/show_bug.cgi?id=39176

            Bug ID: 39176
           Summary: [SimplifyCFG] Merge DebugLoc when speculatively
                    hoisting store instruction
           Product: libraries
           Version: trunk
          Hardware: PC
                OS: Linux
            Status: NEW
          Severity: enhancement
          Priority: P
         Component: Transformation Utilities
          Assignee: unassignedbugs at nondot.org
          Reporter: sontuan.vu119 at gmail.com
                CC: llvm-bugs at lists.llvm.org

Hello,

In lib/Transforms/Utils/SimplifyCFG.cpp, function 'SpeculativelyExecuteBB()',
when we create a new SelectInst (named 'spec.store.select') and make it operand
0 of the speculated store instruction, why do we need to modify the debug
location of the speculated store?

IIUC, the debug location of the speculated store should not be changed: the
debug location of the branching instruction is already attached to the
SelectInst.

Thanks for your time

-- 
You are receiving this mail because:
You are on the CC list for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-bugs/attachments/20181004/91e91585/attachment.html>