[PATCH] D73422: [IR] Delete MODULE_CODE_DEPLIB

[Mehdi AMINI via Phabricator via llvm-commits]

mehdi_amini added inline comments.


================
Comment at: llvm/lib/Bitcode/Reader/BitcodeReader.cpp:3605
-      break;
-    }
     case bitc::MODULE_CODE_SECTIONNAME: {  // SECTIONNAME: [strchr x N]
----------------
void wrote:
> mehdi_amini wrote:
> > Isn't this breaking bitcode backward compatibility?
> > 
> This used to be okay when changing major versions. I'm not too familiar with the current versioning system though or whether that's still the policy.
It changed when we changed the version numbering to not have major/minor anymore. At the time (IIRC) we considered we would break compatibility only if it is really motivated. The current section of the developer policy: https://llvm.org/docs/DeveloperPolicy.html#ir-backwards-compatibility



Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D73422/new/

https://reviews.llvm.org/D73422