[all-commits] [llvm/llvm-project] a3252d: [lldb][NFC] Move various constructor definitions f...

River Riddle via All-commits all-commits at lists.llvm.org
Thu Mar 30 13:26:58 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: a3252d1a2c568792974a4bc7413b0c61a43053e9
      https://github.com/llvm/llvm-project/commit/a3252d1a2c568792974a4bc7413b0c61a43053e9
  Author: River Riddle <riddleriver at gmail.com>
  Date:   2023-03-30 (Thu, 30 Mar 2023)

  Changed paths:
    M lldb/include/lldb/Expression/ExpressionVariable.h
    M lldb/include/lldb/Expression/Materializer.h
    M lldb/include/lldb/Symbol/TypeSystem.h
    M lldb/source/Expression/ExpressionVariable.cpp
    M lldb/source/Expression/Materializer.cpp
    M lldb/source/Symbol/TypeSystem.cpp

  Log Message:
  -----------
  [lldb][NFC] Move various constructor definitions from .h to .cpp

I ran into issues with linking downstream language plugin to liblldb in
debug builds, hitting link time errors of form:

```
undefined reference to `vtable for  lldb_private::<Insert LLDB class here>'
```

Anchoring the vtable to the .cpp files resolved those issues.

Differential Revision: https://reviews.llvm.org/D147252




More information about the All-commits mailing list