hmm, we did need this at one point in time for the packed structs used for shader instruction assembly.. maybe newer gcc doesn't warn about this (since it mostly seems related to a bug in older gcc versions)? I guess since (afaiu) android provides it's own compiler per android release, I suppose we don't have to worry about someone building w/ any random old gcc version, so I don't see any issue dropping this for the android build.
On Wed, Feb 24, 2016 at 1:56 PM, Rob Herring <r...@kernel.org> wrote: > Enabling this warning doesn't generate any warnings with gcc, but is an > unknown option for clang, so drop it. > > Cc: Rob Clark <robdcl...@gmail.com> > Signed-off-by: Rob Herring <r...@kernel.org> Acked-by: Rob Clark <robdcl...@gmail.com> > --- > src/gallium/drivers/freedreno/Android.mk | 3 --- > 1 file changed, 3 deletions(-) > > diff --git a/src/gallium/drivers/freedreno/Android.mk > b/src/gallium/drivers/freedreno/Android.mk > index 8c1ab08..cd46f39 100644 > --- a/src/gallium/drivers/freedreno/Android.mk > +++ b/src/gallium/drivers/freedreno/Android.mk > @@ -32,9 +32,6 @@ LOCAL_SRC_FILES := \ > $(a4xx_SOURCES) \ > $(ir3_SOURCES) > > -LOCAL_CFLAGS := \ > - -Wno-packed-bitfield-compat > - > LOCAL_C_INCLUDES := \ > $(LOCAL_PATH)/ir3 > > -- > 2.5.0 > _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
