Michael Tokarev <m...@tls.msk.ru> writes: > 29.05.2015 19:29, Paolo Bonzini wrote: >> On 29/05/2015 16:14, Alex Bennée wrote: >>> You mean just do: >>> >>> diff --git a/configure b/configure >>> index b707429..f13831a 100755 >>> --- a/configure >>> +++ b/configure >>> @@ -353,7 +353,7 @@ for opt do >>> ;; >>> --cpu=*) cpu="$optarg" >>> ;; >>> - --extra-cflags=*) QEMU_CFLAGS="$optarg $QEMU_CFLAGS" >>> + --extra-cflags=*) QEMU_CFLAGS="$QEMU_CFLAGS $optarg" >>> EXTRA_CFLAGS="$optarg" >>> ;; >>> --extra-ldflags=*) LDFLAGS="$optarg $LDFLAGS" >>> >>> I guess at the time I was trying to be clean and avoiding multiple -O >>> calls. But I guess that will have the same effect. >> >> Yes, that. Most other QEMU_CFLAGS assignments add at the beginning, so >> I guess the remaining ones (including the --extra-cflags one) should too. > > So, what's the final version of this patch?
I can send the second patch but I'm wary about changing all instances to QEMU_CFLAGS to append additional flags. > > Thanks, > > /mjt -- Alex Bennée
