[PATCH] D132988: Changes to use a string pool for symbols

Stefan Gränitz via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 7 09:21:10 PDT 2022


sgraenitz added inline comments.


================
Comment at: llvm/include/llvm/ExecutionEngine/JITLink/JITLink.h:973
   LinkGraph &operator=(LinkGraph &&) = delete;
+  ~LinkGraph();
 
----------------
jaredwy wrote:
> sgraenitz wrote:
> > Do we have a definition for the dtor here?
> Sorry don't follow the question? Implementation is further down if that's what you meant?  Did you want me to move it inline here? 
Ah you are right, sorry missed that.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D132988



More information about the llvm-commits mailing list