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

[Arthur Eubanks via Phabricator via llvm-commits]

aeubanks added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/EmbedBitcodePass.cpp:51
+
+  return PreservedAnalyses::all();
+}
----------------
paulkirth wrote:
> Is there another option we can use here. `none` seems like overkill. Since this is basically the first thing to run on the module, I'm not sure if there is much point in trying to do anything smarter, but it certainly doesn't feel right.
perhaps something like https://reviews.llvm.org/D153855?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D146776