[PATCH] D45850: [ELF] Read the call graph profile from object files.

Rafael Avila de Espindola via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 27 10:27:25 PDT 2018


espindola added inline comments.


================
Comment at: ELF/InputFiles.cpp:219
+  for (const Elf_CGProfile &CGPE : CGProfile) {
+    uint64_t &C = Config->CallGraphProfileSymbols[std::make_pair(
+        &getSymbol(CGPE.cgp_from), &getSymbol(CGPE.cgp_to))];
----------------
Why do you need to store this in Config? The code that is walking CallGraphProfileSymbols could just as easily walk CGProfile, no?


https://reviews.llvm.org/D45850





More information about the llvm-commits mailing list