[PATCH] D90260: [llvm-reduce] Add reduction for aliases and special globals.

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: llvm/tools/llvm-reduce/deltas/ReduceAliasAndSpecialGlobals.cpp:21
+
+static auto SpecialGlobalNames = {"llvm.used", "llvm.compiler.used"};
+
----------------
I almost think reducing these should be a separate step


================
Comment at: llvm/tools/llvm-reduce/deltas/ReduceAliasAndSpecialGlobals.cpp:32
+    if (!O.shouldKeep()) {
+      GA.replaceAllUsesWith(UndefValue::get(GA.getType()));
+      GA.eraseFromParent();
----------------
Would it make more sense to replace this with the aliasee as a first step, and rely on ordinary global reduction as a separate phase?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90260