-
Notifications
You must be signed in to change notification settings - Fork 13.1k
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
fix MIPS target #14455
fix MIPS target #14455
Conversation
@@ -270,7 +270,7 @@ fn initialize_call_frame(regs: &mut Registers, fptr: InitFn, arg: uint, | |||
type Registers = [uint, ..32]; | |||
|
|||
#[cfg(target_arch = "mips")] | |||
fn new_regs() -> Box<Registers> { box [0, .. 32] } | |||
fn new_regs() -> Box<Registers> { box {[0, .. 32]} } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is this change needed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(Oh, doesn't matter: I see it's a hold-over from the old days where ~[0, .. 32]
would construct a ~[int]
, not a ~([int, .. 32])
.)
I removed additional functions and used |
Excuse me. If endianness is the problem, why don't you branch code by |
The original PR had functions that were branched by Hope these answer your question. |
OK, I understand. Thank you! |
Because IPv4 address conversion doesn't consider big-endian target, I add functions to handle that. These function names may need to be changed, but I can't come up with a good one.
…osure, r=Veykril Convert nested function to closure assist Continuation of / closes rust-lang#13467. Resolves rust-lang#13230. r? `@Veykril`
Because IPv4 address conversion doesn't consider big-endian target, I add functions to handle that.
These function names may need to be changed, but I can't come up with a good one.