1761168 - Perma [tier2] /builds/worker/checkouts/gecko/dom/file/uri/BlobURLProtocolHandler.cpp:315:19: error: variable 'i' set but not used [-Werror,-Wunused-but-set-variable]

Reporter

Description

•

4 years ago

treeherder

Filed by: apavel [at] mozilla.com
Parsed log: https://treeherder.mozilla.org/logviewer?job_id=372142544&repo=mozilla-central
Full log: https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/GbI0tp60Rxig7I_O-3ueXQ/runs/0/artifacts/public/logs/live_backing.log

task 2022-03-24T02:43:58.951Z] 02:43:58     INFO -  dom/fetch/Unified_cpp_dom_fetch1.o
[task 2022-03-24T02:43:58.951Z] 02:43:58     INFO -  gmake[4]: Leaving directory '/builds/worker/workspace/obj-build/dom/fetch'
[task 2022-03-24T02:43:59.095Z] 02:43:59     INFO -  gmake[4]: Entering directory '/builds/worker/workspace/obj-build/config/external/icu/common'
[task 2022-03-24T02:43:59.095Z] 02:43:59     INFO -  /builds/worker/fetches/clang/bin/clang++ --sysroot /builds/worker/fetches/sysroot-x86_64-linux-gnu -std=gnu++17 -o uchar.o -c  -I/builds/worker/workspace/obj-build/dist/system_wrappers -include /builds/worker/checkouts/gecko/config/gcc_hidden.h -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=2 -fstack-protector-strong -fstack-clash-protection -DNDEBUG=1 -DTRIMMED=1 -DU_COMMON_IMPLEMENTATION -DU_USING_ICU_NAMESPACE=0 -DU_NO_DEFAULT_INCLUDE_UTF_HEADERS=1 -DU_HIDE_OBSOLETE_UTF_OLD_H=1 -DUCONFIG_NO_LEGACY_CONVERSION -DUCONFIG_NO_TRANSLITERATION -DUCONFIG_NO_REGULAR_EXPRESSIONS -DU_CHARSET_IS_UTF8 -DUNISTR_FROM_CHAR_EXPLICIT=explicit -DUNISTR_FROM_STRING_EXPLICIT=explicit -DU_ENABLE_DYLOAD=0 -DU_HAVE_NL_LANGINFO_CODESET=0 -I/builds/worker/checkouts/gecko/config/external/icu/common -I/builds/worker/workspace/obj-build/config/external/icu/common -I/builds/worker/checkouts/gecko/intl/icu/source/i18n -I/builds/worker/workspace/obj-build/dist/include -I/builds/worker/workspace/obj-build/dist/include/nspr -I/builds/worker/workspace/obj-build/dist/include/nss -DMOZILLA_CLIENT -include /builds/worker/workspace/obj-build/mozilla-config.h -Qunused-arguments -Qunused-arguments -Wall -Wbitfield-enum-conversion -Wdeprecated-this-capture -Wempty-body -Wformat-type-confusion -Wignored-qualifiers -Wpointer-arith -Wshadow-field-in-constructor-modified -Wsign-compare -Wtype-limits -Wno-error=tautological-type-limit-compare -Wunreachable-code -Wunreachable-code-return -Wunused-but-set-parameter -Wno-invalid-offsetof -Wclass-varargs -Wempty-init-stmt -Wfloat-overflow-conversion -Wfloat-zero-conversion -Wloop-analysis -Wno-range-loop-analysis -Wc++2a-compat -Wcomma -Wenum-compare-conditional -Wimplicit-fallthrough -Werror=non-literal-null-conversion -Wstring-conversion -Wno-inline-new-delete -Wno-error=deprecated-declarations -Wno-error=array-bounds -Wno-error=backend-plugin -Wno-error=free-nonheap-object -Wno-error=return-std-move -Wno-error=atomic-alignment -Wno-error=deprecated-copy -Wformat -Wformat-security -Wno-gnu-zero-variadic-macro-arguments -Wno-psabi -Wthread-safety -Wno-unknown-warning-option -fno-sized-deallocation -fno-aligned-new -fno-exceptions -fPIC -fno-rtti -ffunction-sections -fdata-sections -fno-exceptions -fno-math-errno -pthread -pipe -gdwarf-4 -O2 -fno-omit-frame-pointer -funwind-tables -Wno-deprecated-declarations -Wno-type-limits -Wno-unused-but-set-variable -Wno-unused-function -Wno-sign-compare -Wno-maybe-uninitialized -frtti -Wno-c++20-compat -Wno-comma -Wno-implicit-const-int-float-conversion -Wno-macro-redefined -Wno-microsoft-include -Wno-tautological-unsigned-enum-zero-compare -Wno-unreachable-code-loop-increment -Wno-unreachable-code-return -fno-strict-aliasing  -MD -MP -MF .deps/uchar.o.pp   /builds/worker/checkouts/gecko/intl/icu/source/common/uchar.cpp
[task 2022-03-24T02:43:59.095Z] 02:43:59     INFO -  gmake[4]: Leaving directory '/builds/worker/workspace/obj-build/config/external/icu/common'
[task 2022-03-24T02:43:59.101Z] 02:43:59     INFO -  gmake[4]: Entering directory '/builds/worker/workspace/obj-build/config/external/icu/common'
[task 2022-03-24T02:43:59.101Z] 02:43:59     INFO -  config/external/icu/common/ucharstrie.o
[task 2022-03-24T02:43:59.101Z] 02:43:59     INFO -  gmake[4]: Leaving directory '/builds/worker/workspace/obj-build/config/external/icu/common'
[task 2022-03-24T02:43:59.195Z] 02:43:59     INFO -  gmake[4]: Entering directory '/builds/worker/workspace/obj-build/config/external/icu/i18n'
[task 2022-03-24T02:43:59.195Z] 02:43:59     INFO -  /builds/worker/fetches/clang/bin/clang++ --sysroot /builds/worker/fetches/sysroot-x86_64-linux-gnu -std=gnu++17 -o measunit.o -c  -I/builds/worker/workspace/obj-build/dist/system_wrappers -include /builds/worker/checkouts/gecko/config/gcc_hidden.h -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=2 -fstack-protector-strong -fstack-clash-protection -DNDEBUG=1 -DTRIMMED=1 -DU_I18N_IMPLEMENTATION -DU_USING_ICU_NAMESPACE=0 -DU_NO_DEFAULT_INCLUDE_UTF_HEADERS=1 -DU_HIDE_OBSOLETE_UTF_OLD_H=1 -DUCONFIG_NO_LEGACY_CONVERSION -DUCONFIG_NO_TRANSLITERATION -DUCONFIG_NO_REGULAR_EXPRESSIONS -DU_CHARSET_IS_UTF8 -DUNISTR_FROM_CHAR_EXPLICIT=explicit -DUNISTR_FROM_STRING_EXPLICIT=explicit -DU_ENABLE_DYLOAD=0 -DU_HAVE_NL_LANGINFO_CODESET=0 -I/builds/worker/checkouts/gecko/config/external/icu/i18n -I/builds/worker/workspace/obj-build/config/external/icu/i18n -I/builds/worker/checkouts/gecko/intl/icu/source/common -I/builds/worker/workspace/obj-build/dist/include -I/builds/worker/workspace/obj-build/dist/include/nspr -I/builds/worker/workspace/obj-build/dist/include/nss -DMOZILLA_CLIENT -include /builds/worker/workspace/obj-build/mozilla-config.h -Qunused-arguments -Qunused-arguments -Wall -Wbitfield-enum-conversion -Wdeprecated-this-capture -Wempty-body -Wformat-type-confusion -Wignored-qualifiers -Wpointer-arith -Wshadow-field-in-constructor-modified -Wsign-compare -Wtype-limits -Wno-error=tautological-type-limit-compare -Wunreachable-code -Wunreachable-code-return -Wunused-but-set-parameter -Wno-invalid-offsetof -Wclass-varargs -Wempty-init-stmt -Wfloat-overflow-conversion -Wfloat-zero-conversion -Wloop-analysis -Wno-range-loop-analysis -Wc++2a-compat -Wcomma -Wenum-compare-conditional -Wimplicit-fallthrough -Werror=non-literal-null-conversion -Wstring-conversion -Wno-inline-new-delete -Wno-error=deprecated-declarations -Wno-error=array-bounds -Wno-error=backend-plugin -Wno-error=free-nonheap-object -Wno-error=return-std-move -Wno-error=atomic-alignment -Wno-error=deprecated-copy -Wformat -Wformat-security -Wno-gnu-zero-variadic-macro-arguments -Wno-psabi -Wthread-safety -Wno-unknown-warning-option -fno-sized-deallocation -fno-aligned-new -fno-exceptions -fPIC -fno-rtti -ffunction-sections -fdata-sections -fno-exceptions -fno-math-errno -pthread -pipe -gdwarf-4 -O2 -fno-omit-frame-pointer -funwind-tables -frtti -Wno-c++20-compat -Wno-comma -Wno-implicit-const-int-float-conversion -Wno-macro-redefined -Wno-microsoft-include -Wno-tautological-unsigned-enum-zero-compare -Wno-unreachable-code-loop-increment -Wno-unreachable-code-return -fno-strict-aliasing  -MD -MP -MF .deps/measunit.o.pp   /builds/worker/checkouts/gecko/intl/icu/source/i18n/measunit.cpp
[task 2022-03-24T02:43:59.195Z] 02:43:59     INFO -  gmake[4]: Leaving directory '/builds/worker/workspace/obj-build/config/external/icu/i18n'
[task 2022-03-24T02:43:59.201Z] 02:43:59     INFO -  gmake[4]: Entering directory '/builds/worker/workspace/obj-build/config/external/icu/i18n'
[task 2022-03-24T02:43:59.201Z] 02:43:59     INFO -  config/external/icu/i18n/measunit_extra.o
[task 2022-03-24T02:43:59.201Z] 02:43:59     INFO -  gmake[4]: Leaving directory '/builds/worker/workspace/obj-build/config/external/icu/i18n'
[task 2022-03-24T02:43:59.704Z] 02:43:59     INFO -  gmake[4]: Entering directory '/builds/worker/workspace/obj-build/config/external/icu/common'
[task 2022-03-24T02:43:59.708Z] 02:43:59     INFO -  /builds/worker/fetches/clang/bin/clang++ --sysroot /builds/worker/fetches/sysroot-x86_64-linux-gnu -std=gnu++17 -o ucharstrie.o -c  -I/builds/worker/workspace/obj-build/dist/system_wrappers -include /builds/worker/checkouts/gecko/config/gcc_hidden.h -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=2 -fstack-protector-strong -fstack-clash-protection -DNDEBUG=1 -DTRIMMED=1 -DU_COMMON_IMPLEMENTATION -DU_USING_ICU_NAMESPACE=0 -DU_NO_DEFAULT_INCLUDE_UTF_HEADERS=1 -DU_HIDE_OBSOLETE_UTF_OLD_H=1 -DUCONFIG_NO_LEGACY_CONVERSION -DUCONFIG_NO_TRANSLITERATION -DUCONFIG_NO_REGULAR_EXPRESSIONS -DU_CHARSET_IS_UTF8 -DUNISTR_FROM_CHAR_EXPLICIT=explicit -DUNISTR_FROM_STRING_EXPLICIT=explicit -DU_ENABLE_DYLOAD=0 -DU_HAVE_NL_LANGINFO_CODESET=0 -I/builds/worker/checkouts/gecko/config/external/icu/common -I/builds/worker/workspace/obj-build/config/external/icu/common -I/builds/worker/checkouts/gecko/intl/icu/source/i18n -I/builds/worker/workspace/obj-build/dist/include -I/builds/worker/workspace/obj-build/dist/include/nspr -I/builds/worker/workspace/obj-build/dist/include/nss -DMOZILLA_CLIENT -include /builds/worker/workspace/obj-build/mozilla-config.h -Qunused-arguments -Qunused-arguments -Wall -Wbitfield-enum-conversion -Wdeprecated-this-capture -Wempty-body -Wformat-type-confusion -Wignored-qualifiers -Wpointer-arith -Wshadow-field-in-constructor-modified -Wsign-compare -Wtype-limits -Wno-error=tautological-type-limit-compare -Wunreachable-code -Wunreachable-code-return -Wunused-but-set-parameter -Wno-invalid-offsetof -Wclass-varargs -Wempty-init-stmt -Wfloat-overflow-conversion -Wfloat-zero-conversion -Wloop-analysis -Wno-range-loop-analysis -Wc++2a-compat -Wcomma -Wenum-compare-conditional -Wimplicit-fallthrough -Werror=non-literal-null-conversion -Wstring-conversion -Wno-inline-new-delete -Wno-error=deprecated-declarations -Wno-error=array-bounds -Wno-error=backend-plugin -Wno-error=free-nonheap-object -Wno-error=return-std-move -Wno-error=atomic-alignment -Wno-error=deprecated-copy -Wformat -Wformat-security -Wno-gnu-zero-variadic-macro-arguments -Wno-psabi -Wthread-safety -Wno-unknown-warning-option -fno-sized-deallocation -fno-aligned-new -fno-exceptions -fPIC -fno-rtti -ffunction-sections -fdata-sections -fno-exceptions -fno-math-errno -pthread -pipe -gdwarf-4 -O2 -fno-omit-frame-pointer -funwind-tables -Wno-deprecated-declarations -Wno-type-limits -Wno-unused-but-set-variable -Wno-unused-function -Wno-sign-compare -Wno-maybe-uninitialized -frtti -Wno-c++20-compat -Wno-comma -Wno-implicit-const-int-float-conversion -Wno-macro-redefined -Wno-microsoft-include -Wno-tautological-unsigned-enum-zero-compare -Wno-unreachable-code-loop-increment -Wno-unreachable-code-return -fno-strict-aliasing  -MD -MP -MF .deps/ucharstrie.o.pp   /builds/worker/checkouts/gecko/intl/icu/source/common/ucharstrie.cpp
[task 2022-03-24T02:43:59.708Z] 02:43:59     INFO -  gmake[4]: Leaving directory '/builds/worker/workspace/obj-build/config/external/icu/common'
[task 2022-03-24T02:43:59.708Z] 02:43:59     INFO -  gmake[4]: Entering directory '/builds/worker/workspace/obj-build/config/external/icu/common'
[task 2022-03-24T02:43:59.708Z] 02:43:59     INFO -  config/external/icu/common/ucharstriebuilder.o
[task 2022-03-24T02:43:59.708Z] 02:43:59     INFO -  gmake[4]: Leaving directory '/builds/worker/workspace/obj-build/config/external/icu/common'
[task 2022-03-24T02:44:00.332Z] 02:44:00     INFO -  gmake[4]: Entering directory '/builds/worker/workspace/obj-build/dom/file/uri'
[task 2022-03-24T02:44:00.336Z] 02:44:00     INFO -  /builds/worker/fetches/clang/bin/clang++ --sysroot /builds/worker/fetches/sysroot-x86_64-linux-gnu -std=gnu++17 -o Unified_cpp_dom_file_uri0.o -c  -I/builds/worker/workspace/obj-build/dist/stl_wrappers -I/builds/worker/workspace/obj-build/dist/system_wrappers -include /builds/worker/checkouts/gecko/config/gcc_hidden.h -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=2 -fstack-protector-strong -fstack-clash-protection -DNDEBUG=1 -DTRIMMED=1 -DOS_POSIX=1 -DOS_LINUX=1 -DMOZ_HAS_MOZGLUE -DMOZILLA_INTERNAL_API -DIMPL_LIBXUL -DSTATIC_EXPORTABLE_JS_API -I/builds/worker/checkouts/gecko/dom/file/uri -I/builds/worker/workspace/obj-build/dom/file/uri -I/builds/worker/checkouts/gecko/dom/file -I/builds/worker/checkouts/gecko/netwerk/base -I/builds/worker/workspace/obj-build/ipc/ipdl/_ipdlheaders -I/builds/worker/checkouts/gecko/ipc/chromium/src -I/builds/worker/workspace/obj-build/dist/include -I/builds/worker/workspace/obj-build/dist/include/nspr -I/builds/worker/workspace/obj-build/dist/include/nss -DMOZILLA_CLIENT -include /builds/worker/workspace/obj-build/mozilla-config.h -Qunused-arguments -Qunused-arguments -Wall -Wbitfield-enum-conversion -Wdeprecated-this-capture -Wempty-body -Wformat-type-confusion -Wignored-qualifiers -Wpointer-arith -Wshadow-field-in-constructor-modified -Wsign-compare -Wtype-limits -Wno-error=tautological-type-limit-compare -Wunreachable-code -Wunreachable-code-return -Wunused-but-set-parameter -Wno-invalid-offsetof -Wclass-varargs -Wempty-init-stmt -Wfloat-overflow-conversion -Wfloat-zero-conversion -Wloop-analysis -Wno-range-loop-analysis -Wc++2a-compat -Wcomma -Wenum-compare-conditional -Wimplicit-fallthrough -Werror=non-literal-null-conversion -Wstring-conversion -Wno-inline-new-delete -Wno-error=deprecated-declarations -Wno-error=array-bounds -Wno-error=backend-plugin -Wno-error=free-nonheap-object -Wno-error=return-std-move -Wno-error=atomic-alignment -Wno-error=deprecated-copy -Wformat -Wformat-security -Wno-gnu-zero-variadic-macro-arguments -Wno-psabi -Wthread-safety -Wno-unknown-warning-option -fno-sized-deallocation -fno-aligned-new -fno-exceptions -fPIC -fno-rtti -ffunction-sections -fdata-sections -fno-exceptions -fno-math-errno -pthread -pipe -gdwarf-4 -O2 -fno-omit-frame-pointer -funwind-tables -Werror -fno-strict-aliasing  -MD -MP -MF .deps/Unified_cpp_dom_file_uri0.o.pp   Unified_cpp_dom_file_uri0.cpp
[task 2022-03-24T02:44:00.336Z] 02:44:00     INFO -  In file included from Unified_cpp_dom_file_uri0.cpp:29:
[task 2022-03-24T02:44:00.336Z] 02:44:00    ERROR -  /builds/worker/checkouts/gecko/dom/file/uri/BlobURLProtocolHandler.cpp:315:19: error: variable 'i' set but not used [-Werror,-Wunused-but-set-variable]
[task 2022-03-24T02:44:00.336Z] 02:44:00     INFO -      for (uint32_t i = 0; frame; ++i) {
[task 2022-03-24T02:44:00.336Z] 02:44:00     INFO -                    ^
[task 2022-03-24T02:44:00.336Z] 02:44:00     INFO -  1 error generated.
[task 2022-03-24T02:44:00.336Z] 02:44:00    ERROR -  gmake[4]: *** [/builds/worker/checkouts/gecko/config/rules.mk:660: Unified_cpp_dom_file_uri0.o] Error 1
[task 2022-03-24T02:44:00.336Z] 02:44:00     INFO -  gmake[4]: Leaving directory '/builds/worker/workspace/obj-build/dom/file/uri'
[task 2022-03-24T02:44:00.336Z] 02:44:00    ERROR -  gmake[3]: *** [/builds/worker/checkouts/gecko/config/recurse.mk:72: dom/file/uri/target-objects] Error 2
[task 2022-03-24T02:44:00.336Z] 02:44:00     INFO -  gmake[3]: *** Waiting for unfinished jobs....
[task 2022-03-24T02:44:00.336Z] 02:44:00     INFO -  gmake[4]: Entering directory '/builds/worker/workspace/obj-build/config/external/icu/common'
[task 2022-03-24T02:44:00.336Z] 02:44:00     INFO -  config/external/icu/common/ucharstrieiterator.o
[task 2022-03-24T02:44:00.336Z] 02:44:00     INFO -  gmake[4]: Leaving directory '/builds/worker/workspace/obj-build/config/external/icu/common'
[task 2022-03-24T02:44:00.393Z] 02:44:00     INFO -  gmake[4]: Entering directory '/builds/worker/workspace/obj-build/config/external/icu/common'
[task 2022-03-24T02:44:00.393Z] 02:44:00     INFO -  /builds/worker/fetches/clang/bin/clang++ --sysroot /builds/worker/fetches/sysroot-x86_64-linux-gnu -std=gnu++17 -o ucharstriebuilder.o -c  -I/builds/worker/workspace/obj-build/dist/system_wrappers -include /builds/worker/checkouts/gecko/config/gcc_hidden.h -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=2 -fstack-protector-strong -fstack-clash-protection -DNDEBUG=1 -DTRIMMED=1 -DU_COMMON_IMPLEMENTATION -DU_USING_ICU_NAMESPACE=0 -DU_NO_DEFAULT_INCLUDE_UTF_HEADERS=1 -DU_HIDE_OBSOLETE_UTF_OLD_H=1 -DUCONFIG_NO_LEGACY_CONVERSION -DUCONFIG_NO_TRANSLITERATION -DUCONFIG_NO_REGULAR_EXPRESSIONS -DU_CHARSET_IS_UTF8 -DUNISTR_FROM_CHAR_EXPLICIT=explicit -DUNISTR_FROM_STRING_EXPLICIT=explicit -DU_ENABLE_DYLOAD=0 -DU_HAVE_NL_LANGINFO_CODESET=0 -I/builds/worker/checkouts/gecko/config/external/icu/common -I/builds/worker/workspace/obj-build/config/external/icu/common -I/builds/worker/checkouts/gecko/intl/icu/source/i18n -I/builds/worker/workspace/obj-build/dist/include -I/builds/worker/workspace/obj-build/dist/include/nspr -I/builds/worker/workspace/obj-build/dist/include/nss -DMOZILLA_CLIENT -include /builds/worker/workspace/obj-build/mozilla-config.h -Qunused-arguments -Qunused-arguments -Wall -Wbitfield-enum-conversion -Wdeprecated-this-capture -Wempty-body -Wformat-type-confusion -Wignored-qualifiers -Wpointer-arith -Wshadow-field-in-constructor-modified -Wsign-compare -Wtype-limits -Wno-error=tautological-type-limit-compare -Wunreachable-code -Wunreachable-code-return -Wunused-but-set-parameter -Wno-invalid-offsetof -Wclass-varargs -Wempty-init-stmt -Wfloat-overflow-conversion -Wfloat-zero-conversion -Wloop-analysis -Wno-range-loop-analysis -Wc++2a-compat -Wcomma -Wenum-compare-conditional -Wimplicit-fallthrough -Werror=non-literal-null-conversion -Wstring-conversion -Wno-inline-new-delete -Wno-error=deprecated-declarations -Wno-error=array-bounds -Wno-error=backend-plugin -Wno-error=free-nonheap-object -Wno-error=return-std-move -Wno-error=atomic-alignment -Wno-error=deprecated-copy -Wformat -Wformat-security -Wno-gnu-zero-variadic-macro-arguments -Wno-psabi -Wthread-safety -Wno-unknown-warning-option -fno-sized-deallocation -fno-aligned-new -fno-exceptions -fPIC -fno-rtti -ffunction-sections -fdata-sections -fno-exceptions -fno-math-errno -pthread -pipe -gdwarf-4 -O2 -fno-omit-frame-pointer -funwind-tables -Wno-deprecated-declarations -Wno-type-limits -Wno-unused-but-set-variable -Wno-unused-function -Wno-sign-compare -Wno-maybe-uninitialized -frtti -Wno-c++20-compat -Wno-comma -Wno-implicit-const-int-float-conversion -Wno-macro-redefined -Wno-microsoft-include -Wno-tautological-unsigned-enum-zero-compare -Wno-unreachable-code-loop-increment -Wno-unreachable-code-return -fno-strict-aliasing  -MD -MP -MF .deps/ucharstriebuilder.o.pp   /builds/worker/checkouts/gecko/intl/icu/source/common/ucharstriebuilder.cpp
[task 2022-03-24T02:44:00.393Z] 02:44:00     INFO -  gmake[4]: Leaving directory '/builds/worker/workspace/obj-build/config/external/icu/common'
[task 2022-03-24T02:44:00.397Z] 02:44:00     INFO -  gmake[4]: Entering directory '/builds/worker/workspace/obj-build/config/external/icu/common'
[task 2022-03-24T02:44:00.397Z] 02:44:00     INFO -  config/external/icu/common/uchriter.o
[task 2022-03-24T02:44:00.397Z] 02:44:00     INFO -  gmake[4]: Leaving directory '/builds/worker/workspace/obj-build/config/external/icu/common'
[task 2022-03-24T02:44:00.458Z] 02:44:00     INFO -  gmake[4]: Entering directory '/builds/worker/workspace/obj-build/config/external/icu/i18n'
[task 2022-03-24T02:44:00.458Z] 02:44:00     INFO -  /builds/worker/fetches/clang/bin/clang++ --sysroot /builds/worker/fetches/sysroot-x86_64-linux-gnu -std=gnu++17 -o measunit_extra.o -c  -I/builds/worker/workspace/obj-build/dist/system_wrappers -include /builds/worker/checkouts/gecko/config/gcc_hidden.h -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=2 -fstack-protector-strong -fstack-clash-protection -DNDEBUG=1 -DTRIMMED=1 -DU_I18N_IMPLEMENTATION -DU_USING_ICU_NAMESPACE=0 -DU_NO_DEFAULT_INCLUDE_UTF_HEADERS=1 -DU_HIDE_OBSOLETE_UTF_OLD_H=1 -DUCONFIG_NO_LEGACY_CONVERSION -DUCONFIG_NO_TRANSLITERATION -DUCONFIG_NO_REGULAR_EXPRESSIONS -DU_CHARSET_IS_UTF8 -DUNISTR_FROM_CHAR_EXPLICIT=explicit -DUNISTR_FROM_STRING_EXPLICIT=explicit -DU_ENABLE_DYLOAD=0 -DU_HAVE_NL_LANGINFO_CODESET=0 -I/builds/worker/checkouts/gecko/config/external/icu/i18n -I/builds/worker/workspace/obj-build/config/external/icu/i18n -I/builds/worker/checkouts/gecko/intl/icu/source/common -I/builds/worker/workspace/obj-build/dist/include -I/builds/worker/workspace/obj-build/dist/include/nspr -I/builds/worker/workspace/obj-build/dist/include/nss -DMOZILLA_CLIENT -include /builds/worker/workspace/obj-build/mozilla-config.h -Qunused-arguments -Qunused-arguments -Wall -Wbitfield-enum-conversion -Wdeprecated-this-capture -Wempty-body -Wformat-type-confusion -Wignored-qualifiers -Wpointer-arith -Wshadow-field-in-constructor-modified -Wsign-compare -Wtype-limits -Wno-error=tautological-type-limit-compare -Wunreachable-code -Wunreachable-code-return -Wunused-but-set-parameter -Wno-invalid-offsetof -Wclass-varargs -Wempty-init-stmt -Wfloat-overflow-conversion -Wfloat-zero-conversion -Wloop-analysis -Wno-range-loop-analysis -Wc++2a-compat -Wcomma -Wenum-compare-conditional -Wimplicit-fallthrough -Werror=non-literal-null-conversion -Wstring-conversion -Wno-inline-new-delete -Wno-error=deprecated-declarations -Wno-error=array-bounds -Wno-error=backend-plugin -Wno-error=free-nonheap-object -Wno-error=return-std-move -Wno-error=atomic-alignment -Wno-error=deprecated-copy -Wformat -Wformat-security -Wno-gnu-zero-variadic-macro-arguments -Wno-psabi -Wthread-safety -Wno-unknown-warning-option -fno-sized-deallocation -fno-aligned-new -fno-exceptions -fPIC -fno-rtti -ffunction-sections -fdata-sections -fno-exceptions -fno-math-errno -pthread -pipe -gdwarf-4 -O2 -fno-omit-frame-pointer -funwind-tables -frtti -Wno-c++20-compat -Wno-comma -Wno-implicit-const-int-float-conversion -Wno-macro-redefined -Wno-microsoft-include -Wno-tautological-unsigned-enum-zero-compare -Wno-unreachable-code-loop-increment -Wno-unreachable-code-return -fno-strict-aliasing  -MD -MP -MF .deps/measunit_extra.o.pp   /builds/worker/checkouts/gecko/intl/icu/source/i18n/measunit_extra.cpp
[task 2022-03-24T02:44:00.458Z] 02:44:00     INFO -  gmake[4]: Leaving directory '/builds/worker/workspace/obj-build/config/external/icu/i18n'
[task 2022-03-24T02:44:00.461Z] 02:44:00     INFO -  gmake[4]: Entering directory '/builds/worker/workspace/obj-build/config/external/icu/i18n'
[task 2022-03-24T02:44:00.462Z] 02:44:00     INFO -  config/external/icu/i18n/measure.o
[task 2022-03-24T02:44:00.462Z] 02:44:00     INFO -  gmake[4]: Leaving directory '/builds/worker/workspace/obj-build/config/external/icu/i18n'
[task 2022-03-24T02:44:00.511Z] 02:44:00     INFO -  gmake[4]: Entering directory '/builds/worker/workspace/obj-build/dom/cache'
[task 2022-03-24T02:44:00.512Z] 02:44:00     INFO -  /builds/worker/fetches/clang/bin/clang++ --sysroot /builds/worker/fetches/sysroot-x86_64-linux-gnu -std=gnu++17 -o Unified_cpp_dom_cache1.o -c  -I/builds/worker/workspace/obj-build/dist/stl_wrappers -I/builds/worker/workspace/obj-build/dist/system_wrappers -include /builds/worker/checkouts/gecko/config/gcc_hidden.h -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=2 -fstack-protector-strong -fstack-clash-protection -DNDEBUG=1 -DTRIMMED=1 -DOS_POSIX=1 -DOS_LINUX=1 -DMOZ_HAS_MOZGLUE -DMOZILLA_INTERNAL_API -DIMPL_LIBXUL -DSTATIC_EXPORTABLE_JS_API -I/builds/worker/checkouts/gecko/dom/cache -I/builds/worker/workspace/obj-build/dom/cache -I/builds/worker/workspace/obj-build/ipc/ipdl/_ipdlheaders -I/builds/worker/checkouts/gecko/ipc/chromium/src -I/builds/worker/workspace/obj-build/dist/include -I/builds/worker/workspace/obj-build/dist/include/nspr -I/builds/worker/workspace/obj-build/dist/include/nss -DMOZILLA_CLIENT -include /builds/worker/workspace/obj-build/mozilla-config.h -Qunused-arguments -Qunused-arguments -Wall -Wbitfield-enum-conversion -Wdeprecated-this-capture -Wempty-body -Wformat-type-confusion -Wignored-qualifiers -Wpointer-arith -Wshadow-field-in-constructor-modified -Wsign-compare -Wtype-limits -Wno-error=tautological-type-limit-compare -Wunreachable-code -Wunreachable-code-return -Wunused-but-set-parameter -Wno-invalid-offsetof -Wclass-varargs -Wempty-init-stmt -Wfloat-overflow-conversion -Wfloat-zero-conversion -Wloop-analysis -Wno-range-loop-analysis -Wc++2a-compat -Wcomma -Wenum-compare-conditional -Wimplicit-fallthrough -Werror=non-literal-null-conversion -Wstring-conversion -Wno-inline-new-delete -Wno-error=deprecated-declarations -Wno-error=array-bounds -Wno-error=backend-plugin -Wno-error=free-nonheap-object -Wno-error=return-std-move -Wno-error=atomic-alignment -Wno-error=deprecated-copy -Wformat -Wformat-security -Wno-gnu-zero-variadic-macro-arguments -Wno-psabi -Wthread-safety -Wno-unknown-warning-option -fno-sized-deallocation -fno-aligned-new -fno-exceptions -fPIC -fno-rtti -ffunction-sections -fdata-sections -fno-exceptions -fno-math-errno -pthread -pipe -gdwarf-4 -O2 -fno-omit-frame-pointer -funwind-tables -Werror -fno-strict-aliasing  -MD -MP -MF .deps/Unified_cpp_dom_cache1.o.pp   Unified_cpp_dom_cache1.cpp
[task 2022-03-24T02:44:00.512Z] 02:44:00     INFO -  gmake[4]: Leaving directory '/builds/worker/workspace/obj-build/dom/cache'
[task 2022-03-24T02:44:00.517Z] 02:44:00     INFO -  gmake[4]: Entering directory '/builds/worker/workspace/obj-build/dom/cache'
[task 2022-03-24T02:44:00.517Z] 02:44:00     INFO -  dom/cache/Unified_cpp_dom_cache2.o

Andreea Pavel [:apavel]

Comment 1

•

4 years ago

Hi Randell, could this be a regression of bug 1207753?

Flags: needinfo?(rjesup)

Randell Jesup [:jesup] (needinfo me)

Comment 2

•

4 years ago

No, it really can't be

Flags: needinfo?(rjesup)

Natalia Csoregi [:nataliaCs]

Comment 3

•

4 years ago

Might have been caused by something within this merge: https://hg.mozilla.org/mozilla-central/pushloghtml?changeset=10178bda5cdd23c669578c163c350adf728a08a8

Andi, can you take a look at this please?
Thank you.

Flags: needinfo?(bpostelnicu)

Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)

