Re: [RFA]: testsuite/gdb.base/miscexprs.*

[Fernando Nasser <fnasser@redhat.com>]

Michael Snyder wrote:
> 
> Corinna Vinschen wrote:
> >
> > Hi,
> >
> > on my target I have the same problem with the very small (2K) stack
> > when running miscexprs.exp as I already had with call-ar-st.exp.
> >
> > I would like to propose the following patch, which adds the ability
> > to extend miscexprs.exp for targets, which also suffer from the
> > small stack.  I added a compiler switch to define the preprocessor
> > symbol `STORAGE', which by default is set to empty. A target
> > specific conditional can be used for setting it to "static", to
> > allow running the tests with the datastructures allocated in the
> > global data area instead of on the stack.  Obviously, miscexprs.exp
> > in the below incarantion just contains such a conditional only for
> > the target I'm just working on...
> 
> This is a good step, and I'm in favor of accepting it.
> I'd just like to suggest that an even better approach
> might be to add a dejagnu variable that could be defined
> in the baseboard file, rather than put a target ifdef in
> the testsuite script.
> 
> Something like this:
> baseboard file:
>   set_board_info gdb, small_stack_section, 1
>   set_board_info gdb, small_data_section, 1    // future extension
> testsuite file:
>   if [target_info exists gdb,small_stack_section] {
>      set storage "-DSTORAGE=static"
>   }
> 

I was simultaneously writting the same thing.

Thanks Michael, small_stack_section is better than smallstack.



-- 
Fernando Nasser
Red Hat Canada Ltd.                     E-Mail:  fnasser@redhat.com
2323 Yonge Street, Suite #300
Toronto, Ontario   M4P 2C9