Re: [PATCH] Fix -fself-test ICE in non-english locale (PR bootstrap/77569)

2017-01-03 Thread Jakub Jelinek
On Tue, Jan 03, 2017 at 08:09:03PM -0500, David Malcolm wrote: > > LC_ALL=en_US.UTF-8 and LC_ALL=de_DE.UTF-8). Ok for trunk? > > Thanks for looking at this; OK for trunk. Thanks. > I wonder if it makes sense to add a target to gcc/Makefile.in to run > the selftests in a non-English locale, to c

Re: [PATCH] Fix -fself-test ICE in non-english locale (PR bootstrap/77569)

2017-01-03 Thread David Malcolm
On Wed, 2017-01-04 at 00:01 +0100, Jakub Jelinek wrote: > Hi! > > The cb.error hook is called in the case we are looking for with > _("conversion from %s to %s not supported by iconv") > where _(msgid) is dgettext ("cpplib", msgid), so if performing -fself > -test > on iconv that doesn't support e

[PATCH] Fix -fself-test ICE in non-english locale (PR bootstrap/77569)

2017-01-03 Thread Jakub Jelinek
Hi! The cb.error hook is called in the case we are looking for with _("conversion from %s to %s not supported by iconv") where _(msgid) is dgettext ("cpplib", msgid), so if performing -fself-test on iconv that doesn't support ebcdic in a locale that has translations for this string, gcc ICEs. The