feat: updating the number of x86 intrinsics tested to 50%

Author: madhav-madhusoodanan

ci/intrinsic-test.sh

@@ -75,7 +75,7 @@ case ${TARGET} in
         TEST_CXX_COMPILER="clang++"
         TEST_RUNNER="${CARGO_TARGET_X86_64_UNKNOWN_LINUX_GNU_RUNNER}"
         TEST_SKIP_INTRINSICS=crates/intrinsic-test/missing_x86.txt
-        : "${TEST_SAMPLE_INTRINSICS_PERCENTAGE:=20}"
+        : "${TEST_SAMPLE_INTRINSICS_PERCENTAGE:=50}"
         ;;
     *)
         ;;
@@ -85,7 +85,7 @@ esac
 # Arm specific
 case "${TARGET}" in
     aarch64-unknown-linux-gnu*|armv7-unknown-linux-gnueabihf*)
-        CPPFLAGS="${TEST_CPPFLAGS}" RUSTFLAGS="${HOST_RUSTFLAGS}" RUST_LOG=info \
+        CPPFLAGS="${TEST_CPPFLAGS}" RUSTFLAGS="${HOST_RUSTFLAGS}" RUST_LOG=trace \
             cargo run "${INTRINSIC_TEST}" "${PROFILE}" \
             --bin intrinsic-test -- intrinsics_data/arm_intrinsics.json \
             --runner "${TEST_RUNNER}" \
@@ -96,7 +96,7 @@ case "${TARGET}" in
         ;;
 
     aarch64_be-unknown-linux-gnu*)
-        CPPFLAGS="${TEST_CPPFLAGS}" RUSTFLAGS="${HOST_RUSTFLAGS}" RUST_LOG=info \
+        CPPFLAGS="${TEST_CPPFLAGS}" RUSTFLAGS="${HOST_RUSTFLAGS}" RUST_LOG=trace \
             cargo run "${INTRINSIC_TEST}" "${PROFILE}"  \
             --bin intrinsic-test -- intrinsics_data/arm_intrinsics.json \
             --runner "${TEST_RUNNER}" \
@@ -114,7 +114,7 @@ case "${TARGET}" in
         # Hence the use of `env -u`.
         env -u CARGO_TARGET_X86_64_UNKNOWN_LINUX_GNU_RUNNER \
             CPPFLAGS="${TEST_CPPFLAGS}" RUSTFLAGS="${HOST_RUSTFLAGS}" \
-            RUST_LOG=info RUST_BACKTRACE=1 \
+            RUST_LOG=trace RUST_BACKTRACE=1 \
             cargo run "${INTRINSIC_TEST}" "${PROFILE}"  \
             --bin intrinsic-test -- intrinsics_data/x86-intel.xml \
             --runner "${TEST_RUNNER}" \

crates/intrinsic-test/src/common/compare.rs

@@ -17,7 +17,7 @@ pub fn compare_outputs(intrinsic_name_list: &Vec<String>, runner: &str, target:
         .into_par_iter()
         .map(|i| {
             runner_command(runner)
-                .arg(format!("./intrinsic-test-programs_{i}"))
+                .arg(format!("./intrinsic-test-programs-{i}"))
                 .current_dir("c_programs")
                 .output()
         })

crates/intrinsic-test/src/common/gen_rust.rs

@@ -51,8 +51,8 @@ pub fn write_bin_cargo_toml(
     }
 
     for i in 0..binary_count {
-        writeln!(w, "[[bin]]")?;
-        writeln!(w, "name = \"intrinsic-test-programs_{i}\"")?;
+        writeln!(w, "\n[[bin]]")?;
+        writeln!(w, "name = \"intrinsic-test-programs-{i}\"")?;
         writeln!(w, "path = \"src/main_{i}.rs\"")?;
     }
 

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

@@ -98,15 +98,6 @@ where
             .collect::<Result<(), String>>()
             .unwrap();
 
-        let mut file = File::create("c_programs/main.cpp").unwrap();
-        write_main_cpp(
-            &mut file,
-            Self::PLATFORM_C_DEFINITIONS,
-            Self::PLATFORM_C_HEADERS,
-            self.intrinsics().iter().map(|i| i.name.as_str()),
-        )
-        .unwrap();
-
         let available_parallelism = std::thread::available_parallelism().unwrap().get();
         self.intrinsics()
             .par_chunks(available_parallelism)
@@ -153,8 +144,10 @@ where
                     return output;
                 })
                 .inspect(|output| {
+                    trace!("{output:?}");
                     assert!(output.is_ok(), "{output:?}");
                     if let Ok(out) = &output {
+                        trace!("{:?}", out.status.success());
                         assert!(out.status.success(), "{output:?}")
                     }
                 })