<div dir="ltr"><div dir="ltr">Hi Andres,<div><br></div><div>Oops! That's a think-o on my part -- fixed in 0465bdc47f6a. Thanks Andres!</div><div><br></div><div>-- Lang.</div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Oct 4, 2020 at 2:04 PM Andres Freund <<a href="mailto:andres@anarazel.de" target="_blank">andres@anarazel.de</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">Hi,<br>
<br>
On 2020-10-02 11:12:45 -0700, Andres Freund via llvm-dev wrote:<br>
> The error reporting change also works and helps. Need to figure out on<br>
> the PG side how to better associate the reported errors with the user<br>
> action, but that's quite doable. Might just end up re-setting the error<br>
> reporter on a more granular basis, to update the context.<br>
<br>
One thing I noticed is that there's an unconditional error reporting<br>
call:<br>
<br>
LLJIT::~LLJIT() {<br>
  if (CompileThreads)<br>
    CompileThreads->wait();<br>
  ES->reportError(ES->endSession());<br>
}<br>
<br>
Is that intentional? It's easy enough to check whether error is 0 in the<br>
callback, but I don't want to do that if that's not what you intended<br>
API wise.<br>
<br>
Greetings,<br>
<br>
Andres Freund<br>
</blockquote></div>