diff options
author | Hans Wennborg <hans@hanshq.net> | 2014-01-25 01:18:18 +0000 |
---|---|---|
committer | Hans Wennborg <hans@hanshq.net> | 2014-01-25 01:18:18 +0000 |
commit | 503793e834a8d0e5db8b192ce66e4916389249b4 (patch) | |
tree | 797cd4908c85538f8c746388ece092a62c1f72d0 /lib/Transforms/Scalar/Scalar.cpp | |
parent | 2760cc2967c3d289a88a2d9527caa02f0d9e82b6 (diff) | |
download | llvm-503793e834a8d0e5db8b192ce66e4916389249b4.tar.gz llvm-503793e834a8d0e5db8b192ce66e4916389249b4.tar.bz2 llvm-503793e834a8d0e5db8b192ce66e4916389249b4.tar.xz |
Revert "Add Constant Hoisting Pass" (r200034)
This commit caused -Woverloaded-virtual warnings. The two new
TargetTransformInfo::getIntImmCost functions were only added to the superclass,
and to the X86 subclass. The other targets were not updated, and the
warning highlighted this by pointing out that e.g. ARMTTI::getIntImmCost was
hiding the two new getIntImmCost variants.
We could pacify the warning by adding "using TargetTransformInfo::getIntImmCost"
to the various subclasses, or turning it off, but I suspect that it's wrong to
leave the functions unimplemnted in those targets. The default implementations
return TCC_Free, which I don't think is right e.g. for ARM.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@200058 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Transforms/Scalar/Scalar.cpp')
-rw-r--r-- | lib/Transforms/Scalar/Scalar.cpp | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/lib/Transforms/Scalar/Scalar.cpp b/lib/Transforms/Scalar/Scalar.cpp index f774874cb4..6983d06b68 100644 --- a/lib/Transforms/Scalar/Scalar.cpp +++ b/lib/Transforms/Scalar/Scalar.cpp @@ -30,7 +30,6 @@ void llvm::initializeScalarOpts(PassRegistry &Registry) { initializeADCEPass(Registry); initializeSampleProfileLoaderPass(Registry); initializeCodeGenPreparePass(Registry); - initializeConstantHoistingPass(Registry); initializeConstantPropagationPass(Registry); initializeCorrelatedValuePropagationPass(Registry); initializeDCEPass(Registry); |