Skip to content

Implement pattern ranges for all numeric types. #951

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

Closed
wants to merge 1 commit into from

Conversation

jdm
Copy link
Contributor

@jdm jdm commented Sep 21, 2011

No description provided.

@marijnh
Copy link
Contributor

marijnh commented Sep 21, 2011

Code looks great. Merged.

@marijnh marijnh closed this Sep 21, 2011
@marijnh
Copy link
Contributor

marijnh commented Sep 21, 2011

Okay, I had to back this out. It breaks in a weird way on windows, and I can't get Rust to build at all on my windows VM today, so I can't debug it.

@brson brson reopened this Sep 21, 2011
@marijnh
Copy link
Contributor

marijnh commented Sep 23, 2011

This now build fine on my own windows machine, but still breaks on the windows tinderboxes. Possibly an LLVM update would help.

@brson
Copy link
Contributor

brson commented Sep 24, 2011

I spent some time on this today and couldn't get the test to succeed on windows under check-fast though, naturally, it worked when run on it's own.

@marijnh
Copy link
Contributor

marijnh commented Sep 25, 2011

Did you try with llvm HEAD?

@brson
Copy link
Contributor

brson commented Sep 25, 2011

Yeah.

@marijnh
Copy link
Contributor

marijnh commented Sep 28, 2011

So how about integrating this and, for the time, xfailing the test for check-fast? It seems a shame to let it bit-rot when it works in all sane situations.

@brson
Copy link
Contributor

brson commented Sep 28, 2011

Integrated. Thanks!

@brson brson closed this Sep 28, 2011
bors pushed a commit to rust-lang-ci/rust that referenced this pull request Oct 26, 2020
ZuseZ4 pushed a commit to EnzymeAD/rust that referenced this pull request Mar 7, 2023
* Fix LCSSA lookup scope bug

* temp

* tmp

* Fixup

* update test
celinval pushed a commit to celinval/rust-dev that referenced this pull request Jun 4, 2024
tgross35 pushed a commit to tgross35/rust that referenced this pull request Jun 14, 2025
Replace `build.rs` Rust generation with macros, using the unstable
`${concat(...)}`.

Fixes: rust-lang/compiler-builtins#947
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.

3 participants