[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