[llvm-dev] Can DISubprogram be renamed?

Sergei Larin via llvm-dev llvm-dev at lists.llvm.org
Fri Apr 15 09:50:31 PDT 2016


 

Peter,

 

  I am afraid I am missing a big picture with debug info cloning.

 

  Certainly after your patch (and possibly in some cases earlier):

 

    Cloning: Reduce complexity of debug info cloning and fix correctness
issue.

    

    Commit r260791 contained an error in that it would introduce a
cross-module

    reference in the old module. It also introduced O(N^2) complexity in the

    module cloner by requiring the entire module to be visited for each
function.

    Fix both of these problems by avoiding use of the CloneDebugInfoMetadata

    function (which is only designed to do intra-module cloning) and cloning

    function-attached metadata in the same way that we clone all other
metadata.

    

    Differential Revision: http://reviews.llvm.org/D18583

 

During CloneFunctionInto we seem to create a new DISubprogram which still
carries the name of the _original_ function. which creates certain confusion
with debugging scope.

 

Was it designed that way, or am I missing something here. 

 

Thanks a lot.

 

Sergei

 

 

 

---

Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by
The Linux Foundation

 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20160415/72cdd542/attachment.html>


More information about the llvm-dev mailing list