Hi, On 2022-12-29 13:51:37 -0500, Tom Lane wrote: > We might find that some GCC-impostor compilers have trouble with it, > but if so we can adjust the #ifdef here.
Yea. I suspect it's widely enough used that any compiler claiming to be gcc compatible has it, but ... > Getting nitpicky, I suggest calling it "HAVE_PRAGMA_GCC_SYSTEM_HEADER" > to align better with what you actually have to write. Makes sense. > + * Newer versions the perl headers trigger a lot of warnings with our > compiler > > "Newer versions of ..." please. Otherwise LGTM. Oops. > > Should we backpatch this? Given the volume of warnings it's probably a good > > idea. But I'd let it step in HEAD for a few days of buildfarm coverage > > first. > > +1 to both points. Pushed to HEAD. Greetings, Andres Freund