================ @@ -0,0 +1,34 @@ +#ifndef LLVM_TRANSFORMS_UTILS_SPLITMODULECG_H ---------------- mmjjpp wrote:
I've opened the refactoring PR: https://github.com/llvm/llvm-project/pull/221181— it moves the identical externalize() from SplitModule.cpp and AMDGPUSplitModule.cpp into ModuleUtils as a shared llvm::externalize. I looked at AMDGPUSplitModule.cpp, SplitModule.cpp, SplitModuleByCategory.cpp, but the invocation loops and partitioning algorithms differ enough that I didn't find more to extract — happy to do so if you spot any. @teresajohnson @shiltian Could you review the refactoring PR? Once it lands I'll rebase this one on top and drop the local copy. https://github.com/llvm/llvm-project/pull/198702 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
