[PATCH] D70497: [DBG] Teach DebugEntityHistoryCalculator about Kill instructions.

Vedant Kumar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 9 09:37:54 PST 2019


vsk added inline comments.


================
Comment at: llvm/lib/CodeGen/AsmPrinter/DbgEntityHistoryCalculator.cpp:265
 
-      if (MI.isDebugInstr())
+      if (MI.isDebugInstr() || MI.isKill())
         continue;
----------------
aprantl wrote:
> this needs a comment. I don't really understand the reasoning in the patch description either, so if you could explain why this is safe/correct, that would help me!
@TWeaver apologies for the reverse-ping, but I'm also having some trouble following this and think that an explanatory comment would help.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D70497/new/

https://reviews.llvm.org/D70497





More information about the llvm-commits mailing list