[PATCH] D45210: [New-PM] Lift Scop Pipeline to CGSCC-level

[Michael Kruse via Phabricator via llvm-commits]

Meinersbur added a comment.

Thanks for your response. Looking forward to your follow-up patches. I am not too knowledgeable with the new pass manager, so I will let Philip review.
Would it make sense to keep code that analyzes SCoP's function-wise and only use a CGSCC pass when interprocedural analysis is enabled?

If I understand correctly, you are analyzing which data a function accesses. Similar to the "summary functions" in PENCIL <https://hal.inria.fr/hal-01133962/file/ppcg-pencil-v2.pdf>.

Another possibility would have been to inline the callee's scop to get a single scop. We already have a 'hack' for this called `ScopInliner`.

Michael


Repository:
  rPLO Polly

https://reviews.llvm.org/D45210