<div dir="ltr">You needed to add llvm-commits by hand. I'm sorry I didn't notice that, but if you forgot to add that, Phab shows a warning message on top about that.</div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jun 19, 2015 at 2:24 PM, Lang Hames <span dir="ltr"><<a href="mailto:lhames@gmail.com" target="_blank">lhames@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi Davide,<div><br></div><div>This was posted for review at <a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__reviews.llvm.org_D10552&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=mQ4LZ2PUj9hpadE3cDHZnIdEwhEBrbAstXeMaFoB9tg&m=OZ5r9VBvBy5q2XYcqBT-4wGG0xlR7yIDM58cx6UsgjE&s=3tTxGTj-YXL0OCJprC4kLqmLi5q33gz25dAM-O--ss4&e=" target="_blank">http://reviews.llvm.org/D10552</a> and reviewed by Rui, who Ok'd it. The approach was also verbally Ok'd by Nick Kledzik.</div><div><br></div><div>The phab review should have gone out to lld subscribers, but it doesn't look like emails were sent to llvm-commits. That's odd, but I'm still digesting Phabricator's interface, so I might have missed a tag somewhere. I'll try to check next time I post a review.</div><div><br></div><div>Is there any reason not to return std::error_code from passes? It seems reasonable that a pass may fail. The specific case I have in mind is TLV relocation processing on MachO: If the OS version min is too low we want to error out and report back to the user. This kind of use seems to have been anticipated: The PassManager already returns an error code, although it was always default constructed (no error), and not yet checked. </div><span class="HOEnZb"><font color="#888888"><div><br></div><div>- Lang.</div></font></span></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jun 19, 2015 at 1:02 PM, Davide Italiano <span dir="ltr"><<a href="mailto:davide@freebsd.org" target="_blank">davide@freebsd.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>On Fri, Jun 19, 2015 at 10:51 AM, Lang Hames <<a href="mailto:lhames@gmail.com" target="_blank">lhames@gmail.com</a>> wrote:<br>
</span><span>> Author: lhames<br>
> Date: Fri Jun 19 12:51:46 2015<br>
> New Revision: 240147<br>
><br>
> URL: <a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__llvm.org_viewvc_llvm-2Dproject-3Frev-3D240147-26view-3Drev&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=mQ4LZ2PUj9hpadE3cDHZnIdEwhEBrbAstXeMaFoB9tg&m=OZ5r9VBvBy5q2XYcqBT-4wGG0xlR7yIDM58cx6UsgjE&s=0qOitMInM5VJbjHR-Yy31bxNbzVT1ZG-ygCl4OMZYW4&e=" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=240147&view=rev</a><br>
> Log:<br>
> [lld] Allow LLD passes to return error codes.<br>
><br>
<br>
</span>Hi Lang,<br>
this is a relatively large change. For the future it would be nice if<br>
we can discuss this before checking in (e.g. on llvm-devel).<br>
That said, do you have an use-case for this? In the past we explicitly<br>
tried to not change return type from 'void' to 'std::error_code' just<br>
in case.<br>
<br>
Thanks!<br>
<br>
--<br>
Davide<br>
</blockquote></div><br></div>
</div></div><br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br></div>