[PATCH] D41574: [Transforms] Adding a WeakReassociate pass

Hiroshi Yamauchi via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 22 10:59:00 PDT 2018


yamauchi added inline comments.


================
Comment at: lib/Transforms/IPO/PassManagerBuilder.cpp:393
   addInstructionCombiningPass(MPM);
+  MPM.add(createWeakReassociatePass());           // Reassociate expressions
   addExtensionsToPM(EP_Peephole, MPM);
----------------
If we add it here, should we also add it in the corresponding location in PassBuilder::buildFunctionSimplificationPipeline (lib/Passes/PassBuilder.cpp) for the new pass manager?


Repository:
  rL LLVM

https://reviews.llvm.org/D41574





More information about the llvm-commits mailing list