For now, we will continue to allow module's destructors to throw exceptions. Explicitly added 'noexcept(false)' to base class destructors to silence warnings from gcc 6.

Because a module uses dual inheritance to GenericBenchmark and the base class for modules' use noexcept(false) we had to make GenericBenchmark's destructor just as loose.

The EDProducer did not need to inherit from TObject and the inheritance was interfering with the change of the destructor to be noexcept(false).

This class is used as a base class for multiple inheritance with a
framework module base class. For gcc 6 the framework modules explicitly
declared that the destructors are allowed to thow exceptions [noexcept(false)].
However, in the case of multiple inheritance, this meant all other base classes
used also need to allow exceptions else there is a compile time error.