https://gcc.gnu.org/bugzilla/show_bug.cgi?id=113528
--- Comment #1 from Andrew Pinski <pinskia at gcc dot gnu.org> --- > Fragment N4 Is NOT an issue, based on all of the uses of add_char, the character is tested before adding it ... And there 256 can hold all values of signed char. >Fragment N5 Is just a small codiing style issue and actually will be optimized directly to the same thing really in this case as ' ' and '+' test bit will be in the same word (the second word/element). >Fragment N7 Not a bug, was done on purpose, see r5-958-ga33ad58be16a52 (PR 61242 and https://gcc.gnu.org/pipermail/gcc-patches/2014-May/389815.html). > Fragment N8 This code is software generated and the length is 0 in the GTY marker: ctf_dvdef_t ** GTY ((length ("0"))) ctfc_vars_list; So not a bug. > Fragment N9 Looks to be a way to avoid a goto, maybe now we could use a lamdba instead. So just a coding style issue. I have not looked into the other ones yet.