Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Simplify targets logic and prepare exclusion list for next release #1815

Merged
merged 1 commit into from
Aug 14, 2024

Conversation

cesarzc
Copy link
Contributor

@cesarzc cesarzc commented Aug 3, 2024

This change does two things:

  • Simplify the way we determine whether a target supports QIR.
  • Add a target to the exclusion list in preparation for the next release.

Copy link

github-actions bot commented Aug 3, 2024

Benchmark for a96cff8

Click to view benchmark
Test Base PR %
Array append evaluation 339.1±2.04µs 337.5±1.45µs -0.47%
Array literal evaluation 185.3±0.89µs 185.6±0.80µs +0.16%
Array update evaluation 414.2±1.62µs 411.8±2.10µs -0.58%
Core + Standard library compilation 20.9±0.07ms 20.8±0.11ms -0.48%
Deutsch-Jozsa evaluation 5.0±0.07ms 5.0±0.07ms 0.00%
Large file parity evaluation 34.4±0.22ms 34.5±0.21ms +0.29%
Large input file compilation 12.4±0.16ms 12.5±0.46ms +0.81%
Large input file compilation (interpreter) 47.8±0.79ms 47.8±0.71ms 0.00%
Large nested iteration 33.2±0.30ms 33.4±0.95ms +0.60%
Perform Runtime Capabilities Analysis (RCA) on Deutsch-Jozsa sample 1556.3±31.84µs 1555.3±33.22µs -0.06%
Perform Runtime Capabilities Analysis (RCA) on large file sample 7.7±0.08ms 7.7±0.05ms 0.00%
Perform Runtime Capabilities Analysis (RCA) on teleport sample 1414.6±26.16µs 1414.1±29.77µs -0.04%
Perform Runtime Capabilities Analysis (RCA) on the core and std libraries 27.5±0.11ms 27.7±0.26ms +0.73%
Teleport evaluation 92.9±3.45µs 92.7±4.45µs -0.22%

@cesarzc cesarzc added this pull request to the merge queue Aug 14, 2024
Merged via the queue into main with commit a0a31c6 Aug 14, 2024
19 checks passed
@cesarzc cesarzc deleted the cesarzc/clean-up-known-targets branch August 14, 2024 05:03
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.

3 participants