Re: Recent failures on buildfarm member hornet

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Noah Misch <noah(at)leadboat(dot)com>
Cc: pgsql-hackers(at)lists(dot)postgresql(dot)org, David Rowley <dgrowleyml(at)gmail(dot)com>
Subject: Re: Recent failures on buildfarm member hornet
Date: 2020-10-07 17:13:18
Message-ID: 2233910.1602090798@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Noah Misch <noah(at)leadboat(dot)com> writes:
> On Tue, Oct 06, 2020 at 09:56:49PM -0400, Tom Lane wrote:
>> Now that *is* surprising. Could you poke a little further to determine
>> which module is getting miscompiled? (gram.o seems like the next most
>> likely bet.)

> gram.o was it. (The way I rebuilt gram.o also rebuilt parser.o and scan.o,
> but those seem far less likely.)

This suggests that the problem is misoptimization of gram.y's
makeOrderedSetArgs:

/* don't merge into the next line, as list_concat changes directargs */
ndirectargs = list_length(directargs);

return list_make2(list_concat(directargs, orderedargs),
makeInteger(ndirectargs));

I think that if the compiler did what the comment says not to, it'd match
this symptom. However, I'm baffled as to why our recent pg_list.h changes
would've affected that.

regards, tom lane

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Robert Haas 2020-10-07 17:48:49 Re: [Patch] ALTER SYSTEM READ ONLY
Previous Message Pavel Borisov 2020-10-07 16:30:32 Re: [PATCH] Automatic HASH and LIST partition creation