On 17/12/2024 4:24 pm, Nicola Vetrini wrote: > This extension name is missing from the current list of extension used > by Xen, therefore is must be added to that configuration in order to > avoid violations for MISRA C Rule 1.1. > > Signed-off-by: Nicola Vetrini <[email protected]>
Given CI is broken, I've put this in for testing: https://gitlab.com/xen-project/people/andyhhp/xen/-/pipelines/1592826906 But, a question: > --- > automation/eclair_analysis/ECLAIR/toolchain.ecl | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/automation/eclair_analysis/ECLAIR/toolchain.ecl > b/automation/eclair_analysis/ECLAIR/toolchain.ecl > index 86e9a79b5231..9b419e562afa 100644 > --- a/automation/eclair_analysis/ECLAIR/toolchain.ecl > +++ b/automation/eclair_analysis/ECLAIR/toolchain.ecl > @@ -100,8 +100,8 @@ > ext_enum_value_not_int: non-documented GCC extension. > ext_gnu_array_range: see Section \"6.29 Designated Initializers\" of > "GCC_MANUAL". > " > --config=STD.diag,behavior+={c99,GCC_ARM64,"^(ext_paste_comma|ext_missing_varargs_arg|ext_named_variadic_macro|ext_return_has_void_expr|ext_gnu_statement_expr_macro|ext_sizeof_alignof_void_type|ext_forward_ref_enum_def|ext_gnu_array_range)$"} > --config=STD.diag,behavior+={c99,GCC_X86_64,"^(ext_paste_comma|ext_missing_varargs_arg|ext_named_variadic_macro|ext_return_has_void_expr|ext_gnu_statement_expr_macro|ext_sizeof_alignof_void_type|ext_flexible_array_in_struct|ext_flexible_array_in_array|ext_enum_value_not_int|ext_gnu_array_range)$"} > +-config=STD.diag,behavior+={c99,GCC_ARM64,"^(ext_paste_comma|ext_c_missing_varargs_arg|ext_missing_varargs_arg|ext_named_variadic_macro|ext_return_has_void_expr|ext_gnu_statement_expr_macro|ext_sizeof_alignof_void_type|ext_forward_ref_enum_def|ext_gnu_array_range)$"} > +-config=STD.diag,behavior+={c99,GCC_X86_64,"^(ext_paste_comma|ext_c_missing_varargs_arg|ext_missing_varargs_arg|ext_named_variadic_macro|ext_return_has_void_expr|ext_gnu_statement_expr_macro|ext_sizeof_alignof_void_type|ext_flexible_array_in_struct|ext_flexible_array_in_array|ext_enum_value_not_int|ext_gnu_array_range)$"} > -doc_end We're adding ext_c_missing_varargs_arg to the list. But there's also ext_missing_varargs_arg. What's the difference between the two? ~Andrew
