[PATCH] D49454: [DebugInfo] LowerDbgDeclare: Add derefs when handling CallInst users

Adrian Prantl via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 17 22:49:40 PDT 2018


aprantl added a comment.

This looks like the right change as it makes the IR more regular.
Note that in the past this worked because allocas were implicitly indirect since they caused AsmPrinter to emit a what DWARF calls a "memory location". It's possible that that code path no longer exists or isn't triggered any more.

Have you tested the clang debuginfo-tests with this change, particularly the asan and array testcases?


https://reviews.llvm.org/D49454





More information about the llvm-commits mailing list