forked from checkedc/checkedc-clang
-
Notifications
You must be signed in to change notification settings - Fork 5
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
Refactor test RUN
commands into a separate tool
#355
Draft
mattmccutchen-cci
wants to merge
5
commits into
test-command-refactoring.base
Choose a base branch
from
test-command-refactoring
base: test-command-refactoring.base
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Refactor test RUN
commands into a separate tool
#355
mattmccutchen-cci
wants to merge
5
commits into
test-command-refactoring.base
from
test-command-refactoring
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
mattmccutchen-cci
referenced
this pull request
Dec 14, 2020
mattmccutchen-cci
force-pushed
the
test-command-refactoring
branch
from
December 14, 2020 20:12
17b42ff
to
3225c31
Compare
mattmccutchen-cci
force-pushed
the
test-command-refactoring.base
branch
from
January 5, 2021 20:34
9a8a7b2
to
d6670af
Compare
mattmccutchen-cci
force-pushed
the
test-command-refactoring
branch
from
January 5, 2021 20:34
f2f1e08
to
370a623
Compare
Fixes: - Comment about #_MARKER_# - abspath
converted to 3c-regtest. `llvm-lit b9_allsafestructp.c` passes, and `./3c-regtest-unconvert.py b9_allsafestructp.c` prints its content as of the test-command-refactoring.base branch.
3c-regtest. Made needed enhancements to the 3c-regtest tools: - Add support for the rest of the RUN scripts. - Add support for RUN scripts with blank lines in the middle, as used by the common-testgenerator script. A few tests are failing verification, i.e., their existing RUN scripts didn't match the ones generated by the test updating programs. To investigate.
mattmccutchen-cci
force-pushed
the
test-command-refactoring
branch
from
January 13, 2021 20:49
370a623
to
6691373
Compare
This was referenced Jan 14, 2021
13 tasks
14 tasks
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.
Work in progress. Part of #346.
I'm currently using a PR target branch
test-command-refactoring.base
that I set up specifically as a merge of the changes that this PR depends on but that are not part of the PR, so we can see the proper PR diff. After the dependencies are merged, I'll change the PR target branch to the default branch.Current status: All tests managed by the test updating programs are converted to
3c-regtest
, but a few are failing verification, i.e., their existing RUN scripts didn't match the ones generated by the test updating programs.