From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Jesse Zhang <sbjesse(at)gmail(dot)com> |
Cc: | PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Header / Trailer Comment Typos for M4 macros |
Date: | 2020-04-23 00:33:31 |
Message-ID: | 12190.1587602011@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Jesse Zhang <sbjesse(at)gmail(dot)com> writes:
> I didn't make my question clear though: I'm curious what motivated the
> original addition of -Wno-unused-command-line-argument in commit
> 73b416b2e412, and how that problem did't quite manifest itself with Clang++.
We didn't then have the convention of mentioning relevant mailing list
threads in the commit log, but some excavation finds
So it seems to have been specific to clang circa version 6.0. Maybe
the clang boys thought better of this behavior more recently?
[ experiments... ] I see no warnings on current macOS (Apple clang
version 11.0.3) after removing the switch. So I guess they did fix it.
We're pretty conservative about dropping support for old toolchains,
though, so I doubt that we'd want to remove this configure check.
Especially if we don't know how long ago clang changed it.
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Peter Geoghegan | 2020-04-23 01:05:47 | Re: xid wraparound danger due to INDEX_CLEANUP false |
Previous Message | Peter Geoghegan | 2020-04-23 00:24:20 | Re: [PATCH] FIx explicit null dereference pointer (nbtree.c) |