[PATCH] D146776: Reland [llvm] Preliminary fat-lto-objects support

Paul Kirth via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 28 14:30:27 PDT 2023


paulkirth added inline comments.


================
Comment at: llvm/lib/IR/StructuralHash.cpp:62
     // used/compiler.used don't affect analyses.
-    if (GV.getName() == "llvm.compiler.used" || GV.getName() == "llvm.used")
+    if (GV.getName() == "llvm.compiler.used" || GV.getName() == "llvm.used" ||
+        GV.getName() == "llvm.embedded.object")
----------------
aeubanks wrote:
> maybe we should exclude anything that begins with `llvm.`, but that should be a separate patch, so this lgtm
Filed https://github.com/llvm/llvm-project/issues/63590. I can make the follow up patch, but I may not get a chance till tomorrow.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D146776



More information about the llvm-commits mailing list