(mostly) blindly enable more tests on Windows

Author: saethlin

tests/codegen/debug-column.rs

@@ -1,6 +1,6 @@
 // Verify that debuginfo column numbers are 1-based byte offsets.
 //
-//@ ignore-windows
+//@ ignore-msvc
 //@ compile-flags: -C debuginfo=2
 
 #[rustfmt::skip]

tests/codegen/force-no-unwind-tables.rs

@@ -1,5 +1,5 @@
 //@ compile-flags: -C no-prepopulate-passes -C panic=abort -C force-unwind-tables=n
-//@ ignore-windows
+//@ ignore-windows: unwind tables are required for panics on Windows
 
 #![crate_type = "lib"]
 

tests/codegen/gdb_debug_script_load.rs

@@ -1,4 +1,3 @@
-//
 //@ ignore-windows
 //@ ignore-apple
 //@ ignore-wasm

tests/codegen/remap_path_prefix/issue-73167-remap-std.rs

@@ -1,5 +1,3 @@
-//@ ignore-windows
-
 //@ compile-flags: -g  -C no-prepopulate-passes -Z simulate-remapped-rust-src-base=/rustc/xyz
 
 // Here we check that importing std will not cause real path to std source files

tests/codegen/remap_path_prefix/main.rs

@@ -1,6 +1,3 @@
-//@ ignore-windows
-//
-
 //@ compile-flags: -g  -C no-prepopulate-passes --remap-path-prefix={{cwd}}=/the/cwd --remap-path-prefix={{src-base}}=/the/src -Zinline-mir=no
 //@ aux-build:remap_path_prefix_aux.rs
 

tests/codegen/remap_path_prefix/xcrate-generic.rs

@@ -1,4 +1,4 @@
-//@ ignore-windows
+//@ ignore-windows: FIXME this can be enabled on Windows with enough revisions for the target
 //@ compile-flags: -g -C metadata=foo -C no-prepopulate-passes
 //@ aux-build:xcrate-generic.rs
 

tests/codegen/thread-local.rs

@@ -1,6 +1,5 @@
 //@ compile-flags: -O
 //@ aux-build:thread_local_aux.rs
-//@ ignore-windows FIXME(#84933)
 //@ ignore-wasm globals are used instead of thread locals
 //@ ignore-emscripten globals are used instead of thread locals
 //@ ignore-android does not use #[thread_local]

tests/crashes/132981.rs

@@ -2,6 +2,10 @@
 //@compile-flags: -Clink-dead-code=true --crate-type lib
 //@ only-x86_64
 //@ ignore-windows
+// The set of targets this crashes on is really fiddly, because it is deep in our ABI logic. It
+// crashes on x86_64-unknown-linux-gnu, and i686-pc-windows-msvc, but not on
+// x86_64-pc-windows-msvc. If you are trying to fix this crash, don't pay too much attention to the
+// directives.
 
 #![feature(rust_cold_cc)]
 pub extern "rust-cold" fn foo(_: [usize; 3]) {}

tests/incremental/issue-54059.rs

@@ -1,5 +1,4 @@
 //@ proc-macro: issue-54059.rs
-//@ ignore-windows - dealing with weird symbols issues on dylibs isn't worth it
 //@ revisions: rpass1
 
 extern crate issue_54059;

tests/run-make/naked-symbol-visibility/rmake.rs

@@ -1,4 +1,3 @@
-//@ ignore-windows
 //@ only-x86_64
 use run_make_support::object::ObjectSymbol;
 use run_make_support::object::read::{File, Object, Symbol};