Comment 4

•

4 years ago

Likely clang updated to a later version which doesn't require a change in mozilla-central because it always uses the latest version of that external repository.

Flags: needinfo?(bpostelnicu) → needinfo?(mh+mozilla)

Mike Hommey [:glandium]

Comment 5

•

4 years ago

This is a compiler check that was improved in clang trunk and that found a variable that hasn't been used since bug 1257919. Practically speaking, that for (uint32_t i = 0; frame; ++i) is equivalent to while (frame). The question would be whether that's actually correct.

Component: General → DOM: Core & HTML

Flags: needinfo?(mh+mozilla) → needinfo?(htsai)

Product: Firefox Build System → Core

Regressed by: 1257919

BugBot [:suhaib / :marco]

Comment 6

•

4 years ago

Set release status flags based on info from the regressing bug 1257919

status-firefox100: --- → affected

status-firefox98: --- → affected

status-firefox99: --- → affected

status-firefox-esr91: --- → affected

Hsin-Yi Tsai (she/her) [:hsinyi]

Updated

•

4 years ago

Component: DOM: Core & HTML → DOM: File

Flags: needinfo?(htsai) → needinfo?(jstutte)

Comment hidden (Intermittent Failures Robot)

BugBot [:suhaib / :marco]

Updated

•

