Skip to content

Conversation

@graalvmbot
Copy link
Collaborator

On compilation failure, PEBenchmark did not indicate a problem (other than reporting NaN for compilation timings), because the exception it throws gets thrown from a callback on a compiler thread.

This change:

  • adds an explicit check for empty data points.
  • updates listener callbacks to log errors to stderr; adds an mx benchmark failurePattern to check for error logs.
  • adds a new gate-truffle-test-compiler-benchmarks-graalvm-ce-latest-linux-amd64 tier3 job to test for compilation failures.

@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Dec 2, 2025
@graalvmbot graalvmbot merged commit f90a690 into master Dec 2, 2025
12 of 13 checks passed
@graalvmbot graalvmbot deleted the mdsouza/GR-71655-pebenchmark-error-handling branch December 2, 2025 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

OCA Verified All contributors have signed the Oracle Contributor Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants