Hi Neil,

>Oh wow, yes, the if conditionals definately should be checked for each 
>iteration of the for loop. Good eye.

>Still though, seems like a bug in gcc to check the state of the loop index on 
>exit, when its never used to index the array at that value.  Seems a bit like 
>this bug:
>http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45180

>Thomas I'll send a patch to fix this directly Neil

Actually, that index may be used in cause last "ret" value is 0:

if (!ret) {
                        fprintf(stderr,
                                "ERROR: This system does not support \"%s\".\n"
                                "Please check that RTE_MACHINE is set 
correctly.\n",
                                cpu_feature_table[compile_time_flags[i]].name);
                        exit(1);
                }

Thanks,

Pablo de Lara
DPDK SW Engineer

--------------------------------------------------------------
Intel Shannon Limited
Registered in Ireland
Registered Office: Collinstown Industrial Park, Leixlip, County Kildare 
Registered Number: 308263 Business address: Dromore House, East Park, Shannon, 
Co. Clare




Reply via email to