Skip to content

Commit 11375b9

Browse files
chore: move from random testing to testing only the first N intrinsics
1 parent 7b80a1f commit 11375b9

File tree

3 files changed

+9
-79
lines changed

3 files changed

+9
-79
lines changed

Cargo.lock

Lines changed: 7 additions & 70 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

crates/intrinsic-test/Cargo.toml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,4 +22,3 @@ itertools = "0.14.0"
2222
quick-xml = { version = "0.37.5", features = ["serialize", "overlapped-lists"] }
2323
serde-xml-rs = "0.8.0"
2424
regex = "1.11.1"
25-
rand = "0.9.2"

crates/intrinsic-test/src/x86/mod.rs

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -12,8 +12,6 @@ use crate::common::intrinsic::Intrinsic;
1212
use crate::common::intrinsic_helpers::TypeKind;
1313
use intrinsic::X86IntrinsicType;
1414
use itertools::Itertools;
15-
use rand::rng;
16-
use rand::seq::IndexedRandom;
1715
use xml_parser::get_xml_intrinsics;
1816

1917
pub struct X86ArchitectureTest {
@@ -49,10 +47,9 @@ impl SupportedArchitectureTest for X86ArchitectureTest {
4947
let intrinsics =
5048
get_xml_intrinsics(&cli_options.filename).expect("Error parsing input file");
5149

52-
let mut rng = rng();
5350
let sample_percentage: usize = cli_options.sample_percentage as usize;
5451

55-
let intrinsics = intrinsics
52+
let mut intrinsics = intrinsics
5653
.into_iter()
5754
// Not sure how we would compare intrinsic that returns void.
5855
.filter(|i| i.results.kind() != TypeKind::Void)
@@ -68,10 +65,7 @@ impl SupportedArchitectureTest for X86ArchitectureTest {
6865
.collect::<Vec<_>>();
6966

7067
let sample_size = (intrinsics.len() * sample_percentage) / 100;
71-
let mut intrinsics = intrinsics
72-
.choose_multiple(&mut rng, sample_size)
73-
.cloned()
74-
.collect::<Vec<_>>();
68+
intrinsics.truncate(sample_size);
7569

7670
intrinsics.sort_by(|a, b| a.name.cmp(&b.name));
7771
Self {

0 commit comments

Comments
 (0)