On 09/18/2017 03:05 PM, Jakub Jelinek wrote: > On Mon, Sep 18, 2017 at 03:01:53PM +0200, Martin Liška wrote: >> As discussed here: >> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81224 >> >> We have fallout caused by the patch and it's backport to active branches. >> I'm planning to revert the patch and install patch that will ignore empty >> string >> values. I'm testing the patch. >> >> Jakub do we really want it also for GCC 7? Note that the problematic >> test-case is OK on GCC 7 branch >> as it contains your patch mentioned in discussion. > > The question is, has the GCC 7 patch changed solely testcases where we'd > ICE on into ones where we warn, or are there cases where we used to accept > it and now warn? > Generally we don't want to introduce new warnings/errors on release branches > on something that used to be accepted, unless really necessary. > > Jakub >
I've just installed the patch to GCC 5.x and 6.x. I'll do the same for 7.x in order to preserve the behavior. Martin
