[PATCH] D12773: [PM] Port SROA to the new pass manager.

Sanjoy Das via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 10 23:40:41 PDT 2015


sanjoy added inline comments.

================
Comment at: lib/Transforms/Scalar/SROA.cpp:4202
@@ -4306,5 +4201,3 @@
 
-bool SROA::runOnFunction(Function &F) {
-  if (skipOptnoneFunction(F))
-    return false;
-
+PreservedAnalyses SROA::runImpl(Function &F, LLVMContext &RunC,
+                                DominatorTree &RunDT, AssumptionCache &RunAC) {
----------------
This is super minor, but do you need to pass the context separately?  Is it ever different from `F.getContext()`?


http://reviews.llvm.org/D12773





More information about the llvm-commits mailing list