configure | 1 - 1 file changed, 1 deletion(-)
We have now managed to eradicate all the places in the codebase
that triggered clang's -Waddress-of-packed-member warning. Remove
the compiler flag that exempted it from our usual -Werror policy.
This will prevent any new problematic code being added in future.
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
---
configure | 1 -
1 file changed, 1 deletion(-)
diff --git a/configure b/configure
index fbd0825488c..50ac5d90f2e 100755
--- a/configure
+++ b/configure
@@ -1881,7 +1881,6 @@ gcc_flags="-Wno-missing-include-dirs -Wempty-body -Wnested-externs $gcc_flags"
gcc_flags="-Wendif-labels -Wno-shift-negative-value $gcc_flags"
gcc_flags="-Wno-initializer-overrides -Wexpansion-to-defined $gcc_flags"
gcc_flags="-Wno-string-plus-int $gcc_flags"
-gcc_flags="-Wno-error=address-of-packed-member $gcc_flags"
# Note that we do not add -Werror to gcc_flags here, because that would
# enable it for all configure tests. If a configure test failed due
# to -Werror this would just silently disable some features,
--
2.20.1
On 2019-02-08 14:21, Peter Maydell wrote: > We have now managed to eradicate all the places in the codebase > that triggered clang's -Waddress-of-packed-member warning. Remove > the compiler flag that exempted it from our usual -Werror policy. > This will prevent any new problematic code being added in future. > > Signed-off-by: Peter Maydell <peter.maydell@linaro.org> > --- > configure | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/configure b/configure > index fbd0825488c..50ac5d90f2e 100755 > --- a/configure > +++ b/configure > @@ -1881,7 +1881,6 @@ gcc_flags="-Wno-missing-include-dirs -Wempty-body -Wnested-externs $gcc_flags" > gcc_flags="-Wendif-labels -Wno-shift-negative-value $gcc_flags" > gcc_flags="-Wno-initializer-overrides -Wexpansion-to-defined $gcc_flags" > gcc_flags="-Wno-string-plus-int $gcc_flags" > -gcc_flags="-Wno-error=address-of-packed-member $gcc_flags" > # Note that we do not add -Werror to gcc_flags here, because that would > # enable it for all configure tests. If a configure test failed due > # to -Werror this would just silently disable some features, Great! Reviewed-by: Thomas Huth <thuth@redhat.com>
On 08/02/2019 14:33, Thomas Huth wrote: > On 2019-02-08 14:21, Peter Maydell wrote: >> We have now managed to eradicate all the places in the codebase >> that triggered clang's -Waddress-of-packed-member warning. Remove >> the compiler flag that exempted it from our usual -Werror policy. >> This will prevent any new problematic code being added in future. >> >> Signed-off-by: Peter Maydell <peter.maydell@linaro.org> >> --- >> configure | 1 - >> 1 file changed, 1 deletion(-) >> >> diff --git a/configure b/configure >> index fbd0825488c..50ac5d90f2e 100755 >> --- a/configure >> +++ b/configure >> @@ -1881,7 +1881,6 @@ gcc_flags="-Wno-missing-include-dirs -Wempty-body -Wnested-externs $gcc_flags" >> gcc_flags="-Wendif-labels -Wno-shift-negative-value $gcc_flags" >> gcc_flags="-Wno-initializer-overrides -Wexpansion-to-defined $gcc_flags" >> gcc_flags="-Wno-string-plus-int $gcc_flags" >> -gcc_flags="-Wno-error=address-of-packed-member $gcc_flags" >> # Note that we do not add -Werror to gcc_flags here, because that would >> # enable it for all configure tests. If a configure test failed due >> # to -Werror this would just silently disable some features, > > Great! > > Reviewed-by: Thomas Huth <thuth@redhat.com> > Applied to my trivial-patches branch. Thanks, Laurent
On 2/8/19 2:21 PM, Peter Maydell wrote: > We have now managed to eradicate all the places in the codebase > that triggered clang's -Waddress-of-packed-member warning. Remove > the compiler flag that exempted it from our usual -Werror policy. > This will prevent any new problematic code being added in future. Yay! > > Signed-off-by: Peter Maydell <peter.maydell@linaro.org> Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com> > --- > configure | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/configure b/configure > index fbd0825488c..50ac5d90f2e 100755 > --- a/configure > +++ b/configure > @@ -1881,7 +1881,6 @@ gcc_flags="-Wno-missing-include-dirs -Wempty-body -Wnested-externs $gcc_flags" > gcc_flags="-Wendif-labels -Wno-shift-negative-value $gcc_flags" > gcc_flags="-Wno-initializer-overrides -Wexpansion-to-defined $gcc_flags" > gcc_flags="-Wno-string-plus-int $gcc_flags" > -gcc_flags="-Wno-error=address-of-packed-member $gcc_flags" > # Note that we do not add -Werror to gcc_flags here, because that would > # enable it for all configure tests. If a configure test failed due > # to -Werror this would just silently disable some features, >
© 2016 - 2024 Red Hat, Inc.