[PATCH] D57182: [MC] Clean up unused inline function and non-anchor defaulted destructors; NFCI

Hubert Tong via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 8 18:11:57 PST 2019


This revision was automatically updated to reflect the committed changes.
Closed by commit rL353597: [MC] Clean up unused inline function and non-anchor defaulted destructors; NFCI (authored by hubert.reinterpretcast, committed by ).

Changed prior to commit:
  https://reviews.llvm.org/D57182?vs=184171&id=186091#toc

Repository:
  rL LLVM

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

https://reviews.llvm.org/D57182

Files:
  llvm/trunk/include/llvm/MC/MCSectionCOFF.h
  llvm/trunk/include/llvm/MC/MCSectionELF.h
  llvm/trunk/include/llvm/MC/MCSectionWasm.h
  llvm/trunk/lib/MC/MCSectionCOFF.cpp
  llvm/trunk/lib/MC/MCSectionELF.cpp
  llvm/trunk/lib/MC/MCSectionWasm.cpp


Index: llvm/trunk/lib/MC/MCSectionELF.cpp
===================================================================
--- llvm/trunk/lib/MC/MCSectionELF.cpp
+++ llvm/trunk/lib/MC/MCSectionELF.cpp
@@ -17,8 +17,6 @@
 
 using namespace llvm;
 
-MCSectionELF::~MCSectionELF() = default; // anchor.
-
 // Decides whether a '.section' directive
 // should be printed before the section name.
 bool MCSectionELF::ShouldOmitSectionDirective(StringRef Name,
Index: llvm/trunk/lib/MC/MCSectionWasm.cpp
===================================================================
--- llvm/trunk/lib/MC/MCSectionWasm.cpp
+++ llvm/trunk/lib/MC/MCSectionWasm.cpp
@@ -14,8 +14,6 @@
 
 using namespace llvm;
 
-MCSectionWasm::~MCSectionWasm() = default; // anchor.
-
 // Decides whether a '.section' directive
 // should be printed before the section name.
 bool MCSectionWasm::shouldOmitSectionDirective(StringRef Name,
Index: llvm/trunk/lib/MC/MCSectionCOFF.cpp
===================================================================
--- llvm/trunk/lib/MC/MCSectionCOFF.cpp
+++ llvm/trunk/lib/MC/MCSectionCOFF.cpp
@@ -14,8 +14,6 @@
 
 using namespace llvm;
 
-MCSectionCOFF::~MCSectionCOFF() = default; // anchor.
-
 // ShouldOmitSectionDirective - Decides whether a '.section' directive
 // should be printed before the section name
 bool MCSectionCOFF::ShouldOmitSectionDirective(StringRef Name,
Index: llvm/trunk/include/llvm/MC/MCSectionWasm.h
===================================================================
--- llvm/trunk/include/llvm/MC/MCSectionWasm.h
+++ llvm/trunk/include/llvm/MC/MCSectionWasm.h
@@ -48,11 +48,7 @@
       : MCSection(SV_Wasm, K, Begin), SectionName(Section), UniqueID(UniqueID),
         Group(group) {}
 
-  void setSectionName(StringRef Name) { SectionName = Name; }
-
 public:
-  ~MCSectionWasm();
-
   /// Decides whether a '.section' directive should be printed before the
   /// section name
   bool shouldOmitSectionDirective(StringRef Name, const MCAsmInfo &MAI) const;
Index: llvm/trunk/include/llvm/MC/MCSectionELF.h
===================================================================
--- llvm/trunk/include/llvm/MC/MCSectionELF.h
+++ llvm/trunk/include/llvm/MC/MCSectionELF.h
@@ -63,8 +63,6 @@
   void setSectionName(StringRef Name) { SectionName = Name; }
 
 public:
-  ~MCSectionELF();
-
   /// Decides whether a '.section' directive should be printed before the
   /// section name
   bool ShouldOmitSectionDirective(StringRef Name, const MCAsmInfo &MAI) const;
Index: llvm/trunk/include/llvm/MC/MCSectionCOFF.h
===================================================================
--- llvm/trunk/include/llvm/MC/MCSectionCOFF.h
+++ llvm/trunk/include/llvm/MC/MCSectionCOFF.h
@@ -62,8 +62,6 @@
   }
 
 public:
-  ~MCSectionCOFF();
-
   /// Decides whether a '.section' directive should be printed before the
   /// section name
   bool ShouldOmitSectionDirective(StringRef Name, const MCAsmInfo &MAI) const;


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D57182.186091.patch
Type: text/x-patch
Size: 2916 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190209/7dec7593/attachment.bin>


More information about the llvm-commits mailing list