[PATCH] D27775: [ThinLTO] Import composite types as declarations

[Mehdi AMINI via Phabricator via llvm-commits]

mehdi_amini added inline comments.


================
Comment at: lib/Bitcode/Reader/MetadataLoader.cpp:733
+      // by the importing module, in which case the existing definition is
+      // used.
+      if (IsImporting && !ImportFullTypeDefinitions &&
----------------
Can you avoid calling all the getMD and so above in the case you're nullifying them?
As I'm working on lazy-loading, this would still trigger the load from bitcode and create the MD into memory even if you don't attach them in the MD graph.


https://reviews.llvm.org/D27775