<html><head><meta http-equiv="content-type" content="text/html; charset=utf-8"></head><body dir="auto"><div></div><div>Thanks for reviewing the patches. And yes, 3 and 4 are no longer useful. I am seeking better alternatives to achieve them.</div><div><br></div><div>Steven </div><div><br>On Nov 30, 2016, at 1:35 AM, Nico Weber <<a href="mailto:thakis@chromium.org">thakis@chromium.org</a>> wrote:<br><br></div><blockquote type="cite"><div><div dir="ltr">It looks like patches 1 and 2 made it but 3 and 4 didn't. Do you no longer need them?</div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Feb 29, 2016 at 2:08 PM, Steven Wu via cfe-commits <span dir="ltr"><<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Ping. I don't know who is the best review for the patches. Thanks for Rafael looking at the LLVM change. Richard, do you have any opinions on the clang changes?<br>
<br>
Thanks<br>
<span class="HOEnZb"><font color="#888888"><br>
Steven<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
> On Feb 18, 2016, at 9:57 AM, Steven Wu <<a href="mailto:stevenwu@apple.com">stevenwu@apple.com</a>> wrote:<br>
><br>
> Hi all<br>
><br>
> I put up some patches for embedding bitcode inside the object file (-fembed-bitcode) option. As I described in the dev list before, the new option can create normal object file with bitcode embedded in a special section. You can easily recreate the same object file with the embedded bitcode in it.<br>
><br>
> I split the patch into several parts:<br>
> llvm patch:<br>
> <a href="http://reviews.llvm.org/D17388" rel="noreferrer" target="_blank">http://reviews.llvm.org/D17388</a><wbr>: Introduce the section for embedding bitcode in MachO file.<br>
><br>
> There are four clang patches:<br>
> <a href="http://reviews.llvm.org/D17390" rel="noreferrer" target="_blank">http://reviews.llvm.org/D17390</a><wbr>: Implementing the clang driver for the new option.<br>
> <a href="http://reviews.llvm.org/D17392" rel="noreferrer" target="_blank">http://reviews.llvm.org/D17392</a><wbr>: Teach clang how to embed bitcode into the object file.<br>
> <a href="http://reviews.llvm.org/D17393" rel="noreferrer" target="_blank">http://reviews.llvm.org/D17393</a><wbr>: Slightly tweak the bitcode emitted in embed bitcode stage 1.<br>
> <a href="http://reviews.llvm.org/D17394" rel="noreferrer" target="_blank">http://reviews.llvm.org/D17394</a><wbr>: Reduce the amount of option gets embedded in the bitcode by introducing a whitelist and a blacklist.<br>
><br>
> Let me know if anyone is interested in helping reviewing these changes.<br>
><br>
> Thanks<br>
><br>
> Steven<br>
><br>
><br>
><br>
<br>
______________________________<wbr>_________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/cfe-commits</a><br>
</div></div></blockquote></div><br></div>
</div></blockquote></body></html>