[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