Closed
Bug 1412267
Opened 7 years ago
Closed 7 years ago
--disable-debug-symbols no longer has any effect
Categories
(Firefox Build System :: General, defect)
Firefox Build System
General
Tracking
(firefox-esr52 unaffected, firefox56 unaffected, firefox57 unaffected, firefox58 fixed)
RESOLVED
FIXED
mozilla58
Tracking | Status | |
---|---|---|
firefox-esr52 | --- | unaffected |
firefox56 | --- | unaffected |
firefox57 | --- | unaffected |
firefox58 | --- | fixed |
People
(Reporter: jbeich, Assigned: chmanchester)
References
Details
(Keywords: regression)
Attachments
(1 file)
I'm trying to build on i386 host, so --disable-debug-symbols is useful to keep memory usage during build low.
$ c++ -v
FreeBSD clang version 3.4.1 (tags/RELEASE_34/dot1-final 208032) 20140512
Target: i386-unknown-freebsd10.3
Thread model: posix
Selected GCC installation:
$ pkg install llvm40
$ clang++40 -v
clang version 4.0.1 (tags/RELEASE_401/final)
Target: i386-portbld-freebsd10.3
Thread model: posix
InstalledDir: /usr/local/llvm40/bin
$ echo "export CC=clang40 CXX=clang++40 CPP=clang-cpp40" >>.mozconfig
$ echo "ac_add_options --disable-debug-symbols" >>.mozconfig
$ ./mach bootstrap
$ ./mach build
[...]
/usr/bin/ld: failed to set dynamic section sizes: Memory exhausted
clang-4.0: error: linker command failed with exit code 1 (use -v to see invocation)
Here's flags used during build:
$ ps axd
PID TT STAT TIME COMMAND
[...]
98159 8 SN+J 0:01.60 | | `-- python2.7 ./mach build
2777 8 IWN+J 0:00.00 | | |-- /usr/local/bin/gmake -f client.mk -s
2843 8 IWN+J 0:00.00 | | | `-- /usr/local/bin/gmake -f /tmp/mozilla-central/client.mk realbuild CREATE_MOZCONFIG_JSON=
2888 8 IWN+J 0:00.00 | | | `-- /usr/local/bin/gmake -j8 -C /tmp/mozilla-central/obj-i386-unknown-freebsd10.3
4536 8 IWN+J 0:00.00 | | | `-- /usr/local/bin/gmake compile
4540 8 SN+J 0:00.04 | | | `-- /usr/local/bin/gmake recurse_compile
4543 8 SN+J 0:00.02 | | | |-- /usr/local/bin/gmake -C js/src target
7387 8 SN+J 0:00.00 | | | | |-- /bin/sh /usr/local/bin/clang++40 -std=gnu++11 -o Unified_cpp_js_src3.o -c -I/tmp/mozilla-central/obj-i386-unknown-freebsd10.3/dist/system_wrappers -include /tmp/mozilla-central/config/gcc_hidden.h -DNDEBUG=1 -DTRIMMED=1 -DENABLE_BINARYDATA -DENABLE_SIMD -DENABLE_WASM_THREAD_OPS -DJS_CACHEIR_SPEW -DENABLE_SHARED_ARRAY_BUFFER -DEXPORT_JS_API -DJS_HAS_CTYPES -DDLL_PREFIX="lib" -DDLL_SUFFIX=".so" -DFFI_BUILDING -DMOZ_HAS_MOZGLUE -I/tmp/mozilla-central/js/src -I/tmp/mozilla-central/obj-i386-unknown-freebsd10.3/js/src -I/tmp/mozilla-central/obj-i386-unknown-freebsd10.3/js/src/ctypes/libffi/include -I/tmp/mozilla-central/js/src/ctypes/libffi/src/x86 -I/tmp/mozilla-central/obj-i386-unknown-freebsd10.3/dist/include -I/tmp/mozilla-central/obj-i386-unknown-freebsd10.3/dist/include/nspr -fPIC -DMOZILLA_CLIENT -include /tmp/mozilla-central/obj-i386-unknown-freebsd10.3/js/src/js-confdefs.h -Qunused-arguments -I/usr/local/include -Qunused-arguments -Wall -Wc++11-compat -Wempty-body -Wignored-qualifiers -Woverloaded-virtual -Wpointer-arith -Wsign-compare -Wtype-limits -Wunreachable-code -Wunreachable-code-return -Wwrite-strings -Wno-invalid-offsetof -Wclass-varargs -Wloop-analysis -Wc++11-compat-pedantic -Wc++14-compat -Wc++14-compat-pedantic -Wc++1z-compat -Wcomma -Wimplicit-fallthrough -Wstring-conversion -Wno-inline-new-delete -Wno-error=deprecated-declarations -Wno-error=array-bounds -Wformat -Wno-gnu-zero-variadic-macro-arguments -Wformat-security -Wno-unknown-warning-option -Wno-return-type-c-linkage -fno-rtti -fno-exceptions -fno-math-errno -pipe -g -O -fno-omit-frame-pointer -Wno-shadow -Werror=format -fno-strict-aliasing -MD -MP -MF .deps/Unified_cpp_js_src3.o.pp -fcolor-diagnostics /tmp/mozilla-central/obj-i386-unknown-freebsd10.3/js/src/Unified_cpp_js_src3.cpp
Notice stray -g from MOZ_DEBUG_FLAGS? Regressed by mozilla-central changeset 3206a73af545.
Comment hidden (obsolete) |
Comment hidden (obsolete) |
Comment 3•7 years ago
|
||
(In reply to Jan Beich from comment #2)
> (In reply to Jan Beich from comment #1)
> > --disable-optimize + CXXFLAGS += -O3 builds also regressed.
>
> Nevermind. Actually passing --disable-optimize rather than assuming it's the
> default behavior seems to help. This seems to match Beta and ESR52.
Just pass --enable-optimize=-O3
Comment 4•7 years ago
|
||
Ah, so that explains why my builders blows again :/
Updated•7 years ago
|
Assignee: nobody → cmanchester
Comment hidden (mozreview-request) |
Comment on attachment 8922879 [details]
Bug 1412267 - Fix the inclusion of MOZ_DEBUG_FLAGS in the compile command line to depend on MOZ_DEBUG_SYMBOLS and MOZ_DEBUG.
I confirm, FreeBSD i386 now builds fine with --disable-debug-symbols.
Attachment #8922879 -
Flags: feedback+
Comment 7•7 years ago
|
||
mozreview-review |
Comment on attachment 8922879 [details]
Bug 1412267 - Fix the inclusion of MOZ_DEBUG_FLAGS in the compile command line to depend on MOZ_DEBUG_SYMBOLS and MOZ_DEBUG.
https://reviewboard.mozilla.org/r/194026/#review199622
I'll r+ this even though I'm not the domain expert because you have this all paged in and know what you're doing.
One thing I'll ask you to consider is implementing this `MOZ_DEBUG or MOZ_DEBUG_SYMBOLS` in configure. That seems like the better layer for this logic. But I'm not sure what implications that could have. It could certainly be done as a follow-up.
Attachment #8922879 -
Flags: review+
Pushed by cmanchester@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/3db37adcebad
Fix the inclusion of MOZ_DEBUG_FLAGS in the compile command line to depend on MOZ_DEBUG_SYMBOLS and MOZ_DEBUG. r=gps
Comment 9•7 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
Assignee | ||
Updated•7 years ago
|
Attachment #8922879 -
Flags: review?(core-build-config-reviews)
Updated•7 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•