[MPlayer-dev-eng] [PATCH] Add CFLAGS to linking

[Diego Biurrun]

On Wed, Dec 09, 2009 at 11:14:02PM +0000, Carl Eugen Hoyos wrote:
> Diego Biurrun <diego <at> biurrun.de> writes:
> 
> > On Wed, Dec 02, 2009 at 10:55:17PM +0100, Peter Hjalmarsson wrote:
> > > According to the manual for GCC [1], for at least some cases you should
> > > specify the same CFLAGS when linking with GCC that you had while
> > > compiling. This patch adds this for the linking of mplayer/mencoder.
> > > 
> > > [1] http://gcc.gnu.org/onlinedocs/gcc-4.4.2/gcc/Link-Options.html
> > 
> > Patch rejected.  There are no CFLAGS on the linking step on purpose.
> > Besides, I see nothing to support your claim after a cursory reading of
> > the page you linked to.
> 
> I have no opinion about the patch but I'd like to add that icc requires you to
> either not use "-parallel" at all or pass it to both the compiler and the linker
> stage.

Yes.  Such options should then be passed to both the compiler and linker
stage.

Diego