[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