[llvm-commits] PATCH: [lld] Trivial update to ReaderCOFF.cpp

Clow, Marshall mclow at qualcomm.com
Thu Jun 28 08:42:54 PDT 2012


On Jun 27, 2012, at 2:21 PM, Sid Manning wrote:

> This change fixes the following link errors.
> 
> undefined reference to `lld::ReaderOptionsPECOFF::ReaderOptionsPECOFF()'
> undefined reference to `lld::ReaderOptionsPECOFF::~ReaderOptionsPECOFF()'
> 
> By adding empty versions of ReaderOptionsPECOFF and ~ReaderOptionsPECOFF.

If the constructor and destructor are empty, then why not remove the declarations, and let the compiler generate them?

-- Marshall






More information about the llvm-commits mailing list