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.
