[llvm] [GOFF] Emit symbols for functions. (PR #144437)

[Kai Nacke via llvm-commits]

================
@@ -345,9 +372,13 @@ void GOFFWriter::defineSymbols() {
     if (Sym.isTemporary())
       continue;
     auto &Symbol = static_cast<const MCSymbolGOFF &>(Sym);
-    if (Symbol.hasLDAttributes()) {
+    bool IsDefined = Symbol.isDefined();
+    if (IsDefined && static_cast<MCSectionGOFF &>(Symbol.getSection()).isED()) {
----------------
redstar wrote:

Yes, there can be symbols which are in a PR. That's in the last missing part, emitting global data.

https://github.com/llvm/llvm-project/pull/144437