Re: LLVM compile failing in seawasp

Hello Thomas,

>> I would just #undef Min for our small number of .cpp files that
>> include LLVM headers.  It's not as though you need it in C++, which
>> has std::min() from <algorithm>.
>
> Like so.  Fixes the problem for me (llvm-devel-9.0.d20190712).

Hmmm. Not so nice, but if it works, why not, at least the impact is 
much smaller than renaming.

Note that the Min macro is used in several pg headers (ginblock.h, 
ginxlog.h, hash.h, simplehash.h, spgist_private.h), so you might really 
need it depending on what is being done later.

Otherwise, why not simply move llvm C++ includes *before* postgres 
includes? They should be fully independent anyway, so the order should 
not matter?

-- 
Fabien.