diff options
| author | Luboš Luňák <l.lunak@collabora.com> | 2022-08-30 15:48:56 +0200 |
|---|---|---|
| committer | Luboš Luňák <l.lunak@collabora.com> | 2022-08-31 09:13:54 +0200 |
| commit | 067610f46ae57f88439a1b7c5a28209d129e641e (patch) | |
| tree | 0a630ab3cd71c1273736b12707a5183549bd3a25 /sc | |
| parent | 975fa7d700d50d376090de6c370aa00b8b38e8cf (diff) | |
do not accept strings for opencl equal operator
This is part of "using string hashes for strings is kind of stupid"
removal. And it seems this is never used for all of Calc's tests.
Just keep it for numeric comparison.
Change-Id: I1c5e3b10c9cacf3d163a675373f14d879a857db8
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/139052
Tested-by: Jenkins
Reviewed-by: Luboš Luňák <l.lunak@collabora.com>
Diffstat (limited to 'sc')
| -rw-r--r-- | sc/source/core/opencl/formulagroupcl.cxx | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/sc/source/core/opencl/formulagroupcl.cxx b/sc/source/core/opencl/formulagroupcl.cxx index 6855434d37b7..9ab5b83057e9 100644 --- a/sc/source/core/opencl/formulagroupcl.cxx +++ b/sc/source/core/opencl/formulagroupcl.cxx @@ -93,7 +93,6 @@ const char* const publicFunc = "double legalize(double a, double b) { return isnan(a)?b:a;}\n" "double fsub(double a, double b) { return a-b; }\n" "double fdiv(double a, double b) { return a/b; }\n" - "double strequal(unsigned a, unsigned b) { return (a==b)?1.0:0; }\n" "int is_representable_integer(double a) {\n" " long kMaxInt = (1L << 53) - 1;\n" " if (a <= as_double(kMaxInt))\n" @@ -1787,11 +1786,10 @@ public: virtual std::string Gen2( const std::string& lhs, const std::string& rhs ) const override { std::stringstream ss; - ss << "strequal(" << lhs << "," << rhs << ")"; + ss << "(" << lhs << " == " << rhs << ")"; return ss.str(); } virtual std::string BinFuncName() const override { return "eq"; } - virtual bool takeString() const override { return true; } }; class OpLessEqual : public Binary |