[PATCH] D58534: dsymutil support for DW_OP_convert

[Frederic Riss via Phabricator via llvm-commits]

friss added inline comments.


================
Comment at: llvm/tools/dsymutil/DwarfLinker.cpp:1100
+        memset(Buffer, dwarf::DW_OP_nop, ULEBsize);
+        Linker.reportWarning("base type ref doesn't fit.", DMO);
+      }
----------------
if you hit the error, then it looks like you're only going to emit NOPs instead of the ULEB. This is going to be badly misinterpreted by a consumer. You really want to NOP-out the whole convert expression.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D58534/new/

https://reviews.llvm.org/D58534