4 years ago

Has Regression Range: --- → yes

Jens Stutte [:jstutte]

Assignee

Comment 8

•

4 years ago

(In reply to Mike Hommey [:glandium] from comment #5)

This is a compiler check that was improved in clang trunk and that found a variable that hasn't been used since bug 1257919. Practically speaking, that for (uint32_t i = 0; frame; ++i) is equivalent to while (frame). The question would be whether that's actually correct.

So it seems that Stop returning StackFrame instances from exceptions::CreateStack. C++ callers of GetCurrentJSStack or exceptions::CreateStack always check for null anyway, and none of them seem to want this non-JS thing did remove the check:

-    for (uint32_t i = 0; i < maxFrames && frame; ++i) {
+    for (uint32_t i = 0; frame; ++i) {

as we can use now dom::GetCurrentJSStack(maxFrames); which automatically limits the stack depth for us. So I'd assume, a simple while (frame) is the right call here.

Flags: needinfo?(jstutte)

Jens Stutte [:jstutte]

Assignee

Comment 9

•

4 years ago

Attached file Bug 1761168: Transform useless for into while loop. r?#dom-storage-reviewers — Details

Phabricator Automation

Updated

•

4 years ago

Assignee: nobody → jstutte

Status: NEW → ASSIGNED

Pulsebot

Comment 10

•

4 years ago

Pushed by jstutte@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/545fdd2ff252 Transform useless for into while loop. r=dom-storage-reviewers,jari

Cristian Tuns

Comment 11

•

4 years ago

bugherder

https://hg.mozilla.org/mozilla-central/rev/545fdd2ff252

Status: ASSIGNED → RESOLVED

Closed: 4 years ago

status-firefox100: affected → fixed

Resolution: --- → FIXED

Target Milestone: --- → 100 Branch

BugBot [:suhaib / :marco]

Updated

•

4 years ago

Keywords: regression

BugBot [:suhaib / :marco]

Comment 12

•

4 years ago

The patch landed in nightly and beta is affected.
:jstutte, is this bug important enough to require an uplift?
If not please set status_beta to wontfix.

For more information, please visit auto_nag documentation.

Flags: needinfo?(jstutte)

Jens Stutte [:jstutte]

Assignee

Comment 13

•

4 years ago

Hi :aryx, do you happen to know if this is relevant for beta?

Flags: needinfo?(jstutte) → needinfo?(aryx.bugmail)

Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)

Comment 14

•

4 years ago

The failure affects clang-trunk and we don't run this task outside trunk. Nothing left to do.

status-firefox98: affected → wontfix

status-firefox99: affected → wontfix

status-firefox-esr91: affected → wontfix

Flags: needinfo?(aryx.bugmail)

Comment hidden (Intermittent Failures Robot)