diff options
| author | Chris Lattner <sabre@nondot.org> | 2011-04-09 06:57:13 +0000 |
|---|---|---|
| committer | Chris Lattner <sabre@nondot.org> | 2011-04-09 06:57:13 +0000 |
| commit | cfe5aa65d213c6209392a03ed075a34167cd28b9 (patch) | |
| tree | eba5dc50caa729cb3c38a23e606d97a144521699 | |
| parent | 17822fcde92f1cf1125fa2f937ca58c4fa43b295 (diff) | |
| download | bcm5719-llvm-cfe5aa65d213c6209392a03ed075a34167cd28b9.tar.gz bcm5719-llvm-cfe5aa65d213c6209392a03ed075a34167cd28b9.zip | |
Avoid excess precision issues that lead to generating host-compiler-specific code.
Switch lowering probably shouldn't be using FP for this. This resolves PR9581.
llvm-svn: 129199
| -rw-r--r-- | llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp b/llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp index f751bb708e7..8fb881b2700 100644 --- a/llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp +++ b/llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp @@ -2019,9 +2019,13 @@ bool SelectionDAGBuilder::handleBTSplitSwitchCase(CaseRec& CR, APInt Range = ComputeRange(LEnd, RBegin); assert((Range - 2ULL).isNonNegative() && "Invalid case distance"); - double LDensity = (double)LSize.roundToDouble() / + // Use volatile double here to avoid excess precision issues on some hosts, + // e.g. that use 80-bit X87 registers. + volatile double LDensity = + (double)LSize.roundToDouble() / (LEnd - First + 1ULL).roundToDouble(); - double RDensity = (double)RSize.roundToDouble() / + volatile double RDensity = + (double)RSize.roundToDouble() / (Last - RBegin + 1ULL).roundToDouble(); double Metric = Range.logBase2()*(LDensity+RDensity); // Should always split in some non-trivial place |
