This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [bitmap memory]: Allocation part 2
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Nathan Sidwell <nathan at codesourcery dot com>
- Cc: law at redhat dot com, GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Thu, 17 Feb 2005 16:31:37 -0500
- Subject: Re: [bitmap memory]: Allocation part 2
- References: <41A21524.4050102@codesourcery.com> <1101162070.27318.220.camel@localhost.localdomain> <41A4687C.5000204@codesourcery.com> <1101412983.13475.22.camel@localhost.localdomain> <41A7052C.4020405@codesourcery.com> <1106026735.30575.138.camel@localhost.localdomain> <42078D49.5040900@codesourcery.com>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Mon, Feb 07, 2005 at 03:46:17PM +0000, Nathan Sidwell wrote:
> >I think this got delayed when we went into regression fix mode only.
> >Now is probably a reasonable time to resubmit and get this last bit of
> >the bitmap memory allocation cleanup finished.
>
> yes, it missed the deadline. Here's an updated one I just booted and
> checked with. ok?
You forgot to patch s390.c and frv.c.
Committed as obvious.
2005-02-17 Jakub Jelinek <jakub@redhat.com>
* config/s390/s390.c (s390_alloc_pool, s390_free_pool,
s390_chunkify_start): Use BITMAP_ALLOC and BITMAP_FREE.
* config/frv/frv.c (frv_function_epilogue): Likewise.
--- gcc/config/s390/s390.c.jj 2005-01-24 09:49:43.000000000 +0100
+++ gcc/config/s390/s390.c 2005-02-17 21:36:01.774676046 +0100
@@ -5437,7 +5437,7 @@ s390_alloc_pool (void)
pool->label = gen_label_rtx ();
pool->first_insn = NULL_RTX;
pool->pool_insn = NULL_RTX;
- pool->insns = BITMAP_XMALLOC ();
+ pool->insns = BITMAP_ALLOC (NULL);
pool->size = 0;
return pool;
@@ -5464,7 +5464,7 @@ s390_free_pool (struct constant_pool *po
free (c);
}
- BITMAP_XFREE (pool->insns);
+ BITMAP_FREE (pool->insns);
free (pool);
}
@@ -5821,7 +5821,7 @@ s390_chunkify_start (void)
/* Find all labels that are branched into
from an insn belonging to a different chunk. */
- far_labels = BITMAP_XMALLOC ();
+ far_labels = BITMAP_ALLOC (NULL);
for (insn = get_insns (); insn; insn = NEXT_INSN (insn))
{
@@ -5918,7 +5918,7 @@ s390_chunkify_start (void)
}
- BITMAP_XFREE (far_labels);
+ BITMAP_FREE (far_labels);
/* Recompute insn addresses. */
--- gcc/config/frv/frv.c.jj 2005-02-10 11:50:17.000000000 +0100
+++ gcc/config/frv/frv.c 2005-02-17 21:36:38.306182361 +0100
@@ -1876,7 +1876,7 @@ frv_function_epilogue (FILE *file ATTRIB
memset (&frv_ifcvt.tmp_reg, 0, sizeof (frv_ifcvt.tmp_reg));
/* Release the bitmap of created insns. */
- BITMAP_XFREE (frv_ifcvt.scratch_insns_bitmap);
+ BITMAP_FREE (frv_ifcvt.scratch_insns_bitmap);
}
@@ -7722,7 +7722,7 @@ frv_ifcvt_modify_final (ce_if_block_t *c
{
rtx insn = emit_insn_before (frv_ifcvt.scratch_regs[i], existing_insn);
if (! frv_ifcvt.scratch_insns_bitmap)
- frv_ifcvt.scratch_insns_bitmap = BITMAP_XMALLOC ();
+ frv_ifcvt.scratch_insns_bitmap = BITMAP_ALLOC (NULL);
bitmap_set_bit (frv_ifcvt.scratch_insns_bitmap, INSN_UID (insn));
frv_ifcvt.scratch_regs[i] = NULL_RTX;
}
Jakub