diff options
| author | Sanjay Patel <spatel@rotateright.com> | 2015-08-11 17:04:31 +0000 |
|---|---|---|
| committer | Sanjay Patel <spatel@rotateright.com> | 2015-08-11 17:04:31 +0000 |
| commit | 070df89928c2c93bf81a406a41fb9d1cb6de9203 (patch) | |
| tree | a806402353d6465abacb14c8b5bd9267711fc1d5 /llvm/lib/CodeGen/SelectionDAG | |
| parent | caddda56aa87649653684e7f3b13963d9f506be8 (diff) | |
| download | bcm5719-llvm-070df89928c2c93bf81a406a41fb9d1cb6de9203.tar.gz bcm5719-llvm-070df89928c2c93bf81a406a41fb9d1cb6de9203.zip | |
fix minsize detection: minsize attribute implies optimizing for size
llvm-svn: 244631
Diffstat (limited to 'llvm/lib/CodeGen/SelectionDAG')
| -rw-r--r-- | llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp b/llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp index 805e68ff11e..244cc7bc227 100644 --- a/llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp +++ b/llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp @@ -3971,10 +3971,9 @@ static SDValue ExpandPowI(SDLoc DL, SDValue LHS, SDValue RHS, return DAG.getConstantFP(1.0, DL, LHS.getValueType()); const Function *F = DAG.getMachineFunction().getFunction(); - // FIXME: Use Function::optForSize(). - if (!F->hasFnAttribute(Attribute::OptimizeForSize) || - // If optimizing for size, don't insert too many multiplies. This - // inserts up to 5 multiplies. + if (!F->optForSize() || + // If optimizing for size, don't insert too many multiplies. + // This inserts up to 5 multiplies. countPopulation(Val) + Log2_32(Val) < 7) { // We use the simple binary decomposition method to generate the multiply // sequence. There are more optimal ways to do this (for example, |
