<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, May 24, 2016 at 5:09 PM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">With this patch the linker would be using all 3 error handling systems<br>
in llvm, which is way too much.<br>
<br>
What I think needs to happen is for it to have an Error class that is<br>
not converted to error_code. We also have to decide if we are keeping<br>
a list of error or just failing early. I don't think it is worth it<br>
keeping multiple errors, so I would suggest refactoring the code to<br>
return early and only store the error when it hits an interface that<br>
returns void.<br>
<br>
We should also stop logging error to the streamer and instead have the<br>
caller uses log on the Error.<br>
<br>
The attached patch implements part of that. What do you think?<br></blockquote><div><br></div><div>That all seems reasonable to me. I've updated the revision in phab with a new patch based on yours that does all of that.</div><div><br></div><div>I also moved StringError into the Support library, as it seems like a useful general facility. Not sure what others think about that.</div><div><br></div><div>Thanks,</div></div>-- <br><div class="gmail_signature"><div dir="ltr">-- <div>Peter</div></div></div>
</div></div>