[PATCH] D54635: [ThinLTO] Consolidate cache key computation between new/old LTO APIs

[Steven Wu via Phabricator via llvm-commits]

steven_wu added a comment.

I wonder freestanding is still an issue after r316819. There is a function attribute for that now so it should be respected if the module is built with -ffreestanding and it should create different hash. I will do some digging to see if I can find the original report and how they setup the build. @mehdi_amini, do you remember the original problem?

I also need to do a bit more thinking on the preserve symbol. I think the new LTO API went a completely different route from the old API.


Repository:
  rL LLVM

https://reviews.llvm.org/D54635