uefi-ref/binutils-gdb
1
0
Fork 0
mirror of https://sourceware.org/git/binutils-gdb.git synced 2026-08-27 00:26:02 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions

gas: don't use frag_more to repeat copies

Browse source 
It's more elegant to use the repeat capability of an rs_fill frag_var,
and dramatically reduces memory usage for large repeat counts.

For ideal minimal memory use, repeats of up to the 100 bytes or so of
frag overhead should be handled by frag_more, but I don't care to
optimise those cases.  It's enough that the most common case of a
single output value uses frag_more, while stress testing the assembler
with huge repeats doesn't run into OOM.

	* read.c (float_cons): Repeat output using an rs_fill frag_var
	rather than looping with frag_more.
	(s_float_space): Likewise.
This commit is contained in:
Alan Modra 2026-02-19 15:18:12 +10:30
parent 52279d0980
commit 13a7a6aa0d
1 changed files with 11 additions and 13 deletions
Download patch file Download diff file Expand all files Collapse all files

24
gas/read.c
View file

@ -3873,6 +3873,7 @@ s_float_space (int float_type)
char temp[MAXIMUM_NUMBER_OF_CHARS_FOR_FLOAT];
char *stop = NULL;
char stopc = 0;
char *p;
#ifdef md_cons_align
md_cons_align (1);
@ -3906,13 +3907,11 @@ s_float_space (int float_type)
return;
}
while (--count >= 0)
{
char *p;
p = frag_more (flen);
memcpy (p, temp, flen);
}
if (count == 1)
p = frag_more (flen);
else
p = frag_var (rs_fill, flen, flen, 0, NULL, count, NULL);
memcpy (p, temp, flen);
demand_empty_rest_of_line ();
@ -5173,12 +5172,11 @@ float_cons (/* Clobbers input_line-pointer, checks end-of-line. */
count = count_exp.X_add_number;
}
#endif
while (--count >= 0)
{
p = frag_more (length);
memcpy (p, temp, length);
}
if (count == 1)
p = frag_more (length);
else
p = frag_var (rs_fill, length, length, 0, NULL, count, NULL);
memcpy (p, temp, length);
}
SKIP_WHITESPACE ();
}