[PATCH] D54695: [PM] Port Scalarizer to the new pass manager.

Markus Lavin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 19 05:17:23 PST 2018


markus added a comment.

Agree that splitting would give a cleaner solution. Will give that a go.



================
Comment at: lib/Transforms/Scalar/Scalarizer.cpp:159
                    public InstVisitor<Scalarizer, bool> {
+  friend class llvm::ScalarizerPass;
 public:
----------------
fedor.sergeev wrote:
> Can this friend declaration be removed?
If I do the splitting suggested above then yes, most likely it can. 


Repository:
  rL LLVM

https://reviews.llvm.org/D54695





More information about the llvm-commits mailing list