[PATCH] D136208: [llvm-reduce] Attempt to strip debug info

[David Blaikie via Phabricator via llvm-commits]

dblaikie added inline comments.


================
Comment at: llvm/tools/llvm-reduce/deltas/StripDebugInfo.cpp:24
+  if (HasDebugInfo && !O.shouldKeep()) {
+    StripSymbolsPass P;
+    ModuleAnalysisManager MAM;
----------------
Looks like this also strips function/global variable names and register names - any chance we don't want to strip those too? Should we call this llvm-reduce pass "StripSymbols" rather than "stripDebugInfo" to be consistent with the underlying pass and its slightly different/broader semantics?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D136208