[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