diff options
author | Reid Kleckner <rnk@google.com> | 2019-11-07 09:27:43 -0800 |
---|---|---|
committer | Reid Kleckner <rnk@google.com> | 2019-11-07 09:29:48 -0800 |
commit | c989993ba1a666f04f7aee7df51d9f4de0588b71 (patch) | |
tree | c10fbd2bd485ee82f1afe764ff8594252b3aaf30 | |
parent | 7d2b0ec345487537e37a24b323d612c7c06295cd (diff) | |
download | bcm5719-llvm-c989993ba1a666f04f7aee7df51d9f4de0588b71.tar.gz bcm5719-llvm-c989993ba1a666f04f7aee7df51d9f4de0588b71.zip |
[XCOFF] Fix link errors from explicit template instantiation
I happen to be using clang-cl+lld-link locally, and I get these link
errors:
lld-link: error: undefined symbol: public: unsigned short __cdecl llvm::object::XCOFFSectionHeader<struct llvm::object::XCOFFSectionHeader64>::getSectionType(void) const
>>> referenced by C:\src\llvm-project\llvm\tools\llvm-readobj\XCOFFDumper.cpp:106
>>> tools\llvm-readobj\CMakeFiles\llvm-readobj.dir\XCOFFDumper.cpp.obj:(public: virtual void __cdecl `anonymous namespace'::XCOFFDumper::printSectionHeaders(void))
I suspect this is because the explicit template instaniation appears
before the inline method definitions in the .cpp file, so they aren't
available at the point of instantiation. Move the explicit instantiation
later.
Also, forward declare the explicit instantiation for good measure.
-rw-r--r-- | llvm/include/llvm/Object/XCOFFObjectFile.h | 6 | ||||
-rw-r--r-- | llvm/lib/Object/XCOFFObjectFile.cpp | 4 |
2 files changed, 10 insertions, 0 deletions
diff --git a/llvm/include/llvm/Object/XCOFFObjectFile.h b/llvm/include/llvm/Object/XCOFFObjectFile.h index 4f80297b04c..fcdbf7a8095 100644 --- a/llvm/include/llvm/Object/XCOFFObjectFile.h +++ b/llvm/include/llvm/Object/XCOFFObjectFile.h @@ -60,6 +60,12 @@ public: bool isReservedSectionType() const; }; +// Explicit extern template declarations. +struct XCOFFSectionHeader32; +struct XCOFFSectionHeader64; +extern template struct XCOFFSectionHeader<XCOFFSectionHeader32>; +extern template struct XCOFFSectionHeader<XCOFFSectionHeader64>; + struct XCOFFSectionHeader32 : XCOFFSectionHeader<XCOFFSectionHeader32> { char Name[XCOFF::NameSize]; support::ubig32_t PhysicalAddress; diff --git a/llvm/lib/Object/XCOFFObjectFile.cpp b/llvm/lib/Object/XCOFFObjectFile.cpp index 5ca2a6a03f0..3ea42bb8b0e 100644 --- a/llvm/lib/Object/XCOFFObjectFile.cpp +++ b/llvm/lib/Object/XCOFFObjectFile.cpp @@ -61,6 +61,10 @@ bool XCOFFSectionHeader<T>::isReservedSectionType() const { return getSectionType() & SectionFlagsReservedMask; } +// Explictly instantiate template classes. +template struct XCOFFSectionHeader<XCOFFSectionHeader32>; +template struct XCOFFSectionHeader<XCOFFSectionHeader64>; + bool XCOFFRelocation32::isRelocationSigned() const { return Info & XR_SIGN_INDICATOR_MASK; } |