<div dir="ltr">I'm sorry, I actually missed this comment. My bad.<br><br><div class="gmail_quote"><div dir="ltr">On Wed, Oct 14, 2015 at 10:44 AM Mehdi Amini via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Rafael on IRC mentioned that it would be nice to have the option of progressively deprecating bitcode in the future. For instance LLVM 4.1 could be able to read bitcode generated by version 3.14 (random number) and above. And 4.2 deprecate in turn anything older than 3.19 (random number again).<br>
<br>
We can do that by encoding the version directly (major.minor.patch) in separate bytes and use it instead of the epoch.<br>
<br>
What’s your take on this?<br></blockquote><div><br></div><div>Ultimately, I'm worried about adding this without having a clear use case. For example, it means we need to be careful about exactly when we bump the minor version. It also isn't clear how to define it if people are managing their own releases out-of-band with the upstream repository.</div><div><br></div><div>We can extend this with an IDENTIFICATION_CODE_MAJOR, IDENTIFICATION_CODE_MINOR, or whatever other fields we're interested within this block in the future. So I think we should move forward with this patch now, and revisit more detailed information here when there is a clear plan around both 1) how to update them meaningfully and 2) what problem they are solving.</div><div><br></div><div>That make sense to you Rafael?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Thanks,<br>
<br>
—<br>
Mehdi<br>
<br>
<br>
> On Oct 14, 2015, at 12:59 PM, Mehdi AMINI <<a href="mailto:mehdi.amini@apple.com" target="_blank">mehdi.amini@apple.com</a>> wrote:<br>
><br>
> joker.eph updated this revision to Diff 37380.<br>
> joker.eph added a comment.<br>
><br>
> Add the "epoch" field along with the producer string.<br>
><br>
><br>
> <a href="http://reviews.llvm.org/D13666" rel="noreferrer" target="_blank">http://reviews.llvm.org/D13666</a><br>
><br>
> Files:<br>
>  include/llvm/Bitcode/LLVMBitCodes.h<br>
>  lib/Bitcode/Reader/BitcodeReader.cpp<br>
>  lib/Bitcode/Writer/BitcodeWriter.cpp<br>
>  test/Bitcode/identification.ll<br>
>  tools/llvm-bcanalyzer/llvm-bcanalyzer.cpp<br>
><br>
> <D13666.37380.patch><br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div></div>