Re: [PATCH] Native spinlock support on RISC-V

From: Thomas Munro <thomas(dot)munro(at)gmail(dot)com>
To: Andres Freund <andres(at)anarazel(dot)de>
Cc: Christoph Berg <myon(at)debian(dot)org>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Marek Szuba <marecki(at)gentoo(dot)org>, PostgreSQL Development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [PATCH] Native spinlock support on RISC-V
Date: 2021-11-03 04:41:47
Message-ID: CA+hUKGKPnWYMyubBbuKys9ruEpMcz1zfGnJ8zdZiziTemcw1Wg@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Wed, Nov 3, 2021 at 5:13 PM Andres Freund <andres(at)anarazel(dot)de> wrote:
> Any chance you could enable jit_dump_bitcode and manually try a failing query? That should dump. bc files in the data directory. That'd might allow debugging this outside the emulated environment.
>
> I don't see where the undef is originating from, but I think it might suggest that something lead to that code being considered unreachable.

postgres=# set jit_above_cost = 0;
SET
postgres=# select 1 + 1;
FATAL: fatal llvm error: Cannot select: 0x4b3ec1a0: i64,ch =
load<(load 8 from %ir.14)> 0x41ef6fe8, 0x4b3ec138, undef:i64
0x4b3ec138: i64 = add nuw 0x4b3eab60, Constant:i64<24>
0x4b3eab60: i64,ch = load<(load 8 from %ir.7)> 0x41ef6fe8,
0x4b3eaaf8, undef:i64
0x4b3eaaf8: i64 = add nuw 0x4b3ea068, Constant:i64<16>
0x4b3ea068: i64,ch = CopyFromReg 0x41ef6fe8, Register:i64 %4
0x4b3ea000: i64 = Register %4
0x4b3ea888: i64 = Constant<16>
0x4b3ea6e8: i64 = undef
0x4b3ea9c0: i64 = Constant<24>
0x4b3ea6e8: i64 = undef
In function: evalexpr_0_0

Ah, I hadn't noticed this in the log before:

'generic' is not a recognized processor for this target (ignoring processor)

Sounds kinda serious :-)

Resulting .bc files and .ll files (produced by llvm-dis) attached.

Attachment Content-Type Size
75392.0.bc application/octet-stream 4.4 KB
75392.0.optimized.bc application/octet-stream 4.4 KB
75392.0.ll application/octet-stream 13.5 KB
75392.0.optimized.ll application/octet-stream 13.5 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message houzj.fnst@fujitsu.com 2021-11-03 05:37:48 RE: Added schema level support for publication.
Previous Message vignesh C 2021-11-03 04:25:15 Re: Added schema level support for publication.