[NFC] Move Symbol::dump definition to symbol.cpp.

Author: schweitzpgi

flang/include/flang/Semantics/symbol.h

@@ -651,7 +651,7 @@ class Symbol {
   const DerivedTypeSpec *GetParentTypeSpec(const Scope * = nullptr) const;
 
 #if !defined(NDEBUG) || defined(LLVM_ENABLE_DUMP)
-  LLVM_DUMP_METHOD void dump() const { llvm::errs() << *this << '\n'; }
+  LLVM_DUMP_METHOD void dump() const;
 #endif
 
 private:

flang/lib/Semantics/symbol.cpp

@@ -505,6 +505,10 @@ llvm::raw_ostream &operator<<(llvm::raw_ostream &os, const Symbol &symbol) {
   return os;
 }
 
+#if !defined(NDEBUG) || defined(LLVM_ENABLE_DUMP)
+void Symbol::dump() const { llvm::errs() << *this << '\n'; }
+#endif
+
 // Output a unique name for a scope by qualifying it with the names of
 // parent scopes. For scopes without corresponding symbols, use the kind
 // with an index (e.g. Block1, Block2, etc.).