[flac-dev] [PATCH] configure: only use -mstackrealign for mingw32

Tristan Matthews tmatth at videolan.org
Fri Apr 10 11:47:58 PDT 2015


On Fri, Apr 10, 2015 at 1:40 PM, lvqcl <lvqcl.mail at gmail.com> wrote:

> Tristan Matthews wrote:
>
> >       if test "x$asm_optimisation$sse_os" = "xyesyes" ; then
> >               XIPH_ADD_CFLAGS([-msse2])
> > -             XIPH_ADD_CFLAGS([-mstackrealign])
> > +             if test "$host_os" = "mingw32" ; then
> > +                     XIPH_ADD_CFLAGS([-mstackrealign])
> > +                     fi
> >               fi
>
> But sse_os==no doesn't prevent libFLAC from using SSE intrinsics.
> So I think that the code
>
>         if test "$host_os" = "mingw32" ; then
>                 XIPH_ADD_CFLAGS([-mstackrealign])
>                 fi
>
> should be somewhere outside of
>
>         if test "x$asm_optimisation$sse_os" = "xyesyes"
>                 ...
>                 ...
>                 fi
>

Ah, good point, I'll resend.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.xiph.org/pipermail/flac-dev/attachments/20150410/a64dcc86/attachment.htm 


More information about the flac-dev mailing list