[PATCH] D120471: [Bitcode] Fully support opaque pointer auto upgrade

[Arthur Eubanks via Phabricator via llvm-commits]

aeubanks added a comment.

sorta makes sense, but I'm not understanding in what case we'd create a new virtual type id, can you give an example?



================
Comment at: llvm/lib/Bitcode/Reader/BitcodeReader.cpp:492
   DenseMap<unsigned, SmallVector<unsigned, 1>> ContainedTypeIDs;
+  /// In some cases, we need to create a type ID for a type that was not
+  /// explicitly encoded in the bitcode. This map stores the new type ID that
----------------
could you give an example here of a case where we need a new type ID not in the bitcode? 


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

https://reviews.llvm.org/D120471