#554: cleanup; remove unused function, const-ref

- Call to 'getFunction' is done directly on AdapterType

- Removed '&' and specified type for simplicity/clarity
  (oops! force update with correction.. go compilation flags)

src/vt/registry/auto/auto_registry_general_impl.h

@@ -76,7 +76,7 @@ RegistrarGen<RunnableT, RegT, InfoT, FnT>::RegistrarGen() {
   // trace
   std::string event_type_name = AdapterType::traceGetEventType();
   std::string event_name = AdapterType::traceGetEventName();
-  auto const& trace_ep = trace::TraceRegistry::registerEventHashed(
+  trace::TraceEntryIDType trace_ep = trace::TraceRegistry::registerEventHashed(
     event_type_name, event_name);
   reg.emplace_back(InfoT{fn, std::move(indexAccessor), trace_ep});
 #else
@@ -90,13 +90,6 @@ AutoHandlerType registerActiveGen() {
   return RegistrarWrapperGen<RunnableT, RegT, InfoT, FnT>().registrar.index;
 }
 
-template <typename RunnableT, typename RegT, typename InfoT, typename FnT>
-/*static*/ constexpr typename
-RunnableGen<RunnableT, RegT, InfoT, FnT>::FunctionPtrType
-RunnableGen<RunnableT, RegT, InfoT, FnT>::getFunction() {
-  return RunnableT::AdapterType::getFunction();
-}
-
 template <typename AdapterT, typename RegT, typename InfoT, typename FnT>
 AutoHandlerType const RunnableGen<AdapterT, RegT, InfoT, FnT>::idx =
   registerActiveGen<RunnableGen<AdapterT, RegT, InfoT, FnT>, RegT, InfoT, FnT>();