-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviews.gem5.org/r/3454/#review8274
-----------------------------------------------------------
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.
- Bjoern A. Zeeb
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
