*ping* on Alex' behalf and CCing the ARM maintainers. This fix looks obvious to me, and cleans up another couple of FAILs for the ARM port.
Richard/Ramana? Cheers, James On Thu, Mar 26, 2015 at 03:28:15PM +0000, Alex Velenko wrote: > On 04/03/15 11:13, Alex Velenko wrote: > > 2015-03-04 Alex Velenko <alex.vele...@arm.com> > > > > gcc/testsuite > > > > * gcc.target/arm/pr45701-1.c (history_expand_line_internal): Add an > > extra variable to force stack alignment. > > * gcc.target/arm/pr45701-2.c (history_expand_line_internal): Add an > > extra variable to force stack alignment. > > --- > > gcc/testsuite/gcc.target/arm/pr45701-1.c | 5 +++-- > > gcc/testsuite/gcc.target/arm/pr45701-2.c | 5 +++-- > > 2 files changed, 6 insertions(+), 4 deletions(-) > > > > diff --git a/gcc/testsuite/gcc.target/arm/pr45701-1.c > > b/gcc/testsuite/gcc.target/arm/pr45701-1.c > > index 2c690d5..454a087 100644 > > --- a/gcc/testsuite/gcc.target/arm/pr45701-1.c > > +++ b/gcc/testsuite/gcc.target/arm/pr45701-1.c > > @@ -5,6 +5,7 @@ > > /* { dg-final { scan-assembler-not "r8" } } */ > > > > extern int hist_verify; > > +extern int a1; > > extern char *pre_process_line (char*); > > extern char* str_cpy (char*, char*); > > extern int str_len (char*); > > @@ -16,10 +17,10 @@ history_expand_line_internal (char* line) > > { > > char *new_line; > > int old_verify; > > - > > + int a = a1; > > old_verify = hist_verify; > > hist_verify = 0; > > new_line = pre_process_line (line); > > - hist_verify = old_verify; > > + hist_verify = old_verify + a; > > return (new_line == line) ? savestring (line) : new_line; > > } > > diff --git a/gcc/testsuite/gcc.target/arm/pr45701-2.c > > b/gcc/testsuite/gcc.target/arm/pr45701-2.c > > index ee1ee7d..afe0840 100644 > > --- a/gcc/testsuite/gcc.target/arm/pr45701-2.c > > +++ b/gcc/testsuite/gcc.target/arm/pr45701-2.c > > @@ -5,6 +5,7 @@ > > /* { dg-final { scan-assembler-not "r8" } } */ > > > > extern int hist_verify; > > +extern int a1; > > extern char *pre_process_line (char*); > > extern char* savestring1 (char*, char*); > > extern char* str_cpy (char*, char*); > > @@ -17,11 +18,11 @@ history_expand_line_internal (char* line) > > { > > char *new_line; > > int old_verify; > > - > > + int a = a1; > > old_verify = hist_verify; > > hist_verify = 0; > > new_line = pre_process_line (line); > > - hist_verify = old_verify; > > + hist_verify = old_verify + a; > > /* Two tail calls here, but r3 is not used to pass values. */ > > return (new_line == line) ? savestring (line) : savestring1 (new_line, > > line); > > } > >