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

Inconsistent stackmap frames for TwoWord arguments #85

Closed
marchof opened this issue Mar 1, 2013 · 1 comment
Closed

Inconsistent stackmap frames for TwoWord arguments #85

marchof opened this issue Mar 1, 2013 · 1 comment
Labels
type: bug 🐛 Something isn't working
Milestone

Comments

@marchof
Copy link
Member

marchof commented Mar 1, 2013

Inserted frames are incorrect in the following situation:

  • At least one method argument is TwoWord (long or double)
  • A stackmap frame does not contain these arguments as local variables (not used any more).

These situations do not occure with the Java compiler, but with certain tools. See issue jacoco/eclemma#46

In this case the number of Top elements which need so be inserted before the probe variable is too small as the TwoWord size is not properly considered.

@drangor
Copy link

drangor commented Mar 11, 2013

Apologies for the delay in testing this, I've been on hols.

The latest snapshot fixes the problems I had, both in the small test project and the original library where I encountered it. Thanks for the quick fix Marchof. Looking forward to the 0.6.3 release.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
type: bug 🐛 Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants