| From: | Peter Eisentraut <peter(at)eisentraut(dot)org> |
|---|---|
| To: | Holger Hoffstätte <holger(at)applied-asynchrony(dot)com>, pgsql-hackers(at)postgresql(dot)org |
| Subject: | Re: [PATCH] jit: fix build with LLVM-21 |
| Date: | 2025-09-15 06:35:00 |
| Message-ID: | 1b9ee295-8cf8-4a94-87c8-f10e08cb5292@eisentraut.org |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On 12.09.25 09:47, Holger Hoffstätte wrote:
>> As a small style request, I would flip the conditional around so
>> that the new code appears first. I see that we don't do this very
>> consistently in the existing code, but maybe we can start a new
>> trend. ;-)
> I knew this would come up since I pondered the same thing :D
> As you said, the existing code is not consistent, but I can switch this
> to let the new code appear first, if you prefer.
> A new patch is attached.
I have committed this one. For the remaining issue on arm, let's wait
for some additional reviews.
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Чумак Антон | 2025-09-15 06:35:23 | [PATCH] Introduce unified support for composite GUC options |
| Previous Message | David Rowley | 2025-09-15 06:27:59 | Re: Fix missing EvalPlanQual recheck for TID scans |