> -----Original Message-----
> From: Richard Henderson <richard.hender...@linaro.org>
> Sent: Thursday, July 28, 2022 5:11 PM
> To: Paolo Bonzini <pbonz...@redhat.com>; qemu-devel@nongnu.org
> Cc: Taylor Simpson <tsimp...@quicinc.com>
> Subject: Re: [PATCH] configure: pass correct cflags to container-based cross
> compilers
> 
> On 7/28/22 15:22, Paolo Bonzini wrote:
> > probe_target_compiler returns nonempty $target_cc for installed
> > toolchains and $container_cross_cc for container-based toolchains.  In
> > both cases however the flags (coming from
> > $cross_cc_cflags_${target_arch}) must be in $target_cflags.
> >
> > Therefore, do not clear them prior to returning from
> probe_target_compiler.
> >
> > Reported-by: Taylor Simpson <tsimp...@quicinc.com>
> > Fixes: 92e288fcfb ("build: try both native and cross compilers",
> > 2022-07-08)
> > Signed-off-by: Paolo Bonzini <pbonz...@redhat.com>
> > ---
> >   configure | 1 -
> >   1 file changed, 1 deletion(-)
> >
> > diff --git a/configure b/configure
> > index c4c02b8438..72ab03f11a 100755
> > --- a/configure
> > +++ b/configure
> > @@ -2173,7 +2173,6 @@ probe_target_compiler() {
> >       build_static=
> >       target_cc=
> >       target_ccas=
> > -    target_cflags=
> >       target_ar=
> >       target_as=
> >       target_ld=
> 
> Reviewed-by: Richard Henderson <richard.hender...@linaro.org>
> 
> r~

Reviewed-by: Taylor Simpson <tsimp...@quicinc.com>

Reply via email to