Jacob Champion <jacob.champ...@enterprisedb.com> writes: > On Tue, Jun 24, 2025 at 2:50 PM Tom Lane <t...@sss.pgh.pa.us> wrote: >> Can't say that I find this to be impressive software engineering: >> rather than having only one probe failure mode to worry about, >> we have two, depending on whether the compiler knows __has_include(). >> Pretty close to the worst of all possible worlds.
> I did a double-take on the code you posted, but! It looks like they're > running the preprocessor only. That doesn't seem so bad to me (though > they could probably do better than calling it a "compile" in the log). Oh! Okay, then the two cases should be mostly semantically equivalent, I think, ie it's just a "does the header exist" test. There are probably some edge cases where the effects are different, but nothing that would be likely to matter for us. regards, tom lane