On 25 Aug 2017, at 20:07, Ryan Libby <rli...@gmail.com> wrote: > > On Wed, Aug 23, 2017 at 4:30 PM, John Baldwin <j...@freebsd.org> wrote: >> Author: jhb >> Date: Wed Aug 23 23:30:25 2017 >> New Revision: 322824 >> URL: https://svnweb.freebsd.org/changeset/base/322824 >> >> Log: >> Improve the coverage of debug symbols for MK_DEBUG_FILES. >> >> - Include debug symbols in static libraries. This permits binaries >> to include debug symbols for functions obtained from static libraries. >> - Permit the C/C++ compiler flags added for MK_DEBUG_FILES to be >> overridden by setting DEBUG_FILES_CFLAGS. Use this to limit the debug >> information for llvm libraries and binaries. >> >> Reviewed by: emaste >> Sponsored by: DARPA / AFRL >> Differential Revision: https://reviews.freebsd.org/D12025 >> >> Added: >> head/lib/clang/Makefile.inc (contents, props changed) >> Modified: >> head/share/mk/bsd.lib.mk >> head/share/mk/bsd.prog.mk >> head/share/mk/bsd.sys.mk >> head/usr.bin/clang/Makefile.inc > > This causes llvm to emit hundreds of new warnings [1] for > "DWARF2 only supports one section per compilation unit" > > Are these expected? Are they a concern? Should we silence them? > Is this an upstream llvm bug [2]?
Last time this came up, I put up an upstream review to fix the warning, since it is nonsensical to warn about a non-code section. I've pinged the review again, but now I'm inclined to just commit it. :) -Dimitry
signature.asc
Description: Message signed with OpenPGP