[Xen-devel] [PATCH for-4.9] xen: append -Wno-unused-function to C99 header check

[Wei Liu <wei.liu2@xxxxxxxxxx>]

The check builds header file as if it is a C file. Clang doesn't like
the idea of having dead code in C file. The check as-is fails on Clang
with unused function warnings.

We know for sure we don't care about those unused function warnings.
Just ignore them.

Signed-off-by: Wei Liu <wei.liu2@xxxxxxxxxx>
---
Cc: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
Cc: George Dunlap <George.Dunlap@xxxxxxxxxxxxx>
Cc: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>
Cc: Jan Beulich <jbeulich@xxxxxxxx>
Cc: Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>
Cc: Stefano Stabellini <sstabellini@xxxxxxxxxx>
Cc: Tim Deegan <tim@xxxxxxx>
Cc: Wei Liu <wei.liu2@xxxxxxxxxx>
---
 xen/include/Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/xen/include/Makefile b/xen/include/Makefile
index 65a732a707..b0bacc99b5 100644
--- a/xen/include/Makefile
+++ b/xen/include/Makefile
@@ -111,7 +111,7 @@ headers.chk: $(PUBLIC_ANSI_HEADERS) Makefile
 headers99.chk: $(PUBLIC_C99_HEADERS) Makefile
        rm -f $@.new
        $(foreach i, $(filter %.h,$^),                                        \
-           $(CC) -x c -std=c99 -Wall -Werror                                 \
+           $(CC) -x c -std=c99 -Wall -Werror -Wno-unused-function            \
            -include stdint.h $(foreach j, $($(i)-prereq), -include $(j).h)   \
            -S -o /dev/null $(i)                                              \
            || exit $$?; echo $(i) >> $@.new;)
-- 
2.11.0


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel