> On April 26, 2016, 8:54 p.m., Bjoern A. Zeeb wrote:
> > This was fixed in HEAD after I noticed that we aren't spec compliant; but
> > older versions indeed need the cast.
> >
> > I am currently using at least an #ifdef here:
> >
> > --- a/src/sim/init_signals.cc Tue Apr 05 08:08:12 2016 -0500
> > +++ b/src/sim/init_signals.cc Tue Apr 26 20:51:26 2016 +0000
> > @@ -66,7 +66,11 @@
> > setupAltStack()
> > {
> > stack_t stack;
> > +#ifdef __FreeBSD__
> > + stack.ss_sp = (char *)fatalSigStack;
> > +#else
> > stack.ss_sp = fatalSigStack;
> > +#endif
> > stack.ss_size = sizeof(fatalSigStack);
> > stack.ss_flags = 0;
> >
> > But for the FreeBSD block we almost also need to check __FreeBSD_version >=
> > 11xxxxxx but I haven't done the due diligence yet what the exact version
> > number should be.
https://svnweb.freebsd.org/base?view=revision&revision=294930
PS: this has nothing to do with clang really.
- Bjoern A.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviews.gem5.org/r/3454/#review8274
-----------------------------------------------------------
On April 26, 2016, 6:02 p.m., Pierre-Yves Péneau wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviews.gem5.org/r/3454/
> -----------------------------------------------------------
>
> (Updated April 26, 2016, 6:02 p.m.)
>
>
> Review request for Default.
>
>
> Repository: gem5
>
>
> Description
> -------
>
> Appease clang-3.4.1
>
>
> Diffs
> -----
>
> src/sim/init_signals.cc e41eca4aecbb
>
> Diff: http://reviews.gem5.org/r/3454/diff/
>
>
> Testing
> -------
>
> Fresh install of FreeBSD 10.3-RELEASE (i386), default clang compiler (3.4.1)
>
>
> Thanks,
>
> Pierre-Yves Péneau
>
>
_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev
