[PATCH] D93762: SCCP: Refactor SCCPSolver
Florian Hahn via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Dec 23 07:45:56 PST 2020
fhahn added a comment.
I think it makes sense to expose this more widely in general. But could you share the places where you intend to make use of it?
I think we should restrict the public interface exposed to what really is required for external clients, and keep other details in the `.cpp` file (e.g. most of the members related to function/argument tracking and the worklists could stay in the `.cpp`
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D93762/new/
https://reviews.llvm.org/D93762
More information about the llvm-commits
mailing list