[PATCH] D93838: [SCCP] Add Function Specialization pass

[Florian Hahn via Phabricator via llvm-commits]

fhahn added a comment.

I'm not quite sure why the implementation is in `llvm/lib/Transforms/Scalar/FunctionSpecialization.cpp`, rather than `llvm/lib/Transforms/IPO/FunctionSpecialization.cpp`? It's interprocedural only, right? The implementation in `SCCP.cpp` works on both a single function and a module, that's probably with its in the `Scalar` sub-directory.  It's also a separate pass from `IPSCCP`, so I am not sure if the pass definition should be in `llvm/lib/Transforms/IPO/SCCP.cpp`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93838