Re: Remove a FIXME and unused variables in Meson - Mailing list pgsql-hackers

From Peter Eisentraut
Subject Re: Remove a FIXME and unused variables in Meson
Date
Msg-id da6beba4-6ca5-4055-b4ee-ef11d9f44aef@eisentraut.org
Whole thread Raw
In response to Remove a FIXME and unused variables in Meson  ("Tristan Partin" <tristan@neon.tech>)
List pgsql-hackers
On 14.03.24 06:13, Tristan Partin wrote:
> One of them adds version gates to two LLVM flags (-frwapv, 
> -fno-strict-aliasing). I believe we moved the minimum LLVM version 
> recently, so these might not be necessary, but maybe it helps for 
> historictal reasons. If not, I'll just remove the comment in a different 
> patch.

We usually remove version gates once the overall minimum required 
version is new enough.  So this doesn't seem like a step in the right 
direction.

> Second patch removes some unused variables. Were they analogous to 
> things in autotools and the Meson portions haven't been added yet?

Hmm, yeah, no idea.  These were not used even in the first commit for 
Meson support.  Might have had a purpose in earlier draft patches.




pgsql-hackers by date:

Previous
From: Dean Rasheed
Date:
Subject: Re: MERGE ... RETURNING
Next
From: Daniel Gustafsson
Date:
Subject: Re: What about Perl autodie?