[PATCH] D18494: ThinLTO: add module caching handling.

Mehdi AMINI via llvm-commits llvm-commits at lists.llvm.org
Sat Mar 26 22:58:56 PDT 2016


joker.eph added inline comments.

================
Comment at: lib/LTO/ThinLTOCodeGenerator.cpp:334
@@ +333,3 @@
+      Hasher.write((const char *)&ModHash[0], sizeof(ModHash));
+    }
+
----------------
joker.eph wrote:
> Unfortunately this is not enough to handle internalization, or linkonce_odr optimization like in D18346, I'll have to give more thought about it.
Another issue is that promotion will rename later a global with a name that will different between two different links if the object is not in the same position. The hashing here does not account for this unfortunately. 
It seems like an intrinsically hard problem to solve (unless we give-up on sharing objects across two different links)


http://reviews.llvm.org/D18494





More information about the llvm-commits mailing list