159264 2016-06-29 09:00:52 -0700 [GTK] Debug build broken with Clang 2016-07-20 13:20:22 -0700 1 1 1 Unclassified WebKit WebKitGTK WebKit Nightly Build Unspecified Unspecified RESOLVED DUPLICATE 159931 P2 Normal --- 1 clopez webkit-unassigned bugs-noreply cdumez cgarcia commit-queue jfernandez koivisto mcatanzaro pnormand oldest_to_newest 1206392 0 clopez 2016-06-29 09:00:52 -0700 Trying to do a debug build of WebKitGTK+ (on r202623) with Clang and I get this build error: [5825/6148] Linking CXX shared library lib/libwebkit2gtk-4.0.so.37.14.1 FAILED: : && /usr/bin/clang++ -fPIC -Wno-error -std=c++1y -fcolor-diagnostics -gsplit-dwarf -Qunused-arguments -g -Wl,--no-undefined -L/home/clopez/webkit/webkit/WebKitBuild/DependenciesGTK/Root/lib -fuse-ld=gold -Wl,--disable-new-dtags -Wl,--gdb-index -Wl,--gdb-index -shared -Wl,-soname,libwebkit2gtk-4.0.so.37 -o lib/libwebkit2gtk-4.0.so.37.14.1 @CMakeFiles/WebKit2.rsp && : ../../Source/WebKit2/NetworkProcess/cache/NetworkCacheSubresourcesEntry.cpp:76: error: undefined reference to 'WebKit::NetworkCache::Data::~Data()' clang: error: linker command failed with exit code 1 (use -v to see invocation) ninja: build stopped: subcommand failed. However, the same build with GCC (gcc-4.9) works. 1206395 1 282350 jfernandez 2016-06-29 09:12:46 -0700 Created attachment 282350 Preliminary approach This simple change seems to solve the issue, but I guess we should figure out why we need a destructor now and if that's the case, implement it properly instead of just an empty one. 1211371 2 jfernandez 2016-07-15 02:53:48 -0700 The bug is only reproducible using clang versions above 3.5 . 1211413 3 jfernandez 2016-07-15 07:46:21 -0700 I'm not totally sure, but perhaps bug #154885 is related to this one. 1212859 4 mcatanzaro 2016-07-20 13:20:22 -0700 *** This bug has been marked as a duplicate of bug 159931 *** 282350 2016-06-29 09:12:46 -0700 2016-06-29 09:12:46 -0700 Preliminary approach patch.diff text/plain 469 jfernandez ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQyL05ldHdvcmtQcm9jZXNzL2NhY2hlL05ldHdvcmtD YWNoZURhdGEuaCBiL1NvdXJjZS9XZWJLaXQyL05ldHdvcmtQcm9jZXNzL2NhY2hlL05ldHdvcmtD YWNoZURhdGEuaAppbmRleCAyMjU1ODk4Li40ZDhjODgxIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2Vi S2l0Mi9OZXR3b3JrUHJvY2Vzcy9jYWNoZS9OZXR3b3JrQ2FjaGVEYXRhLmgKKysrIGIvU291cmNl L1dlYktpdDIvTmV0d29ya1Byb2Nlc3MvY2FjaGUvTmV0d29ya0NhY2hlRGF0YS5oCkBAIC0xMDUs NiArMTA1LDcgQEAgY2xhc3MgRGF0YSB7CiBwdWJsaWM6CiAgICAgRGF0YSgpIHsgfQogICAgIERh dGEoY29uc3QgdWludDhfdCosIHNpemVfdCk7CisgICAgfkRhdGEoKSB7IH0KIAogICAgIHN0YXRp YyBEYXRhIGVtcHR5KCk7CiAgICAgc3RhdGljIERhdGEgYWRvcHRNYXAodm9pZCogbWFwLCBzaXpl X3QsIGludCBmZCk7Cg==