[PATCH] D82514: Remove references to the 4.0 release as a major breaking (NFC)

Thu Jun 25 15:18:12 PDT 2020

mehdi_amini marked an inline comment as done.
mehdi_amini added inline comments.

================
Comment at: llvm/lib/IR/DataLayout.cpp:263
     case 's':
       // Ignored for backward compatibility.
       break;
----------------
echristo wrote:
> dexonsmith wrote:
> > I think @echristo is suggesting updating this comment to match the new ones elsewhere ("Deprecated, but ...").
> I was yes :)
Ah OK.

This isn't bitcode related though: we have a few occurrences of these in the LL parser as well that I didn't touch. I don't think there is a policy on these, what should I write?  `// Deprecated, but ignoring here to preserve loading older textual llvm ASM file` ?

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82514