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

use LocalCopyProp in bmv2 and v12test backends #36

Closed
wants to merge 1 commit into from

Conversation

ChrisDodd
Copy link
Contributor

Adding a call to these backends works fine, just changing the dump output in the expected ways.

@mbudiu-bfn
Copy link
Contributor

Is this only done for actions?
There should be tons of copies in control blocks in some v1to12 tests due to the inlining.

@ChrisDodd
Copy link
Contributor Author

Yes, currently this only works on actions. It shoudl be relativelye easy
to extend to control blocks, so I'll look into that soon

On Mon, May 2, 2016 at 5:51 PM, mbudiu-bfn notifications@github.com wrote:

Is this only done for actions?
There should be tons of copies in control blocks in some v1to12 tests due
to the inlining.


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#36 (comment)

@ChrisDodd ChrisDodd force-pushed the cdodd branch 2 times, most recently from 296b695 to a49a325 Compare May 5, 2016 04:31
@ChrisDodd ChrisDodd closed this May 5, 2016
@ChrisDodd ChrisDodd deleted the local-copyprop branch December 2, 2016 20:06
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