Add FIXMEs for those ignored tests.

Author: ywxt

compiler/rustc_thread_pool/src/broadcast/tests.rs

@@ -64,6 +64,7 @@ fn spawn_broadcast_self() {
     assert!(v.into_iter().eq(0..7));
 }
 
+// FIXME: We should fix or remove this ignored test.
 #[test]
 #[ignore]
 #[cfg_attr(any(target_os = "emscripten", target_family = "wasm"), ignore)]
@@ -99,6 +100,7 @@ fn spawn_broadcast_mutual() {
     assert_eq!(rx.into_iter().count(), 3 * 7);
 }
 
+// FIXME: We should fix or remove this ignored test.
 #[test]
 #[ignore]
 #[cfg_attr(any(target_os = "emscripten", target_family = "wasm"), ignore)]

compiler/rustc_thread_pool/src/join/tests.rs

@@ -96,6 +96,7 @@ fn join_context_both() {
     assert!(b_migrated);
 }
 
+// FIXME: We should fix or remove this ignored test.
 #[test]
 #[ignore]
 #[cfg_attr(any(target_os = "emscripten", target_family = "wasm"), ignore)]

compiler/rustc_thread_pool/src/scope/tests.rs

@@ -289,6 +289,7 @@ macro_rules! test_order {
     }};
 }
 
+// FIXME: We should fix or remove this ignored test.
 #[test]
 #[ignore]
 #[cfg_attr(any(target_os = "emscripten", target_family = "wasm"), ignore)]
@@ -299,6 +300,7 @@ fn lifo_order() {
     assert_eq!(vec, expected);
 }
 
+// FIXME: We should fix or remove this ignored test.
 #[test]
 #[ignore]
 #[cfg_attr(any(target_os = "emscripten", target_family = "wasm"), ignore)]
@@ -335,6 +337,7 @@ macro_rules! test_nested_order {
     }};
 }
 
+// FIXME: We should fix or remove this ignored test.
 #[test]
 #[ignore]
 #[cfg_attr(any(target_os = "emscripten", target_family = "wasm"), ignore)]
@@ -345,6 +348,7 @@ fn nested_lifo_order() {
     assert_eq!(vec, expected);
 }
 
+// FIXME: We should fix or remove this ignored test.
 #[test]
 #[ignore]
 #[cfg_attr(any(target_os = "emscripten", target_family = "wasm"), ignore)]
@@ -355,6 +359,7 @@ fn nested_fifo_order() {
     assert_eq!(vec, expected);
 }
 
+// FIXME: We should fix or remove this ignored test.
 #[test]
 #[ignore]
 #[cfg_attr(any(target_os = "emscripten", target_family = "wasm"), ignore)]
@@ -365,6 +370,7 @@ fn nested_lifo_fifo_order() {
     assert_eq!(vec, expected);
 }
 
+// FIXME: We should fix or remove this ignored test.
 #[test]
 #[ignore]
 #[cfg_attr(any(target_os = "emscripten", target_family = "wasm"), ignore)]
@@ -407,6 +413,7 @@ macro_rules! test_mixed_order {
     }};
 }
 
+// FIXME: We should fix or remove this ignored test.
 #[test]
 #[ignore]
 #[cfg_attr(any(target_os = "emscripten", target_family = "wasm"), ignore)]
@@ -418,6 +425,7 @@ fn mixed_lifo_order() {
     assert_eq!(vec, expected);
 }
 
+// FIXME: We should fix or remove this ignored test.
 #[test]
 #[ignore]
 #[cfg_attr(any(target_os = "emscripten", target_family = "wasm"), ignore)]
@@ -427,6 +435,7 @@ fn mixed_fifo_order() {
     assert_eq!(vec, expected);
 }
 
+// FIXME: We should fix or remove this ignored test.
 #[test]
 #[ignore]
 #[cfg_attr(any(target_os = "emscripten", target_family = "wasm"), ignore)]
@@ -438,6 +447,7 @@ fn mixed_lifo_fifo_order() {
     assert_eq!(vec, expected);
 }
 
+// FIXME: We should fix or remove this ignored test.
 #[test]
 #[ignore]
 #[cfg_attr(any(target_os = "emscripten", target_family = "wasm"), ignore)]
@@ -553,6 +563,7 @@ fn scope_fifo_spawn_broadcast() {
     assert_eq!(sum.into_inner(), n * (n - 1) / 2);
 }
 
