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

Steven Wu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 26 10:06:41 PST 2018


steven_wu accepted this revision.
steven_wu added a comment.
This revision is now accepted and ready to land.

I did some experiment and put some more thought into this patch. Not hashing PreservedSymbol passed to the C API is fine because the reason you mentioned but freestanding/no-builtin bits stills need to be handled separately in current status. That should be a different discussion so this patch is LGTM.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D54635/new/

https://reviews.llvm.org/D54635





More information about the llvm-commits mailing list