search

beehive-lab / TornadoVM

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

[fix] Profiler CopyIns fixed - READ_ONLY data fixed #458

Closed jjfumero closed 3 months ago

jjfumero commented 3 months ago

Description

This patch fixes the copy in events to be used in the profiler when the profiler option is on. Consecuently, it fixes a bug triggering more copies that needed when the device selected was not the default one.

Problem description

The profiler used to print COPY_IN for read_only data if the execution plan was executed multiple times. The problem was that the internal copy functions were not returning the list of events properly.

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 BACKEND=spirv,ptx,opencl
make tests
jjfumero commented 3 months ago

Updated with the latest develop