51669 2010-12-27 23:29:33 -0800 gcc compile error 'control reaches end of non-void function' with some option 2010-12-28 14:14:57 -0800 1 1 1 Unclassified WebKit WebCore Misc. 528+ (Nightly build) All All RESOLVED FIXED P3 Trivial --- 0 toyoshim webkit-unassigned ap commit-queue oldest_to_newest 327198 0 toyoshim 2010-12-27 23:29:33 -0800 gcc detects 'control reaches end of non-void function' error in case of using '-finstrument-functions' option. This problem appears in a debug configuration with heap leak checker of chromium. (http://codereview.chromium.org/5991005/) 327199 1 77540 toyoshim 2010-12-27 23:41:56 -0800 Created attachment 77540 Patch 327294 2 ap 2010-12-28 11:43:03 -0800 Looks like a gcc bug. Which gcc version are you seeing this with? 327344 3 77540 eric 2010-12-28 13:51:37 -0800 Comment on attachment 77540 Patch OK. 327370 4 77540 commit-queue 2010-12-28 14:14:51 -0800 Comment on attachment 77540 Patch Clearing flags on attachment: 77540 Committed r74725: <http://trac.webkit.org/changeset/74725> 327371 5 commit-queue 2010-12-28 14:14:57 -0800 All reviewed patches have been landed. Closing bug. 77540 2010-12-27 23:41:56 -0800 2010-12-28 14:14:51 -0800 Patch bug-51669-20101228164153.patch text/plain 2586 toyoshim SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n CShyZXZpc2lvbiA3NDcwOCkKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA IC0xLDMgKzEsMTggQEAKKzIwMTAtMTItMjcgIFRha2FzaGkgVG95b3NoaW1hICA8dG95b3NoaW1A Z29vZ2xlLmNvbT4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAg ICAgICBnY2MgZGV0ZWN0ZWQgJ2NvbnRyb2wgcmVhY2hlcyBlbmQgb2Ygbm9uLXZvaWQgZnVuY3Rp b24nIHdpdGgKKyAgICAgICAgLWZpbnN0cnVtZW50LWZ1bmN0aW9ucyBvcHRpb24uCisgICAgICAg IGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD01MTY2OQorCisgICAgICAg IE5vIG5ldyB0ZXN0cy4gVGhlc2UgZml4ZXMgYXJlIHRyaXZpYWwuCisKKyAgICAgICAgKiBkb20v RWxlbWVudC5jcHA6CisgICAgICAgIChXZWJDb3JlOjpFbGVtZW50OjpjaGlsZFR5cGVBbGxvd2Vk KToKKyAgICAgICAgKiBpbnNwZWN0b3IvSW5zcGVjdG9yUmVzb3VyY2VBZ2VudC5jcHA6CisgICAg ICAgIChXZWJDb3JlOjpjYWNoZWRSZXNvdXJjZVR5cGVTdHJpbmcpOgorCiAyMDEwLTEyLTI3ICBI ZWxkZXIgQ29ycmVpYSAgPGhlbGRlckBzZW5jaGEuY29tPgogCiAgICAgICAgIFJldmlld2VkIGJ5 IEFyaXlhIEhpZGF5YXQuCkluZGV4OiBXZWJDb3JlL2RvbS9FbGVtZW50LmNwcAo9PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 Ci0tLSBXZWJDb3JlL2RvbS9FbGVtZW50LmNwcAkocmV2aXNpb24gNzQ2ODApCisrKyBXZWJDb3Jl L2RvbS9FbGVtZW50LmNwcAkod29ya2luZyBjb3B5KQpAQCAtMTEwNSwxNyArMTEwNSwxNyBAQCB2 b2lkIEVsZW1lbnQ6OmNsZWFyU2hhZG93Um9vdCgpCiBib29sIEVsZW1lbnQ6OmNoaWxkVHlwZUFs bG93ZWQoTm9kZVR5cGUgdHlwZSkKIHsKICAgICBzd2l0Y2ggKHR5cGUpIHsKLSAgICAgICAgY2Fz ZSBFTEVNRU5UX05PREU6Ci0gICAgICAgIGNhc2UgVEVYVF9OT0RFOgotICAgICAgICBjYXNlIENP TU1FTlRfTk9ERToKLSAgICAgICAgY2FzZSBQUk9DRVNTSU5HX0lOU1RSVUNUSU9OX05PREU6Ci0g ICAgICAgIGNhc2UgQ0RBVEFfU0VDVElPTl9OT0RFOgotICAgICAgICBjYXNlIEVOVElUWV9SRUZF UkVOQ0VfTk9ERToKLSAgICAgICAgICAgIHJldHVybiB0cnVlOwotICAgICAgICAgICAgYnJlYWs7 Ci0gICAgICAgIGRlZmF1bHQ6Ci0gICAgICAgICAgICByZXR1cm4gZmFsc2U7CisgICAgY2FzZSBF TEVNRU5UX05PREU6CisgICAgY2FzZSBURVhUX05PREU6CisgICAgY2FzZSBDT01NRU5UX05PREU6 CisgICAgY2FzZSBQUk9DRVNTSU5HX0lOU1RSVUNUSU9OX05PREU6CisgICAgY2FzZSBDREFUQV9T RUNUSU9OX05PREU6CisgICAgY2FzZSBFTlRJVFlfUkVGRVJFTkNFX05PREU6CisgICAgICAgIHJl dHVybiB0cnVlOworICAgIGRlZmF1bHQ6CisgICAgICAgIGJyZWFrOwogICAgIH0KKyAgICByZXR1 cm4gZmFsc2U7CiB9CiAKIHN0YXRpYyB2b2lkIGNoZWNrRm9yU2libGluZ1N0eWxlQ2hhbmdlcyhF bGVtZW50KiBlLCBSZW5kZXJTdHlsZSogc3R5bGUsIGJvb2wgZmluaXNoZWRQYXJzaW5nQ2FsbGJh Y2ssCkluZGV4OiBXZWJDb3JlL2luc3BlY3Rvci9JbnNwZWN0b3JSZXNvdXJjZUFnZW50LmNwcAo9 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09Ci0tLSBXZWJDb3JlL2luc3BlY3Rvci9JbnNwZWN0b3JSZXNvdXJjZUFnZW50LmNw cAkocmV2aXNpb24gNzQ2ODApCisrKyBXZWJDb3JlL2luc3BlY3Rvci9JbnNwZWN0b3JSZXNvdXJj ZUFnZW50LmNwcAkod29ya2luZyBjb3B5KQpAQCAtMjM3LDcgKzIzNyw2IEBAIHN0YXRpYyBTdHJp bmcgY2FjaGVkUmVzb3VyY2VUeXBlU3RyaW5nKGMKICAgICBzd2l0Y2ggKGNhY2hlZFJlc291cmNl LnR5cGUoKSkgewogICAgIGNhc2UgQ2FjaGVkUmVzb3VyY2U6OkltYWdlUmVzb3VyY2U6CiAgICAg ICAgIHJldHVybiAiSW1hZ2UiOwotICAgICAgICBicmVhazsKICAgICBjYXNlIENhY2hlZFJlc291 cmNlOjpGb250UmVzb3VyY2U6CiAgICAgICAgIHJldHVybiAiRm9udCI7CiAgICAgY2FzZSBDYWNo ZWRSZXNvdXJjZTo6Q1NTU3R5bGVTaGVldDoKQEAgLTI0OSw4ICsyNDgsOSBAQCBzdGF0aWMgU3Ry aW5nIGNhY2hlZFJlc291cmNlVHlwZVN0cmluZyhjCiAgICAgY2FzZSBDYWNoZWRSZXNvdXJjZTo6 U2NyaXB0OgogICAgICAgICByZXR1cm4gIlNjcmlwdCI7CiAgICAgZGVmYXVsdDoKLSAgICAgICAg cmV0dXJuICJPdGhlciI7CisgICAgICAgIGJyZWFrOwogICAgIH0KKyAgICByZXR1cm4gIk90aGVy IjsKIH0KIAogc3RhdGljIFBhc3NSZWZQdHI8SW5zcGVjdG9yT2JqZWN0PiBidWlsZE9iamVjdEZv ckNhY2hlZFJlc291cmNlKERvY3VtZW50TG9hZGVyKiBsb2FkZXIsIGNvbnN0IENhY2hlZFJlc291 cmNlJiBjYWNoZWRSZXNvdXJjZSkK