[lld] [LLD] Add support for statically resolved vendor-specific RISCV relocations. (PR #169273)

[Owen Anderson via llvm-commits]

================
@@ -8,12 +8,19 @@
 TARGET:
   nop
 
-.global INVALID_VENDOR
+.local INVALID_VENDOR
----------------
resistor wrote:

> The QUALCOMM vendor relocations require the local symbol to be defined.

I wasn't aware of that requirement. Is it documented somewhere? I checked the Xqci specification and don't immediately see it.

> I am not sure we should have the custom diagnostic when the symbol is not defined.

I don't see why that would be the case. In the scenario where the vendor symbol name matches a known one, and the relocation number matches a known one, and the only thing that's missing is that the symbol is undefined rather than defined, why would we prefer to give a more opaque error message?



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