This is an archive of the discontinued LLVM Phabricator instance.

Fix: ClangHighlighter.cpp should not be in CopyFiles, but in lldb-core targets
AbandonedPublic

Authored by sgraenitz on Aug 2 2018, 9:12 AM.

Details

Summary

Fix linker error:
Undefined symbols for architecture x86_64:

"lldb_private::ClangHighlighter::ClangHighlighter()", referenced from:
    lldb_private::ObjCLanguage::ObjCLanguage() in liblldb-core.a(ObjCLanguage.o)
    lldb_private::CPlusPlusLanguage::CPlusPlusLanguage() in liblldb-core.a(CPlusPlusLanguage.o)
    lldb_private::ObjCPlusPlusLanguage::ObjCPlusPlusLanguage() in liblldb-core.a(ObjCPlusPlusLanguage.o)
"vtable for lldb_private::ClangHighlighter", referenced from:
    lldb_private::ClangHighlighter::~ClangHighlighter() in liblldb-core.a(ObjCLanguage.o)
    lldb_private::ClangHighlighter::~ClangHighlighter() in liblldb-core.a(CPlusPlusLanguage.o)
    lldb_private::ClangHighlighter::~ClangHighlighter() in liblldb-core.a(ObjCPlusPlusLanguage.o)

Event Timeline

sgraenitz created this revision.Aug 2 2018, 9:12 AM

This was changed by rLLDB338712 so I think Tim probably knows better what's going on here.

davide added a subscriber: davide.Aug 2 2018, 9:16 AM

My understanding is that @t.p.northover just committed the same patch.