On Tue, Mar 20, 2012 at 05:10:48PM -0700, Darren Hart wrote:
>
>
> On 03/20/2012 04:58 PM, John Kacur wrote:
> > On Tue, Mar 20, 2012 at 8:05 PM, Darren Hart <dvhart@xxxxxxxxxxxxxxx> wrote:
> >> Accept user supplied CFLAGS and LDFLAGS, overwriting the
> >> Makefile supplied versions. This can cause the build to
> >> fail if the user does not provide at least what the Makefile
> >> defines, but so be it.
> >>
> >> Signed-off-by: Darren Hart <dvhart@xxxxxxxxxxxxxxx>
> >> CC: Clark Williams <williams@xxxxxxxxxx>
> >> CC: John Kacur <jkacur@xxxxxxxxxx>
> >> CC: Denys Dmytriyenko <denis@xxxxxxxxx>
> >> ---
> >
> > I was just wondering what you need LDFLAGS for? Chatting with Darren
> > on IRC, it seems like you're using -Wl to pass options via gcc to the
> > linker, and we don't have loadable libs either. Maybe you could resend
> > the patch with just the CFLAGS change until we have a real world
> > reason for LDFLAGS
>
> Denys,
>
> Am I missing a reason why we need LDFLAGS? With the current Makefile. we
> could just add anything we want to CFLAGS in a pinch anyway...
Darren,
>From OE-Core config files:
LINKER_HASH_STYLE ??= "gnu"
TARGET_LINK_HASH_STYLE ?= "${@['-Wl,--hash-style=gnu',''][d.getVar('LINKER_HASH_STYLE', True) != 'gnu']}"
export TARGET_LDFLAGS = "-Wl,-O1 ${TARGET_LINK_HASH_STYLE}"
ASNEEDED = "-Wl,--as-needed"
TARGET_LDFLAGS += "${ASNEEDED}"
export LDFLAGS = "${TARGET_LDFLAGS}"
So, those are still linker flags (altough passed through -Wl to gcc), hence
they belong to LDFLAGS, not CFLAGS. Arguably, you only need to pass CFLAGS
during compile stage and LDFLAGS during link stage. On the other hand, as a
workaround, I was passing them to TARGET_CC_ARCH, which gets embedded into CC
and won't distinguish between compile/link stages...
So rt-tests just gets away not using LDFLAGS and re-using CFLAGS for the link
stage. :) But the current Makefile as it is now won't honor CFLAGS being set
from outside, unless you pass them explicitly on the command line to make, or
call make with -e flag.
--
Denys
--
To unsubscribe from this list: send the line "unsubscribe linux-rt-users" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
