[PATCH] Add a fence elimination pass

hfinkel at anl.gov hfinkel at anl.gov
Tue Nov 11 09:40:32 PST 2014


Have you been able to get compile-time data for large functions yet? We should move this forward unless there are practical problems with the min-cut algorithm.

================
Comment at: lib/Target/PowerPC/PPCTargetMachine.cpp:184
@@ +183,3 @@
+    // FIXME: breaks a bunch of brittle tests
+    // addPass(createCFGSimplificationPass());
+  }
----------------
If doing this is generally a good thing, then we should always do it (and on what target would that not be true?).

Otherwise, the CFG simplification pass is mostly a wrapper around the SimplifyCFG utility function, and perhaps it should just be called directly from the FencesPRE pass?

http://reviews.llvm.org/D5758






More information about the llvm-commits mailing list