Hi, On 2022-07-17 14:54:48 -0400, Tom Lane wrote: > Beyond that, I think this is committable. We're not likely to learn much > more about any potential issues except by exposing it to the buildfarm > and developer usage.
Leaving egg on my face aside, seems to work well so far. It looks like we might be missing out on benefiting from this on more platforms - the test right now is in the gcc specific section of configure.ac, but it looks like at least Intel's, sun's and AIX's compilers might all support -fvisibility with the same syntax. Given that that's just about all compilers we support using configure, perhaps we should just move that out of the compiler specific section? Doesn't look like there's much precedent for that so far... Greetings, Andres Freund