[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