[PATCH] D51763: Remove addBlockByrefAddress(), it is dead code as far as clang is concerned.
David Blaikie via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Sep 6 17:14:14 PDT 2018
dblaikie added inline comments.
================
Comment at: lib/CodeGen/AsmPrinter/DwarfCompileUnit.cpp:981-982
addComplexAddress(DV, Die, dwarf::DW_AT_location, Location);
- else if (DV.isBlockByrefVariable())
- addBlockByrefAddress(DV, Die, dwarf::DW_AT_location, Location);
else
----------------
Should this then be an assertion? And possibly a verifier check?
https://reviews.llvm.org/D51763
More information about the llvm-commits
mailing list