From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp.tuxdriver.com (charlotte.tuxdriver.com [70.61.120.58]) by dpdk.org (Postfix) with ESMTP id B3F5012A1 for ; Mon, 21 Apr 2014 16:41:42 +0200 (CEST) Received: from hmsreliant.think-freely.org ([2001:470:8:a08:7aac:c0ff:fec2:933b] helo=localhost) by smtp.tuxdriver.com with esmtpsa (TLSv1:AES128-SHA:128) (Exim 4.63) (envelope-from ) id 1WcFPe-0007Y0-UX; Mon, 21 Apr 2014 10:41:42 -0400 Date: Mon, 21 Apr 2014 10:41:37 -0400 From: Neil Horman To: Thomas Monjalon Message-ID: <20140421144137.GC25821@hmsreliant.think-freely.org> References: <1397585169-14537-2-git-send-email-nhorman@tuxdriver.com> <465940823.XQ0HDUOTVm@xps13> <20140418131849.GC4053@localhost.localdomain> <49923176.Wq7YvMNVY1@xps13> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <49923176.Wq7YvMNVY1@xps13> User-Agent: Mutt/1.5.21 (2010-09-15) X-Spam-Score: -2.9 (--) X-Spam-Status: No Cc: dev@dpdk.org Subject: Re: [dpdk-dev] [PATCH 01/15 v2] makefiles: Fixed -share command line option error X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 21 Apr 2014 14:41:42 -0000 On Fri, Apr 18, 2014 at 03:29:01PM +0200, Thomas Monjalon wrote: > 2014-04-18 09:18, Neil Horman: > > On Fri, Apr 18, 2014 at 01:23:19PM +0200, Thomas Monjalon wrote: > > > I think that CPU_LDFLAGS should be prefixed with -Wl, in case of CC > > > linking. So blindly assigning CC to LD variable seems a bad idea. > > > Other makefiles have different O_TO_S commands depending of LINK_USING_CC. > > > > I'm not so sure about that. Or more specifically, I wonder if some more > > rework isn't needed here. I say that because, while what you say makes > > sense in terms of formatting the CPU_FLAGS variable for use with CC, the > > only current use of CPU_LDFLAGS set -melf_i386, which IIRC is a gcc flag, > > not meant to be passed to LD. I can change the makefile to completely > > rewrite the comand based on LINK_USING_CC, but it seems to me that > > CPU_LDFLAGS should not be passed in the use of the LD case. > > Right, -melf_i386 shouldn't be a LDFLAG. > Feel free to fix it. > By the way, It's cleaner to prepare -Wl prefixing and keep an empty LDFLAGS. > Actually, I looked into this a bit more, turns out -melf_i386 is an LDFLAG. Its a bit silly to have it there because its passed automatically from the compiler to the linker depending on the machine that gcc is building for, or ld does it natively depending on the input file format. But perhaps some compiler or linker version has a bug where that doesn't happen properly. Either way, it can be prefixed with a -Wl, so I'm just going to leave it alone Neil > -- > Thomas >