Use Err(ctx) instead of fatal and use getLocation in error message

Author: kovdan01

lld/ELF/Relocations.cpp

@@ -1110,9 +1110,10 @@ void RelocationScanner::processAux(RelExpr expr, RelType type, uint64_t offset,
       if (!(flags & NEEDS_GOT)) {
         sym.setFlags(needsGotAuth ? (NEEDS_GOT | NEEDS_GOT_AUTH) : NEEDS_GOT);
       } else if (needsGotAuth != static_cast<bool>(flags & NEEDS_GOT_AUTH)) {
-        fatal("both AUTH and non-AUTH GOT entries for '" + sym.getName() +
-              "' requested, but only one type of GOT entry per symbol is "
-              "supported");
+        Err(ctx) << "both AUTH and non-AUTH GOT entries for '" << sym.getName()
+                 << "' requested, but only one type of GOT entry per symbol is "
+                    "supported"
+                 << getLocation(ctx, *sec, sym, offset);
       }
     }
   } else if (needsPlt(expr)) {

lld/test/ELF/aarch64-got-relocations-pauth.s

@@ -84,7 +84,12 @@ _start:
 
 # RUN: not ld.lld err.o a.so -pie -o /dev/null 2>&1 | FileCheck %s --check-prefix=ERR
 
-# ERR: error: both AUTH and non-AUTH GOT entries for 'bar' requested, but only one type of GOT entry per symbol is supported
+# ERR:      error: both AUTH and non-AUTH GOT entries for 'bar' requested, but only one type of GOT entry per symbol is supported
+# ERR-NEXT: >>> defined in a.so
+# ERR-NEXT: >>> referenced by err.o:(.text+0x8)
+# ERR:      error: both AUTH and non-AUTH GOT entries for 'bar' requested, but only one type of GOT entry per symbol is supported
+# ERR-NEXT: >>> defined in a.so
+# ERR-NEXT: >>> referenced by err.o:(.text+0xC)
 
 .globl _start
 _start: