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.

Reply via email to