[PATCH] D70420: Use lld::make<T> to make TpiSource objects

[Rui Ueyama via Phabricator via llvm-commits]

ruiu marked an inline comment as done.
ruiu added inline comments.


================
Comment at: lld/COFF/DebugTypes.h:35
+  TpiSource(TpiKind k, const ObjFile *f) : kind(k), file(f) {}
   virtual ~TpiSource() {}
 
----------------
rnk wrote:
> nit, but if you put the ctor and dtor out of line, you will achieve the object size saving goals of this LLVM coding guideline:
> https://llvm.org/docs/CodingStandards.html#provide-a-virtual-method-anchor-for-classes-in-headers
Fixed before submitting. Thanks!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70420