frank400 commented on pull request #9724:
URL: https://github.com/apache/arrow/pull/9724#issuecomment-804032232
@emkornfield, can you help me identify the building problem that I'm getting
with AMD64 Windows MinGW 32 C++ action? This is the log that I'm getting:
```
D:/a/arrow/arrow/cpp/src/gandiva/dex_visitor.h:65:16: error: 'virtual void
gandiva::DexVisitor::Visit(const gandiva::InExprDexBase<long double>&)' cannot
be overloaded with 'virtual void gandiva::DexVisitor::Visit(const
gandiva::InExprDexBase<long double>&)'
65 | virtual void Visit(const InExprDexBase<double_t>& dex) = 0;
| ^~~~~
D:/a/arrow/arrow/cpp/src/gandiva/dex_visitor.h:64:16: note: previous
declaration 'virtual void gandiva::DexVisitor::Visit(const
gandiva::InExprDexBase<long double>&)'
64 | virtual void Visit(const InExprDexBase<float_t>& dex) = 0;
| ^~~~~
D:/a/arrow/arrow/cpp/src/gandiva/dex_visitor.h:71:8: error: 'void
gandiva::DexDefaultVisitor::Visit(const gandiva::InExprDexBase<long double>&)'
cannot be overloaded with 'void gandiva::DexDefaultVisitor::Visit(const
gandiva::InExprDexBase<long double>&)'
71 | void Visit(const DEX_CLASS& dex) override { DCHECK(0); }
| ^~~~~
D:/a/arrow/arrow/cpp/src/gandiva/dex_visitor.h:90:3: note: in expansion of
macro 'VISIT_DCHECK'
90 | VISIT_DCHECK(InExprDexBase<double_t>)
| ^~~~~~~~~~~~
D:/a/arrow/arrow/cpp/src/gandiva/dex_visitor.h:71:8: note: previous
declaration 'void gandiva::DexDefaultVisitor::Visit(const
gandiva::InExprDexBase<long double>&)'
71 | void Visit(const DEX_CLASS& dex) override { DCHECK(0); }
| ^~~~~
D:/a/arrow/arrow/cpp/src/gandiva/dex_visitor.h:89:3: note: in expansion of
macro 'VISIT_DCHECK'
89 | VISIT_DCHECK(InExprDexBase<float_t>)
| ^~~~~~~~~~~~
In file included from
D:/a/arrow/arrow/cpp/src/gandiva/bitmap_accumulator.h:23,
from
D:/a/arrow/arrow/cpp/src/gandiva/bitmap_accumulator_test.cc:18,
from
src/gandiva/CMakeFiles/gandiva-internals-test.dir/Unity/unity_0_cxx.cxx:3:
D:/a/arrow/arrow/cpp/src/gandiva/dex.h:336:7: error: redefinition of 'class
gandiva::InExprDex<long double>'
336 | class InExprDex<double_t> : public InExprDexBase<double_t> {
| ^~~~~~~~~~~~~~~~~~~
D:/a/arrow/arrow/cpp/src/gandiva/dex.h:326:7: note: previous definition of
'class gandiva::InExprDex<long double>'
326 | class InExprDex<float_t> : public InExprDexBase<float_t> {
| ^~~~~~~~~~~~~~~~~~
In file included from
D:/a/arrow/arrow/cpp/src/gandiva/llvm_generator_test.cc:18,
from
src/gandiva/CMakeFiles/gandiva-internals-test.dir/Unity/unity_0_cxx.cxx:8:
D:/a/arrow/arrow/cpp/src/gandiva/llvm_generator.h:112:10: error: 'void
gandiva::LLVMGenerator::Visitor::Visit(const gandiva::InExprDexBase<long
double>&)' cannot be overloaded with 'void
gandiva::LLVMGenerator::Visitor::Visit(const gandiva::InExprDexBase<long
double>&)'
112 | void Visit(const InExprDexBase<double_t>& dex) override;
| ^~~~~
D:/a/arrow/arrow/cpp/src/gandiva/llvm_generator.h:111:10: note: previous
declaration 'void gandiva::LLVMGenerator::Visitor::Visit(const
gandiva::InExprDexBase<long double>&)'
111 | void Visit(const InExprDexBase<float_t>& dex) override;
| ^~~~~
```
I am also getting a similar problem with the Travis CI build for 39056.2 C++
on s390x.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
