Skip to content

Conversation

@mikepapadim
Copy link
Member

No description provided.

@mikepapadim mikepapadim self-assigned this Jun 9, 2025
@CLAassistant
Copy link

CLAassistant commented Jun 9, 2025

CLA assistant check
All committers have signed the CLA.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Updates the TornadoVM API to leverage the new pre-compilation feature instead of the previous warm-up approach and bumps the external TornadoVM submodule.

  • Replaces withWarmUp() with withPreCompilation() to force JIT compilation from Java to GPU code.
  • Updates the external/tornadovm submodule reference to the latest commit.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/main/java/com/example/tornadovm/TornadoVMMasterPlan.java Swapped the warm-up method call for the new pre-compilation method
external/tornadovm Bumped submodule commit to da80056791474135a310a2faac718fa6361ab754
Comments suppressed due to low confidence (1)

src/main/java/com/example/tornadovm/TornadoVMMasterPlan.java:69

  • The preceding comment still refers to 'warmup iterations', but the code now invokes withPreCompilation(). Update the comment to accurately describe the new pre-compilation step.
tornadoVMPlan.executionPlan.withPreCompilation(); // Force JIT compilation from Java to GPU code

@mikepapadim mikepapadim merged commit 90a719f into beehive-lab:main Jun 9, 2025
1 check was pending
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants