[PATCH] D74702: [CSInfo][TailDuplicator] Delete the call site info when removing dead MBBs
    Djordje Todorovic via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Mon Feb 17 11:44:15 PST 2020
    
    
  
djtodoro marked an inline comment as done.
djtodoro added a comment.
> I believe this supersedes https://reviews.llvm.org/D74614?
I think I can come up with a test case describing we need the D74614 <https://reviews.llvm.org/D74614> as well (but modified, that is why I marked that as not finished).
================
Comment at: llvm/lib/CodeGen/TailDuplicator.cpp:1023
+      MF->eraseCallSiteInfo(&MI);
+  });
+
----------------
vsk wrote:
> Just so that the history makes sense, could you add -debug-entry-values to a TailDuplicator test in this patch, then remove it in D73534?
Sure, I’ll add that.
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D74702/new/
https://reviews.llvm.org/D74702
    
    
More information about the llvm-commits
mailing list