Rewrite of tools/pti #90
Open
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
… directory and configure the make file to build it along with the other tools.
…nfigure the CARLsim makefiles to build it along with the other tools.
…n be inserted by doxygen, and work on getting the PTI examples to build again.
…with the development version of ECJ (soon to be ECJ 28).
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.
This PR is a rewrite of the old ECJ parameter-tuning interface at
tools/ecj_pti. It replacesecj_ptiwith a newtools/ptidirectory, and tweaks the makefiles so that the PTI builds along withlibcarlsimby default like the other "tools."The old Java code has been deleted entirely (its functionality will be handled out-of-the-box by ECJ 28 when it's released), and the C++ classes and examples that define the standard PTI interface have been simplified and documented.
I do not currently have access to GPU machines, so I could use someone's helping running the tests to verify this PR before it's merged!
This PR does not include updates to the tutorial or user guide. I'll address those with a future PR.