[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