Hi! While wrapper_sect_offset is always initialized if (gnu_sections_found & SOMO_WRAPPING) != 0 and used only guarded with that same condition, as the PR says apparently we get a false positive maybe uninitialized warning for it still. I'd say it is a good programming style to just initialize such vars, especially in performance non-critical code.
Ok for trunk? 2013-03-05 Jakub Jelinek <ja...@redhat.com> PR middle-end/56526 * simple-object-mach-o.c (simple_object_mach_o_segment): Initialize wrapper_sect_offset to avoid a warning. --- libiberty/simple-object-mach-o.c.jj 2013-01-07 14:14:46.000000000 +0100 +++ libiberty/simple-object-mach-o.c 2013-03-05 11:46:19.574157009 +0100 @@ -432,7 +432,7 @@ simple_object_mach_o_segment (simple_obj size_t index_size; unsigned int n_wrapped_sects; size_t wrapper_sect_size; - off_t wrapper_sect_offset; + off_t wrapper_sect_offset = 0; fetch_32 = (omr->is_big_endian ? simple_object_fetch_big_32 Jakub