[PATCH] D34063: [ThinLTO] YAML traits for module summaries

Charles Saternos via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 20 09:19:15 PDT 2017


ncharlie added inline comments.


================
Comment at: include/llvm/IR/ModuleSummaryIndexYAML.h:406
+  static void inputOne(IO &io, StringRef Key, NameMapTy &V) {
+    // do nothing - this map is only used for output
+  }
----------------
tejohnson wrote:
> Should there be an assert here that this is never invoked then?
I realized that mapping wasn't necessary so I removed it.


https://reviews.llvm.org/D34063





More information about the llvm-commits mailing list