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

beam-fp-insts #152

Closed
wants to merge 6 commits into from
Closed

beam-fp-insts #152

wants to merge 6 commits into from

Conversation

proxyles
Copy link
Contributor

No description provided.

The compiler shouldn't crash when fed an already-optimised BEAM assembly file.
If an allocate_zero instruction is fed to beam_block and the beam_type
pass is not used afterwards (e.g. with erlc +no_topt), the 'no_opt' atom
will be rejected by beam_flatten.
Any init instruction following an allocate is put in the Inits list of the
corresponding alloc tuple.
This makes applying the pass a second time a no-op.
@OTP-Maintainer
Copy link

Patch has passed first testings and has been assigned to be reviewed

@nox
Copy link
Contributor

nox commented Dec 5, 2013

Why is my name "nox" for that patch on the Development page?

@garazdawi
Copy link
Contributor

Because free-text fields are evil

@proxyles
Copy link
Contributor Author

Graduated

@proxyles proxyles closed this Jan 23, 2014
uabboli pushed a commit to uabboli/otp that referenced this pull request Dec 1, 2020
* Support a map generator

Simply parametrize it from using two generators for keys and values.
The map is built from using a list of key/value pairs and converting
them.

This implementation was chosen for its simplicity, but also because it
allows to use a PropEr version that supports maps in an older Erlang
version without compile errors (nor runtime errors as long as the
generator is not used).

* Fix type spec to match rest of module
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.

None yet

4 participants