[PATCH] D27722: [GVNHoist] Move GVNHoist to function simplification part of pipeline.

Sebastian Pop via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 13 12:09:20 PST 2016


sebpop added inline comments.


================
Comment at: lib/Transforms/IPO/PassManagerBuilder.cpp:253
-  if(EnableGVNHoist)
-    FPM.add(createGVNHoistPass());
   FPM.add(createLowerExpectIntrinsicPass());
----------------
Could you please try to see the perf of keeping the hoist pass here and adding an extra one below inline as you did? 


https://reviews.llvm.org/D27722





More information about the llvm-commits mailing list