Skip to content

Commit

Permalink
Fix tests using only-i686 to use the correct only-x86 directive
Browse files Browse the repository at this point in the history
We translate `i686` to `x86` which means tests marked as `only-i686`
never ran. Update those tests to use `only-x86`.
  • Loading branch information
wesleywiser committed Nov 4, 2021
1 parent 37f70a0 commit 8c56ef0
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion src/test/ui/extern/extern-methods.rs
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
// run-pass
// only-i686
// only-x86

trait A {
extern "fastcall" fn test1(i: i32);
Expand Down
2 changes: 1 addition & 1 deletion src/test/ui/extern/extern-thiscall.rs
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
// run-pass
// only-i686
// only-x86

#![feature(abi_thiscall)]

Expand Down
2 changes: 1 addition & 1 deletion src/test/ui/extern/extern-vectorcall.rs
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
// run-pass
// revisions: x64 x32
// [x64]only-x86_64
// [x32]only-i686
// [x32]only-x86

#![feature(abi_vectorcall)]

Expand Down
2 changes: 1 addition & 1 deletion src/tools/compiletest/src/header/tests.rs
Original file line number Diff line number Diff line change
Expand Up @@ -168,7 +168,7 @@ fn only_target() {
let mut config = config();
config.target = "x86_64-pc-windows-gnu".to_owned();

assert!(check_ignore(&config, "// only-i686"));
assert!(check_ignore(&config, "// only-x86"));
assert!(check_ignore(&config, "// only-linux"));
assert!(check_ignore(&config, "// only-msvc"));
assert!(check_ignore(&config, "// only-32bit"));
Expand Down

0 comments on commit 8c56ef0

Please sign in to comment.