Hash :
4b1e58d9
Author :
Date :
2024-10-17T09:33:53
Fix for float constant precision in the GLSL backend. Increase the precision of floating point values written out via std::ostringstream. 8 digits is not sufficient to represent all floating point values. Note: the reason the locale test was modified is because it was using a value of 1.9, which has no exact fp32 representation. Increasing the precision causes it to print as 1.8999998 instead of 1.9, failing the test. I've adjusted the value to 1.5, since this does have an exact fp32 representation. (However, note that I couldn't get the test to fail when I removed the locale setting, with either 1.9 or 1.5. Perhaps the locale is being handled at a different level.) Bug: angleproject:374013421 Change-Id: Icb79eb9acd562c83d079f2cc2cdba253220e581e Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5938473 Commit-Queue: Stephen White <senorblanco@chromium.org> Reviewed-by: Geoff Lang <geofflang@chromium.org>
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76 77 78 79 80 81 82 83 84 85 86 87 88 89 90 91 92 93 94 95 96 97 98 99 100 101 102 103 104 105 106 107 108 109 110 111 112 113 114 115 116 117 118 119 120 121 122 123 124 125 126 127 128 129 130 131 132 133 134 135 136 137 138 139 140 141 142 143 144 145 146 147 148 149 150 151 152
//
// Copyright 2002 The ANGLE Project Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
//
#ifndef COMPILER_TRANSLATOR_INFOSINK_H_
#define COMPILER_TRANSLATOR_INFOSINK_H_
#include <math.h>
#include <stdlib.h>
#include "GLSLANG/ShaderLang.h"
#include "compiler/translator/Common.h"
#include "compiler/translator/Severity.h"
namespace sh
{
class ImmutableString;
class TType;
// Returns the fractional part of the given floating-point number.
inline float fractionalPart(float f)
{
float intPart = 0.0f;
return modff(f, &intPart);
}
class ImmutableString;
//
// Encapsulate info logs for all objects that have them.
//
// The methods are a general set of tools for getting a variety of
// messages and types inserted into the log.
//
class TInfoSinkBase
{
public:
TInfoSinkBase() {}
template <typename T>
TInfoSinkBase &operator<<(const T &t)
{
TPersistStringStream stream = sh::InitializeStream<TPersistStringStream>();
stream << t;
sink.append(stream.str());
return *this;
}
// Override << operator for specific types. It is faster to append strings
// and characters directly to the sink.
TInfoSinkBase &operator<<(char c)
{
sink.append(1, c);
return *this;
}
TInfoSinkBase &operator<<(const char *str)
{
sink.append(str);
return *this;
}
TInfoSinkBase &operator<<(const TPersistString &str)
{
sink.append(str);
return *this;
}
TInfoSinkBase &operator<<(const TString &str)
{
sink.append(str.c_str());
return *this;
}
TInfoSinkBase &operator<<(const ImmutableString &str);
TInfoSinkBase &operator<<(const TType &type);
// Make sure floats are written with correct precision.
TInfoSinkBase &operator<<(float f)
{
// Make sure that at least one decimal point is written. If a number
// does not have a fractional part, the default precision format does
// not write the decimal portion which gets interpreted as integer by
// the compiler.
TPersistStringStream stream = sh::InitializeStream<TPersistStringStream>();
if (fractionalPart(f) == 0.0f)
{
stream.precision(1);
stream << std::showpoint << std::fixed << f;
}
else
{
stream.unsetf(std::ios::fixed);
stream.unsetf(std::ios::scientific);
stream.precision(9);
stream << f;
}
sink.append(stream.str());
return *this;
}
// Write boolean values as their names instead of integral value.
TInfoSinkBase &operator<<(bool b)
{
const char *str = b ? "true" : "false";
sink.append(str);
return *this;
}
void erase()
{
sink.clear();
binarySink.clear();
}
int size() { return static_cast<int>(isBinary() ? binarySink.size() : sink.size()); }
const TPersistString &str() const
{
ASSERT(!isBinary());
return sink;
}
const char *c_str() const
{
ASSERT(!isBinary());
return sink.c_str();
}
void prefix(Severity severity);
void location(int file, int line);
bool isBinary() const { return !binarySink.empty(); }
void setBinary(BinaryBlob &&binary) { binarySink = std::move(binary); }
const BinaryBlob &getBinary() const
{
ASSERT(isBinary());
return binarySink;
}
private:
// The data in the info sink is either in human readable form (|sink|) or binary (|binarySink|).
TPersistString sink;
BinaryBlob binarySink;
};
class TInfoSink
{
public:
TInfoSinkBase info;
TInfoSinkBase debug;
TInfoSinkBase obj;
};
} // namespace sh
#endif // COMPILER_TRANSLATOR_INFOSINK_H_