[PATCH] D57182: [MC] Remove unused inline function and stale comments; NFC

Sam Clegg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 4 12:22:57 PST 2019


sbc100 added a comment.

lgtm.

I assume its safe to remove the destructors here as they were only used for anchors and these class have other virtual functions in the cpp files already.   The rules for anchor methods are here: https://llvm.org/docs/CodingStandards.html#provide-a-virtual-method-anchor-for-classes-in-headers.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D57182





More information about the llvm-commits mailing list