This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Committed, varasm.c: fix remaining output_constant_def INTEGER_CSTtypo, ENCODE_SECTION_INFO
- From: Hans-Peter Nilsson <hp at bitrange dot com>
- To: <gcc-patches at gcc dot gnu dot org>
- Date: Sun, 28 Apr 2002 23:58:58 -0400 (EDT)
- Subject: Committed, varasm.c: fix remaining output_constant_def INTEGER_CSTtypo, ENCODE_SECTION_INFO
See <URL:http://gcc.gnu.org/ml/gcc-patches/2002-04/msg01725.html>
(my previous message),
<URL:http://gcc.gnu.org/ml/gcc-patches/2002-03/msg01052.html>
(Olivier Hainque's original patch)
<URL:http://gcc.gnu.org/ml/gcc-cvs/2002-03/msg00839.html>
(commit message on 3.1) and
<URL:http://gcc.gnu.org/ml/gcc-cvs/2002-03/msg00840.html>
(commit with two code typos for trunk, one previosly fixed).
Also spotted a comment typo in the original patch and 3.1 commit,
no action taken except fixing it in this trunk commit.
Bootstrapped and checked on i686-pc-linux-gnu (and fixes the MMIX
ghostscript build problem where the bug was spotted), committed
as obvious.
* varasm.c (output_constant_def): Correct test for not calling
ENCODE_SECTION_INFO for INTEGER_CST.
Index: varasm.c
===================================================================
RCS file: /cvs/gcc/egcs/gcc/varasm.c,v
retrieving revision 1.264
diff -p -c -r1.264 varasm.c
*** varasm.c 29 Mar 2002 21:46:04 -0000 1.264
--- varasm.c 29 Apr 2002 03:27:30 -0000
*************** output_constant_def (exp, defer)
*** 3284,3290 ****
encoded in it. */
if (! found)
{
! if (TREE_CODE (exp) == INTEGER_CST)
ENCODE_SECTION_INFO (exp, true);
desc->rtl = rtl;
--- 3284,3292 ----
encoded in it. */
if (! found)
{
! /* Take care not to invoke ENCODE_SECTION_INFO for constants
! which don't have a TREE_CST_RTL. */
! if (TREE_CODE (exp) != INTEGER_CST)
ENCODE_SECTION_INFO (exp, true);
desc->rtl = rtl;
brgds, H-P