Skip to content

Mark the {min,max}_value functions on integers #[inline]. #25010

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
May 1, 2015

Conversation

huonw
Copy link
Member

@huonw huonw commented Apr 30, 2015

These compile down to mov $CONSTANT, register; ret, but the lack of
#[inline] meant they have a full call ... when used from external
crates.

@rust-highfive
Copy link
Contributor

r? @alexcrichton

(rust_highfive has picked a reviewer for you, use r? to override)

@alexcrichton
Copy link
Member

@bors: r+ df9cc27

@alexcrichton
Copy link
Member

@bors: rollup

@huonw huonw changed the title Mark the {min,max}_value functions in integers #[inline]. Mark the {min,max}_value functions on integers #[inline]. Apr 30, 2015
These compile down to `mov $CONSTANT, register; ret`, but the lack of
`#[inline]` meant they have a full `call ...` when used from external
crates.
@huonw huonw force-pushed the inline-int-extremes branch from df9cc27 to 7ba5f16 Compare April 30, 2015 23:15
@huonw
Copy link
Member Author

huonw commented Apr 30, 2015

@bors r=alexcrichton 7ba5f16 rollup

Manishearth added a commit to Manishearth/rust that referenced this pull request May 1, 2015
…ichton

These compile down to `mov $CONSTANT, register; ret`, but the lack of
`#[inline]` meant they have a full `call ...` when used from external
crates.
Manishearth added a commit to Manishearth/rust that referenced this pull request May 1, 2015
…ichton

These compile down to `mov $CONSTANT, register; ret`, but the lack of
`#[inline]` meant they have a full `call ...` when used from external
crates.
Manishearth added a commit to Manishearth/rust that referenced this pull request May 1, 2015
…ichton

These compile down to `mov $CONSTANT, register; ret`, but the lack of
`#[inline]` meant they have a full `call ...` when used from external
crates.
bors added a commit that referenced this pull request May 1, 2015
@bors bors merged commit 7ba5f16 into rust-lang:master May 1, 2015
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants