[llvm] r207350 - ARM: MSVC does not support = default

[Aaron Ballman]

On Sun, Apr 27, 2014 at 1:28 AM, Saleem Abdulrasool
<compnerd at compnerd.org> wrote:
> Author: compnerd
> Date: Sun Apr 27 00:28:10 2014
> New Revision: 207350
>
> URL: http://llvm.org/viewvc/llvm-project?rev=207350&view=rev
> Log:
> ARM: MSVC does not support = default
>
> Explicitly "implement" the destructor as MSVC does not support defaulted methods
> yet.
>
> Modified:
>     llvm/trunk/lib/Target/ARM/MCTargetDesc/ARMWinCOFFObjectWriter.cpp
>
> Modified: llvm/trunk/lib/Target/ARM/MCTargetDesc/ARMWinCOFFObjectWriter.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/ARM/MCTargetDesc/ARMWinCOFFObjectWriter.cpp?rev=207350&r1=207349&r2=207350&view=diff
> ==============================================================================
> --- llvm/trunk/lib/Target/ARM/MCTargetDesc/ARMWinCOFFObjectWriter.cpp (original)
> +++ llvm/trunk/lib/Target/ARM/MCTargetDesc/ARMWinCOFFObjectWriter.cpp Sun Apr 27 00:28:10 2014
> @@ -23,7 +23,7 @@ public:
>      : MCWinCOFFObjectTargetWriter(COFF::IMAGE_FILE_MACHINE_ARMNT) {
>      assert(!Is64Bit && "AArch64 support not yet implemented");
>    }
> -  virtual ~ARMWinCOFFObjectWriter() = default;
> +  virtual ~ARMWinCOFFObjectWriter() { }

Would it make more sense to add LLVM_DEFAULTED_FUNCTION to Compiler.h
to complement LLVM_DELETED_FUNCTION?

~Aaron