[lld] [llvm] RFC: [LLD] [COFF] Fix linking MSVC generated implib header objects (PR #122811)

[Jacek Caban via llvm-commits]

Martin =?utf-8?q?Storsjö?= <martin at martin.st>
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/122811 at github.com>


================
@@ -219,7 +219,9 @@ class DefinedRegular : public DefinedCOFF {
     return s->kind() == DefinedRegularKind;
   }
 
-  uint64_t getRVA() const { return (*data)->getRVA() + sym->Value; }
+  uint64_t getRVA() const {
+    return (*data)->getRVA() + (sym->Value != 0xC0000040 ? sym->Value : 0);
----------------
cjacek wrote:

I wonder if we could avoid special-casing that value. From what I can see with binutils, value is simply ignored for such symbols. Maybe we could just always create a symbol with 0 value when handling `isEmptySectionDeclaration` in lld-link.

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