[PATCH] D42842: [MCJIT] Eliminate overhead in case of multiple subsequent calls to llvm::MCJIT::getFunctionAddress

Lang Hames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 2 10:18:01 PST 2018


lhames added a comment.

Did you measure overhead from this in real-world cases? If so, it would be better to fix this down in RuntimeDyld. The current fix will either do nothing, or re-apply all relocations. If we manage the relocations-to-be-applied list in RuntimeDyld properly (clearing it after each application) we should be able to always apply only the relocations that have not been applied yet.


Repository:
  rL LLVM

https://reviews.llvm.org/D42842





More information about the llvm-commits mailing list