[PATCH] D46738: [DebugInfo] Fix PR37395.

Adrian Prantl via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 29 09:36:21 PDT 2018


aprantl added a comment.

My general problem with this is that a debug info intrinsic with a null operand is usually indicative of a bug in a transformation pass that forgot to update the debug intrinsic after modifying the value it was pointing to. If the value goes away then the debug intrinsic should use an undef value (to end the range started by an earlier dbg intrinsic describing the same variable) or be removed entirely (if all instructions dealing with that variable have been removed as dead code). I don't want to make it easier to for transformation passes to not update the debug info correctly. If all you need is that DILabels are handled correctly, then this patch should just specifically enable that case.


Repository:
  rL LLVM

https://reviews.llvm.org/D46738





More information about the llvm-commits mailing list