+// FIXME: We should fix or remove this ignored test.
 #[test]
 #[ignore]
 fn scope_spawn_broadcast_nested() {

compiler/rustc_thread_pool/src/spawn/tests.rs

@@ -166,6 +166,7 @@ macro_rules! test_order {
     }};
 }
 
+// FIXME: We should fix or remove this ignored test.
 #[test]
 #[ignore]
 #[cfg_attr(any(target_os = "emscripten", target_family = "wasm"), ignore)]
@@ -176,6 +177,7 @@ fn lifo_order() {
     assert_eq!(vec, expected);
 }
 
+// FIXME: We should fix or remove this ignored test.
 #[test]
 #[ignore]
 #[cfg_attr(any(target_os = "emscripten", target_family = "wasm"), ignore)]
@@ -186,6 +188,7 @@ fn fifo_order() {
     assert_eq!(vec, expected);
 }
 
+// FIXME: We should fix or remove this ignored test.
 #[test]
 #[ignore]
 #[cfg_attr(any(target_os = "emscripten", target_family = "wasm"), ignore)]
@@ -196,6 +199,7 @@ fn lifo_fifo_order() {
     assert_eq!(vec, expected);
 }
 
+// FIXME: We should fix or remove this ignored test.
 #[test]
 #[ignore]
 #[cfg_attr(any(target_os = "emscripten", target_family = "wasm"), ignore)]
@@ -233,6 +237,7 @@ macro_rules! test_mixed_order {
     }};
 }
 
+// FIXME: We should fix or remove this ignored test.
 #[test]
 #[ignore]
 #[cfg_attr(any(target_os = "emscripten", target_family = "wasm"), ignore)]
@@ -242,6 +247,7 @@ fn mixed_lifo_fifo_order() {
     assert_eq!(vec, expected);
 }
 
+// FIXME: We should fix or remove this ignored test.
 #[test]
 #[ignore]
 #[cfg_attr(any(target_os = "emscripten", target_family = "wasm"), ignore)]

compiler/rustc_thread_pool/src/thread_pool/tests.rs

@@ -151,6 +151,7 @@ fn self_install() {
     assert!(pool.install(|| pool.install(|| true)));
 }
 
+// FIXME: We should fix or remove this ignored test.
 #[test]
 #[ignore]
 #[cfg_attr(any(target_os = "emscripten", target_family = "wasm"), ignore)]
@@ -172,6 +173,7 @@ fn mutual_install() {
     assert!(ok);
 }
 
+// FIXME: We should fix or remove this ignored test.
 #[test]
 #[ignore]
 #[cfg_attr(any(target_os = "emscripten", target_family = "wasm"), ignore)]
@@ -228,6 +230,7 @@ macro_rules! test_scope_order {
     }};
 }
 
+// FIXME: We should fix or remove this ignored test.
 #[test]
 #[ignore]
 #[cfg_attr(any(target_os = "emscripten", target_family = "wasm"), ignore)]
@@ -237,6 +240,7 @@ fn scope_lifo_order() {
     assert_eq!(vec, expected);
 }
 
+// FIXME: We should fix or remove this ignored test.
 #[test]
 #[ignore]
 #[cfg_attr(any(target_os = "emscripten", target_family = "wasm"), ignore)]
@@ -279,6 +283,7 @@ fn spawn_fifo_order() {
     assert_eq!(vec, expected);
 }
 
+// FIXME: We should fix or remove this ignored test.
 #[test]
 #[ignore]
 #[cfg_attr(any(target_os = "emscripten", target_family = "wasm"), ignore)]
@@ -316,6 +321,7 @@ fn nested_scopes() {
     assert_eq!(counter.into_inner(), pools.len());
 }
 
+// FIXME: We should fix or remove this ignored test.
 #[test]
 #[ignore]
 #[cfg_attr(any(target_os = "emscripten", target_family = "wasm"), ignore)]

compiler/rustc_thread_pool/tests/stack_overflow_crash.rs

@@ -35,6 +35,7 @@ fn overflow_code() -> Option<i32> {
     ExitStatus::from_raw(0xc00000fd /*STATUS_STACK_OVERFLOW*/).code()
 }
 
+// FIXME: We should fix or remove this test on Windows.
 #[test]
 #[cfg_attr(not(any(unix)), ignore)]
 fn stack_overflow_crash() {