Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Jim Blandy <jimb@redhat.com>
To: gdb-patches@sources.redhat.com
Subject: RFC: sim: don't use -Wuninitialized without -O
Date: Tue, 15 Mar 2005 21:08:00 -0000	[thread overview]
Message-ID: <vt2fyywr576.fsf@zenia.home> (raw)


I committed a similar patch for GDB a few days ago; this one is for
the simulators.  I think I've avoided using '==' in this one.  :)
Diffs to generated files omitted.

sim/common/ChangeLog:
2005-02-28  Jim Blandy  <jimb@redhat.com>

	* aclocal.m4 (SIM_AC_OPTION_WARNINGS): Don't include
	-Wuninitialized in the default list of build warnings if CFLAGS is
	set, and doesn't include -O.  (Using -Wuninitialized without
	optimization produces a warning, which interferes with compilation
	with -Werror.)

sim/ChangeLog:
2005-02-28  Jim Blandy  <jimb@redhat.com>

	* d10v/configure.ac, igen/configure.ac, m68hc11/configure.ac,
	mips/configure.ac, mn10300/configure.ac, v850/configure.ac:
	Regenerated, after change to common/aclocal.m4.

Index: sim/common/aclocal.m4
===================================================================
RCS file: /cvs/src/src/sim/common/aclocal.m4,v
retrieving revision 1.9
diff -c -p -r1.9 aclocal.m4
*** sim/common/aclocal.m4	12 Jan 2005 16:23:13 -0000	1.9
--- sim/common/aclocal.m4	28 Feb 2005 20:38:02 -0000
*************** AC_DEFUN(SIM_AC_OPTION_WARNINGS,
*** 790,796 ****
  # NOTE: If you add to this list, remember to update
  # gdb/doc/gdbint.texinfo.
  build_warnings="-Wimplicit -Wreturn-type -Wcomment -Wtrigraphs \
! -Wformat -Wparentheses -Wpointer-arith -Wuninitialized"
  # Up for debate: -Wswitch -Wcomment -trigraphs -Wtrigraphs
  # -Wunused-function -Wunused-label -Wunused-variable -Wunused-value
  # -Wchar-subscripts -Wtraditional -Wshadow -Wcast-qual
--- 790,807 ----
  # NOTE: If you add to this list, remember to update
  # gdb/doc/gdbint.texinfo.
  build_warnings="-Wimplicit -Wreturn-type -Wcomment -Wtrigraphs \
! -Wformat -Wparentheses -Wpointer-arith"
! # GCC supports -Wuninitialized only with -O or -On, n != 0.
! if test x${CFLAGS+set} = xset; then
!   case "${CFLAGS}" in
!     *"-O0"* ) ;;
!     *"-O"* )
!       build_warnings="${build_warnings} -Wuninitialized"
!     ;;
!   esac
! else
!   build_warnings="${build_warnings} -Wuninitialized"
! fi
  # Up for debate: -Wswitch -Wcomment -trigraphs -Wtrigraphs
  # -Wunused-function -Wunused-label -Wunused-variable -Wunused-value
  # -Wchar-subscripts -Wtraditional -Wshadow -Wcast-qual


             reply	other threads:[~2005-03-15 21:08 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-03-15 21:08 Jim Blandy [this message]
2005-03-15 23:27 ` Jim Blandy

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=vt2fyywr576.fsf@zenia.home \
    --to=jimb@redhat.com \
    --cc=gdb-patches@sources.redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox