[PATCH] D125777: [llvm-debuginfo-analyzer] 02 - Driver and documentation

[Sergei Barannikov via Phabricator via llvm-commits]

barannikov88 added inline comments.


================
Comment at: llvm/include/llvm/DebugInfo/LogicalView/Core/LVObject.h:53
+  std::string lineNumberAsStringStripped(bool ShowZero = false) const;
+};
+
----------------
CarlosAlbertoEnciso wrote:
> psamolysov wrote:
> > The virtual destructor is required, isn't it?
> I see your point. What this patch adds is just the driver and the documentation. No logical elements are created; basically no instances of `LVObject`, `LVElement`, `LVLine`, ``LVScope` or `LVSymbol` are created.
Is it going to be added in a separate patch? Currently, the absence of the virtual destructor causes compilation warnings.
The vtable should also be pinned to a single translation unit; the destructor could be used as an anchor.



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125777