On Mon, Feb 10, 2020 at 07:07:46PM +0100, Jakub Jelinek wrote: > We use sizeof ("...") - 1 or sizeof "..." - 1 in a lot of places though.
[...] > config/rs6000/rs6000.c: spaces += sizeof (" Reload=sl") - 1; > config/rs6000/rs6000.c: spaces += sizeof (" Upper=y") - 1; > config/rs6000/rs6000.c: strcpy (name, bname + sizeof ("__builtin_") - 1); > config/rs6000/rs6000.c: len += sizeof ("no-") - 1; > config/rs6000/rs6000.c: len += sizeof ("no-") - 1; > config/rs6000/rs6000.c: comma_len = sizeof (", ") - 1; Okay, I fixed those :-) Segher