Re: Decouple C++ support in Meson's PGXS from LLVM enablement

On 04.01.26 00:07, Jelte Fennema-Nio wrote:
> On Sat Jan 3, 2026 at 10:42 AM CET, Jelte Fennema-Nio wrote:
>> 3. Default to g++ instead of c++. I know I said I liked the c++ better,
>>    and I still do, but it seems better to align with autoconf for now
>>    and then change both of these later if we want to.
>
> Apparently I had sent an outdated patch... And didn't actually include
> that. Fixed now.

This looks good to me.

There is an unfortunate cpp/cxx naming conflict. For example, this
seems very confusing:

var_cpp = ' '.join(cc.cmd_array() + ['-E'])
...
if have_cpp
var_cxx = ' '.join(cpp.cmd_array())

Here, we use "cpp" to mean two different things in the space of a few lines.

Perhaps we should prefer to use the less ambiguous name "cxx"
throughout. So change

+have_cpp = add_languages('cpp', required: false, native: false)
+if have_cpp
+ cpp = meson.get_compiler('cpp')
+endif

to

+have_cxx = add_languages('cpp', required: false, native: false)
+if have_cxx
+ cxx = meson.get_compiler('cpp')
+endif

Then the potentially confusing use of "cpp" is contained in these two lines.