[PATCH] D138654: [IPSCCP] Move the IPSCCP run function under the IPO directory.

Alexandros Lamprineas via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 9 08:44:22 PST 2022


labrinea added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/CMakeLists.txt:69
   ProfileData
   Scalar
   Support
----------------
thakis wrote:
> Since this is now still here, doesn't this render "The LLVMipo library no longer depends on the Scalar component." incorrect?
Right. The commit message is outdated. What this patch achieves is that D126455 no longer needs to add Scalar->IPO dependency which would create a cycle.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D138654



More information about the llvm-commits mailing list