Re: [Mingw-w64-public] [PATCH 1/6] widl: Readd CFLAGS for optimization

[Jacek C. <ja...@co...>]

On 01/12/2018 09:14 AM, Martin Storsjö wrote:
> These seem to have been removed by mistake as part of a change
> touching the line above, in 4581de347f.
>
> Signed-off-by: Martin Storsjö <ma...@ma...>
> ---
>  mingw-w64-tools/widl/Makefile.am | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/mingw-w64-tools/widl/Makefile.am b/mingw-w64-tools/widl/Makefile.am
> index 1e348c7..a10c996 100644
> --- a/mingw-w64-tools/widl/Makefile.am
> +++ b/mingw-w64-tools/widl/Makefile.am
> @@ -59,6 +59,6 @@ widl_SOURCES = src/widl.h \
>    include/winnt.rh
>  
>  widl_CPPFLAGS = -I$(top_srcdir)/include -DDEFAULT_INCLUDE_DIR=\"$(includedir)/../$(target)/include\" -DDEFAULT_BINDIR=\"$(bindir)\"
> -widl_CFLAGS = -Wall -Wformat -Wpacked -Wmissing-declarations -Wimplicit-function-declaration -Wmissing-prototypes -Wstrict-aliasing=2
> +widl_CFLAGS = -O3 -g -Wall -Wformat -Wpacked -Wmissing-declarations -Wimplicit-function-declaration -Wmissing-prototypes -Wstrict-aliasing=2
>  
>  DISTCHECK_CONFIGURE_FLAGS = --host=$(host) --target=$(target)


Looks good to me.


Thanks,

Jacek