beehive-lab / TornadoVM

TornadoVM: A practical and efficient heterogeneous programming framework for managed languages
https://www.tornadovm.org
Apache License 2.0
1.17k stars 110 forks source link

Fix Native Test to pass on modern platforms with no support of FP64 #358

Closed jjfumero closed 5 months ago

jjfumero commented 5 months ago

Description

The purpose of the native tests is to check that the middle-ware API works as expected, not to check if we can run FP64. Since many OpenCL platforms these days do not support FP64, we do not need to check this in this unit-tests. This PR just replaces the double with float and test the interaction with the JIT compiler and runtime using the internal APIs.

Note that this PR is built in top of #357 .

Backend/s tested

Mark the backends affected by this PR.

OS tested

Mark the OS where this PR is tested.

Did you check on FPGAs?

If it is applicable, check your changes on FPGAs.

How to test the new patch?

$ make 
$ test-native.sh 

Testing the Native OpenCL API
Running Native: uk.ac.manchester.tornado.drivers.opencl.tests.TestOpenCLJITCompiler ................ [PASS]
jjfumero commented 5 months ago

This is ready for review