[PATCH] D8717: Add a callback to FunctionPass to enable skipping execution on a per-function basis

[Vladimir Sukharev]

vsukharev added a subscriber: vsukharev.
vsukharev added a comment.

Hi Akira, Eric
Performing tidying up features for default processors on my checkout, I face a regression due to this patch.
test/CodeGen/ARM/2014-08-04-muls-it.ll failed, after I removed "+db" as a default feature for architectures, including thumbv7.
The effect I see is the following:
without FeatureDB, the pass "SimplifyCFG" is not called at all, and optimization "SpeculativelyExecuteBB" (lib/Transform/Utils/SimplifyCFG.cpp::line 1462) is not called, that is essential for this test.

I guess SpeculativelyExecuteBB must not depend on FeatureDB presence, as well other useful CFG simplifications in this pass.
What's your vision?
Vladimir


Repository:
  rL LLVM

http://reviews.llvm.org/D8717