Skip to content
This repository was archived by the owner on May 9, 2024. It is now read-only.

Commit 363c149

Browse files
committed
Fix default compilation options construction in arrow sql runner implemenation
1 parent 6bda9c2 commit 363c149

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

omniscidb/Tests/ArrowSQLRunner/ArrowSQLRunner.cpp

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,6 +49,10 @@ class ArrowSQLRunnerImpl {
4949

5050
bool gpusPresent() { return data_mgr_->gpusPresent(); }
5151

52+
bool isGPUPlatformL0() {
53+
return gpusPresent() && data_mgr_->getGpuMgr()->getPlatform() == GpuMgrPlatform::L0;
54+
}
55+
5256
void printStats() {
5357
std::cout << "Total Calcite parsing time: " << (calcite_time_ / 1000) << "ms."
5458
<< std::endl;
@@ -141,7 +145,7 @@ class ArrowSQLRunnerImpl {
141145
}
142146

143147
CompilationOptions getCompilationOptions(ExecutorDeviceType device_type) {
144-
auto co = CompilationOptions::defaults(device_type);
148+
auto co = CompilationOptions::defaults(device_type, isGPUPlatformL0());
145149
co.hoist_literals = config_->exec.codegen.hoist_literals;
146150
return co;
147151
}

0 commit comments

Comments
 (0)