From 8e1f03ada09bf87c8c8f4c59a27422654b590165 Mon Sep 17 00:00:00 2001 From: Luis Felipe Strano Moraes Date: Tue, 18 Oct 2022 17:01:51 -0700 Subject: [PATCH] anv: reword info flag in intel_clc's getopt to avoid clash The info keyword was using the same short description that was listed for input files on the struct for long_options. Rewording it to 'v' and 'verbose' to be more in line with expectations. Reviewed-by: Lionel Landwerlin Part-of: --- src/intel/compiler/intel_clc.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/src/intel/compiler/intel_clc.c b/src/intel/compiler/intel_clc.c index 0d55488..fb8f0ec 100644 --- a/src/intel/compiler/intel_clc.c +++ b/src/intel/compiler/intel_clc.c @@ -266,6 +266,7 @@ print_usage(char *exec_name, FILE *f) " --prefix Prefix for variable names in generated C code.\n" " -g, --out Specify the output filename.\n" " -s, --spv Specify the output filename for spirv.\n" +" -v, --verbose Print more information during compilation.\n" , exec_name); } @@ -299,7 +300,7 @@ int main(int argc, char **argv) {"in", required_argument, 0, 'i'}, {"out", required_argument, 0, 'o'}, {"spv", required_argument, 0, 's'}, - {"info", no_argument, 0, 'i'}, + {"verbose", no_argument, 0, 'v'}, {0, 0, 0, 0} }; @@ -318,7 +319,7 @@ int main(int argc, char **argv) util_dynarray_init(&spirv_ptr_objs, mem_ctx); int ch; - while ((ch = getopt_long(argc, argv, "he:p:s:o:i", long_options, NULL)) != -1) + while ((ch = getopt_long(argc, argv, "he:p:s:o:v", long_options, NULL)) != -1) { switch (ch) { @@ -337,7 +338,7 @@ int main(int argc, char **argv) case 's': spv_outfile = optarg; break; - case 'i': + case 'v': print_info = true; break; case OPT_PREFIX: -- 2.7.4