Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork32k
gh-105481: add pseudo-instructions to the cases DSL#105506
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
Uh oh!
There was an error while loading.Please reload this page.
I'll look into adding unit tests for the generator. |
Actually the test are not running anymore, right? (I mean the ones in Tools/cases_generator/test_generator.py). |
Sadly those must be run manually (since they use pytest) and I have neglected to do so, so they are broken. I've filedgh-105540 for that. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
A great start. I suppose before you land this you are going to make it so that the new metadata is actually used? Or will that be a separate PR?
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
I guess still waiting for the pseudo PR? |
I’m not sure where we would use the metadata for pseudo instructions. It probably makes sense to add assertions in the places where we map them to real instructions, that the metadata is the same. I’ll have a look tomorrow (or so). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Sorry, I was confusing this PR withgh-105482. That oneuses the metadata. This one LGTM!
bedevere-bot commentedJun 11, 2023
|
bedevere-bot commentedJun 11, 2023
|
I found some bugs, here are fixes: |
Uh oh!
There was an error while loading.Please reload this page.