Skip to content

Conversation

@crabtw
Copy link
Contributor

@crabtw crabtw commented May 9, 2013

I changed RED_ZONE_SIZE to RZ_MAC_32 because of stack canary failure.
Here is a LLVM patch for MIPS segmented stacks.
http://people.cs.nctu.edu.tw/~jyyou/rust/mips-segstk.patch

Current test results

failures:
    rand::tests::test_rng_seeded_custom_seed2
    run::tests::test_forced_destroy_actually_kills
    run::tests::test_unforced_destroy_actually_kills
    time::tests::run_tests
    uv_ll::test::test_uv_ll_struct_size_addrinfo
    uv_ll::test::test_uv_ll_struct_size_uv_timer_t

segfaults:
    rt::io::option::test::test_option_writer_error
    rt::local_services::test::unwind
    rt::sched::test_swap_tasks_then
    stackwalk::test_simple
    stackwalk::test_simple_deep

@brson brson mentioned this pull request May 9, 2013
@brson
Copy link
Contributor

brson commented May 9, 2013

I opened #6367 to batch up pending LLVM patch.s

@brson
Copy link
Contributor

brson commented May 9, 2013

Thanks @crabtw 🐰

bors added a commit that referenced this pull request May 10, 2013
I changed ```RED_ZONE_SIZE``` to ```RZ_MAC_32``` because of stack canary failure.
Here is a LLVM patch for MIPS segmented stacks.
http://people.cs.nctu.edu.tw/~jyyou/rust/mips-segstk.patch

Current test results
```
failures:
    rand::tests::test_rng_seeded_custom_seed2
    run::tests::test_forced_destroy_actually_kills
    run::tests::test_unforced_destroy_actually_kills
    time::tests::run_tests
    uv_ll::test::test_uv_ll_struct_size_addrinfo
    uv_ll::test::test_uv_ll_struct_size_uv_timer_t

segfaults:
    rt::io::option::test::test_option_writer_error
    rt::local_services::test::unwind
    rt::sched::test_swap_tasks_then
    stackwalk::test_simple
    stackwalk::test_simple_deep
```
@bors bors closed this May 10, 2013
flip1995 pushed a commit to flip1995/rust that referenced this pull request Jan 30, 2021
…in_deref, r=llogiq

Fix a wrong suggestion of `ref_in_deref`

Fix rust-lang#6358.

changelog: Fix a wrong suggestion of `ref_in_deref`
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