From d751680f7ecc9323369909e41ad3455b8b3e49dd Mon Sep 17 00:00:00 2001 From: John Healy Date: Fri, 6 Jul 2001 19:09:23 +0000 Subject: 2001-07-06 John Healy * cgen.c (gas_cgen_save_fixups): Modified to allow more than one set of fixups to be stored. (gas_cgen_restore_fixups): Modified to allow the fixup chain to be restored to be chosen from any that are saved. (gas_cgen_swap_fixups): Modified to allow the current set of fixups to be swapped with any other set that has been saved. (gas_cgen_initialize_saved_fixups_array): New routine. * cgen.h: Modifed prototypes for gas_cgen_save_fixups, gas_cgen_restore_fixups, and gas_cgen_swap_fixups. Added definitions or MAX_SAVED_FIXUP_CHAINS. * config/tc-m32r.c (assemble_two_insns): Changed calls to fixup store, swap and restore fuctions to reflect the new interface. --- gas/cgen.c | 145 +++++++++++++++++++++++++++++++++++++++++-------------------- 1 file changed, 99 insertions(+), 46 deletions(-) (limited to 'gas/cgen.c') diff --git a/gas/cgen.c b/gas/cgen.c index 759b382008..7c4d12f3c1 100644 --- a/gas/cgen.c +++ b/gas/cgen.c @@ -94,62 +94,115 @@ queue_fixup (opindex, opinfo, expP) ++ num_fixups; } -/* The following three functions allow a backup of the fixup chain to be made, - and to have this backup be swapped with the current chain. This allows - certain ports, eg the m32r, to swap two instructions and swap their fixups - at the same time. */ -/* ??? I think with cgen_asm_finish_insn (or something else) there is no - more need for this. */ +/* The following functions allow fixup chains to be stored, retrieved, + and swapped. They are a generalization of a pre-existing scheme + for storing, restoring and swapping fixup chains that was used by + the m32r port. The functionality is essentially the same, only + instead of only being able to store a single fixup chain, an entire + array of fixup chains can be stored. It is the user's responsibility + to keep track of how many fixup chains have been stored and which + elements of the array they are in. + + The algorithms used are the same as in the old scheme. Other than the + "array-ness" of the whole thing, the functionality is identical to the + old scheme. + + gas_cgen_initialize_saved_fixups_array(): + Sets num_fixups_in_chain to 0 for each element. Call this from + md_begin() if you plan to use these functions and you want the + fixup count in each element to be set to 0 intially. This is + not necessary, but it's included just in case. It performs + the same function for each element in the array of fixup chains + that gas_init_parse() performs for the current fixups. + + gas_cgen_save_fixups (element): + element - element number of the array you wish to store the fixups + to. No mechanism is built in for tracking what element + was last stored to. + + gas_cgen_restore_fixups (element): + element - element number of the array you wish to restore the fixups + from. + + gas_cgen_swap_fixups(int element): + element - swap the current fixups with those in this element number. +*/ + +struct saved_fixups { + struct fixup fixup_chain[GAS_CGEN_MAX_FIXUPS]; + int num_fixups_in_chain; +}; -static struct fixup saved_fixups[GAS_CGEN_MAX_FIXUPS]; -static int saved_num_fixups; +static struct saved_fixups stored_fixups[MAX_SAVED_FIXUP_CHAINS]; -void -gas_cgen_save_fixups () +void +gas_cgen_initialize_saved_fixups_array () { - saved_num_fixups = num_fixups; - - memcpy (saved_fixups, fixups, sizeof (fixups[0]) * num_fixups); - - num_fixups = 0; + int i = 0; + while (i < MAX_SAVED_FIXUP_CHAINS) + stored_fixups[i++].num_fixups_in_chain = 0; } -void -gas_cgen_restore_fixups () +void +gas_cgen_save_fixups (int i) { - num_fixups = saved_num_fixups; - - memcpy (fixups, saved_fixups, sizeof (fixups[0]) * num_fixups); - - saved_num_fixups = 0; + if (i < 0 || i >= MAX_SAVED_FIXUP_CHAINS) + { + as_fatal("Index into stored_fixups[] out of bounds."); + return; + } + stored_fixups[i].num_fixups_in_chain = num_fixups; + memcpy(stored_fixups[i].fixup_chain, fixups, + sizeof (fixups[0])*num_fixups); + num_fixups = 0; } -void -gas_cgen_swap_fixups () +void +gas_cgen_restore_fixups (int i) { - int tmp; - struct fixup tmp_fixup; - - if (num_fixups == 0) - { - gas_cgen_restore_fixups (); - } - else if (saved_num_fixups == 0) - { - gas_cgen_save_fixups (); - } - else - { - tmp = saved_num_fixups; - saved_num_fixups = num_fixups; - num_fixups = tmp; + if (i < 0 || i >= MAX_SAVED_FIXUP_CHAINS) + { + as_fatal("Index into stored_fixups[] out of bounds."); + return; + } + num_fixups = stored_fixups[i].num_fixups_in_chain; + memcpy(fixups,stored_fixups[i].fixup_chain, + (sizeof (stored_fixups[i].fixup_chain[0]))*num_fixups); + stored_fixups[i].num_fixups_in_chain = 0; +} - for (tmp = GAS_CGEN_MAX_FIXUPS; tmp--;) - { - tmp_fixup = saved_fixups [tmp]; - saved_fixups [tmp] = fixups [tmp]; - fixups [tmp] = tmp_fixup; - } +void +gas_cgen_swap_fixups (int i) +{ + int tmp; + struct fixup tmp_fixup; + + if (i < 0 || i >= MAX_SAVED_FIXUP_CHAINS) + { + as_fatal("Index into stored_fixups[] out of bounds."); + return; + } + + if (num_fixups == 0) + { + gas_cgen_restore_fixups (i); + } + else if (stored_fixups[i].num_fixups_in_chain == 0) + { + gas_cgen_save_fixups (i); + } + else + { + tmp = stored_fixups[i].num_fixups_in_chain; + stored_fixups[i].num_fixups_in_chain = num_fixups; + num_fixups = tmp; + + for (tmp = GAS_CGEN_MAX_FIXUPS; tmp--;) + { + tmp_fixup = stored_fixups[i].fixup_chain [tmp]; + stored_fixups[i].fixup_chain[tmp] = fixups [tmp]; + fixups [tmp] = tmp_fixup; + } } } -- cgit v1.2.1