[PATCH] D21316: Remove the ScalarReplAggregates pass

Karthik Bhat via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 13 20:32:27 PDT 2016


karthikthecool added a subscriber: karthikthecool.

================
Comment at: lib/Transforms/Scalar/Scalar.cpp:205
@@ -204,3 +202,3 @@
 void LLVMAddScalarReplAggregatesPassSSA(LLVMPassManagerRef PM) {
-  unwrap(PM)->add(createScalarReplAggregatesPass(-1, false));
 }
----------------
May be we can remove this function as this is same as LLVMAddScalarReplAggregatesPass ?

================
Comment at: lib/Transforms/Scalar/Scalar.cpp:208
@@ -207,3 +205,3 @@
 
 void LLVMAddScalarReplAggregatesPassWithThreshold(LLVMPassManagerRef PM,
                                                   int Threshold) {
----------------
May be we can remove this function as well as we are not using the Threshold and this is same as LLVMAddScalarReplAggregatesPass ?


http://reviews.llvm.org/D21316





More information about the llvm-commits mailing list