[PATCH] D119878: [FuncSpec] Remove definitions of fully specialized functions.

[Chuanqi Xu via Phabricator via llvm-commits]

ChuanqiXu added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/FunctionSpecialization.cpp:515
+        })) {
       Solver.markFunctionUnreachable(AI.Fn);
+      FullySpecialized.push_back(AI.Fn);
----------------
It surprised me that the unreachable function wouldn't be removed. Have you tested to run opt -O3 for the tests? I think they might be removed in later passes.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D119878