Skip to content

Commit

Permalink
Fix tests that rely on LLVM IR verification
Browse files Browse the repository at this point in the history
Pass -Z verify-llvm-ir to tests that rely on it, to make sure they
pass regardless of the value of verify-llvm-ir in config.toml.

Also remove the 109681.rs test, because it is a duplicat of
common-linkage-non-zero-init.rs.
  • Loading branch information
nikic authored and Nikita Popov committed Nov 29, 2024
1 parent d3ad000 commit fc9f727
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 10 deletions.
9 changes: 0 additions & 9 deletions tests/crashes/109681.rs

This file was deleted.

2 changes: 1 addition & 1 deletion tests/crashes/34127.rs
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
//@ compile-flags: -g -Copt-level=0
//@ compile-flags: -g -Copt-level=0 -Z verify-llvm-ir
//@ known-bug: #34127
//@ only-x86_64

Expand Down
1 change: 1 addition & 0 deletions tests/ui/linkage-attr/common-linkage-non-zero-init.rs
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@
//@ failure-status: 101
//@ known-bug: #109681
//@ ignore-wasm32 this appears to SIGABRT on wasm, not fail cleanly
//@ compile-flags: -Z verify-llvm-ir

// This test verifies that we continue to hit the LLVM error for common linkage with non-zero
// initializers, since it generates invalid LLVM IR.
Expand Down

0 comments on commit fc9f727

Please sign in to comment.