[PATCH] D51976: [DebugInfo][Dexter] Speculated BB presents illegal variable value to debugger

Adrian Prantl via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 14 13:54:05 PDT 2018


aprantl added a comment.

Having read the real testcase now, I understand why you are dropping the debug info now. I would like to point out that we could do better. If we extended llvm.dbg.value to take more than one LLVM SSA Value, then we could produce a DIExpression that selects either %add or %sub depending on the value of %cmp. Let me know if you are interested in implementing this, it should be relatively straight forward to do and would come in useful in all sorts of other situations, too.


https://reviews.llvm.org/D51976





More information about the llvm-commits mailing list