[llvm] r231113 - Remove explicit no-op dtor in favor of the implicit dtor so as not to disable/deprecate the copy operations.

David Blaikie dblaikie at gmail.com
Tue Mar 3 11:53:02 PST 2015


Author: dblaikie
Date: Tue Mar  3 13:53:02 2015
New Revision: 231113

URL: http://llvm.org/viewvc/llvm-project?rev=231113&view=rev
Log:
Remove explicit no-op dtor in favor of the implicit dtor so as not to disable/deprecate the copy operations.

Modified:
    llvm/trunk/include/llvm/Option/Option.h
    llvm/trunk/lib/Option/Option.cpp

Modified: llvm/trunk/include/llvm/Option/Option.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Option/Option.h?rev=231113&r1=231112&r2=231113&view=diff
==============================================================================
--- llvm/trunk/include/llvm/Option/Option.h (original)
+++ llvm/trunk/include/llvm/Option/Option.h Tue Mar  3 13:53:02 2015
@@ -70,7 +70,6 @@ protected:
 
 public:
   Option(const OptTable::Info *Info, const OptTable *Owner);
-  ~Option();
 
   bool isValid() const {
     return Info != nullptr;

Modified: llvm/trunk/lib/Option/Option.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Option/Option.cpp?rev=231113&r1=231112&r2=231113&view=diff
==============================================================================
--- llvm/trunk/lib/Option/Option.cpp (original)
+++ llvm/trunk/lib/Option/Option.cpp Tue Mar  3 13:53:02 2015
@@ -35,9 +35,6 @@ Option::Option(const OptTable::Info *inf
   }
 }
 
-Option::~Option() {
-}
-
 void Option::dump() const {
   llvm::errs() << "<";
   switch (getKind()) {





More information about the llvm-commits mailing list