Hi Jakub, On Thu, Feb 23, 2023 at 2:34 AM Jakub Jelinek <ja...@redhat.com> wrote: > The translation PR complains that these 4 messages from xtensa-dynconfig.c > are marked in po/gcc.pot as c-format (which doesn't allow %qs) while they > should be gcc-internal-format. > > The problem is in the manual translation of the strings with _(), > that should be both unnecessary because fatal_error invokes _() on its > argument already, but also incorrect for the above reason, for > gcc-internal-format strings one should use G_("...") instead if really > needed. > > The following patch drops those _("..."), tested by regenerating po/gcc.pot > to see they are now gcc-internal-format, but not really tested on xtensa > target. > > Ok for trunk?
Ok. > BTW, why is the file using .c extension rather than .cc? It was initially developed when backend code was still in .c files and I failed to update this part during forward porting. I'll fix it. > Why isn't t-xtensa using $(COMPILE) and $(POSTCOMPILE) > to compile it like for most other extra_objs on other targets? > And, why does that file use <> style includes of gcc internal > headers rather than "" style which is used everywhere else > in gcc? No real reason for either. I'll fix it. Thanks for your review. -- Max