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

i#2399 add reuse time tool : count instructions. #2711

Merged
merged 3 commits into from
Nov 22, 2017

Conversation

s-kanev
Copy link
Contributor

@s-kanev s-kanev commented Nov 21, 2017

Used downstream in derived classes.

@s-kanev
Copy link
Contributor Author

s-kanev commented Nov 22, 2017

AppVeyor test failure seems unrelated (and also surprising -- #2589 should have added code_api|client.drmgr-test to the ignore list, but it's the only failure that doesn't seem ignored).

Copy link
Contributor

@derekbruening derekbruening left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems like the result should be printed out.

@derekbruening
Copy link
Contributor

The drmgr-test ignore from #2589 is for 64-bit only

@s-kanev
Copy link
Contributor Author

s-kanev commented Nov 22, 2017

It seems like the result should be printed out.

Done.

@derekbruening derekbruening merged commit 5ab09cc into DynamoRIO:master Nov 22, 2017
fhahn pushed a commit that referenced this pull request Dec 4, 2017
Adds instruction count to the reuse time tool.

Issue: #2399
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.

2 participants