<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Nov 8, 2016 at 4:19 PM, Pete Cooper <span dir="ltr"><<a href="mailto:peter_cooper@apple.com" target="_blank">peter_cooper@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Thanks Peter.  This is great!<br>
<br>
The only thing that i’m not sure about is these methods:<br>
<br>
+std::error_code errorToErrorCodeAndEmitErrors(<wbr>LLVMContext &Ctx, Error Err) {<br>
+  if (Err) {<br>
+    std::error_code EC;<br>
+    handleAllErrors(std::move(Err)<wbr>, [&](ErrorInfoBase &EIB) {<br>
+      EC = EIB.convertToErrorCode();<br>
+      Ctx.emitError(EIB.message());<br>
+    });<br>
+    return EC;<br>
+  }<br>
+  return std::error_code();<br>
 }<br>
<br>
I think you can instead just use expectedToError() and errorToErrorCode().  Saying that, you are also logging these to the diagnostic handler and/or context, so perhaps this is special enough that the code you have is the right way to do it.<br></blockquote><div><br></div><div>Right, the invalid bitcode tests would probably fail without this as they are expecting precise error messages.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Will leave it up to you as you know the code better.  These methods are probably only temporary anyway until the rest of the API is converted to Error and Expected. </blockquote><div><br></div><div>Yes, in 1 or 2 patches time these functions will be removed from BitcodeReader.cpp. We will still need one of them for the C API to satisfy the API contract there though.</div><div><br></div><div>Peter</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
LGTM.<br>
<br>
Cheers,<br>
Pete<br>
<div><div class="h5">> On Nov 8, 2016, at 4:10 PM, Peter Collingbourne <<a href="mailto:peter@pcc.me.uk">peter@pcc.me.uk</a>> wrote:<br>
><br>
> pcc created this revision.<br>
> pcc added reviewers: lhames, mehdi_amini, pete.<br>
> pcc added a subscriber: llvm-commits.<br>
><br>
> <a href="https://reviews.llvm.org/D26430" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D26430</a><br>
><br>
> Files:<br>
>  llvm/lib/Bitcode/Reader/<wbr>BitcodeReader.cpp<br>
><br>
</div></div>> <D26430.77281.patch><br>
<br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">-- <div>Peter</div></div></div>
</div></div>