Use outsourced examples in CI #149
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
@wysiwyng may I ask why not LLVM v16+? |
|
The APIs ETISS uses have changed since LLVM 11.x. The bump here from 11.0.0 to 11.1.0 is only to support the LLVM version available from Ubuntu's software repos. |
|
Thanks for elaborating that. I guess we can open up a (low-prio) ticket to deal with supporting the newer api sometime in the future… |
PhilippvK
approved these changes
Jul 4, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #148
LLVM 11.1 was also allowed for ETISS to allow LLVM installation via
apton Ubuntu.The old performance comparisons are omitted for now, they could be added back if desired. Dhrystone benchmarks are still run, the results of which are available in the GitHub Actions artifacts.