rounding

Author: sayantn

crates/intrinsic-test/missing_x86_gcc.txt

@@ -21,9 +21,9 @@ _mm512_mask_reduce_min_pd
 _mm512_mask_reduce_min_ps
 
 # Rounding errors in release mode
-_mm_maskz_fmadd_sd
-_mm_maskz_fmadd_ss
-_mm_maskz_fmsub_sd
-_mm_maskz_fmsub_ss
-_mm_maskz_fnmadd_sd
-_mm_maskz_fnmadd_ss
+#_mm_maskz_fmadd_sd
+#_mm_maskz_fmadd_ss
+#_mm_maskz_fmsub_sd
+#_mm_maskz_fmsub_ss
+#_mm_maskz_fnmadd_sd
+#_mm_maskz_fnmadd_ss

crates/intrinsic-test/missing_x86_icx.txt

@@ -12,16 +12,16 @@ _mm512_mask_reduce_min_pd
 _mm512_mask_reduce_min_ps
 
 # Rounding errors in release mode
-_mm_maskz_fmadd_round_sd
-_mm_maskz_fmadd_round_ss
-_mm_maskz_fmadd_round_sh
-_mm_maskz_fmsub_round_sd
-_mm_maskz_fmsub_round_ss
-_mm_maskz_fmsub_round_sh
-_mm_maskz_fnmadd_round_sd
-_mm_maskz_fnmadd_round_ss
-_mm_maskz_fnmadd_round_sh
-_mm_maskz_fnmsub_round_sd
-_mm_maskz_fnmsub_round_ss
-_mm_maskz_fnmsub_sd
-_mm_maskz_fnmsub_ss
+#_mm_maskz_fmadd_round_sd
+#_mm_maskz_fmadd_round_ss
+#_mm_maskz_fmadd_round_sh
+#_mm_maskz_fmsub_round_sd
+#_mm_maskz_fmsub_round_ss
+#_mm_maskz_fmsub_round_sh
+#_mm_maskz_fnmadd_round_sd
+#_mm_maskz_fnmadd_round_ss
+#_mm_maskz_fnmadd_round_sh
+#_mm_maskz_fnmsub_round_sd
+#_mm_maskz_fnmsub_round_ss
+#_mm_maskz_fnmsub_sd
+#_mm_maskz_fnmsub_ss

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

@@ -4,6 +4,7 @@ use itertools::Itertools;
 
 use super::intrinsic_helpers::IntrinsicTypeDefinition;
 use crate::common::argument::ArgumentList;
+use crate::common::cli::ProcessedCli;
 use crate::common::intrinsic::Intrinsic;
 use crate::common::intrinsic_helpers::TypeKind;
 
@@ -314,11 +315,17 @@ pub fn write_build_rs(
     w: &mut impl std::io::Write,
     i: usize,
     arch_flags: &[&str],
+    cli_options: &ProcessedCli,
 ) -> std::io::Result<()> {
-    const COMMON_FLAGS: &[&str] = &[
-        "-ffp-contract=off",
+    const COMMON_FLAGS: &[&str] = &["-ffp-contract=off", "-Wno-narrowing"];
+    const CLANG_FLAGS: &[&str] = &["-ffp-model=strict"];
+    const GCC_FLAGS: &[&str] = &[
         "-flax-vector-conversions",
-        "-Wno-narrowing",
+        "-fno-fast-math",
+        "-frounding-math",
+        "-fexcess-precision=standard",
+        "-ftrapping-math",
+        "-fsignaling-nans",
     ];
 
     write!(
@@ -333,7 +340,17 @@ pub fn write_build_rs(
         i = i
     )?;
 
-    for flag in COMMON_FLAGS.iter().chain(arch_flags) {
+    let compiler_specific_flags = match &*cli_options.cc {
+        "gcc" => GCC_FLAGS,
+        "clang" | "icx" => CLANG_FLAGS,
+        cc => unimplemented!("Unknown C compiler {cc}"),
+    };
+
+    for flag in COMMON_FLAGS
+        .iter()
+        .chain(compiler_specific_flags)
+        .chain(arch_flags)
+    {
         writeln!(w, "\"{flag}\",")?;
     }
 

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

@@ -97,7 +97,7 @@ pub trait SupportedArchitectureTest {
                 trace!("generating `{build_rs_filename}`");
                 let mut file = File::create(&build_rs_filename).unwrap();
 
-                write_build_rs(&mut file, i, &arch_flags).unwrap();
+                write_build_rs(&mut file, i, &arch_flags, &cli_options).unwrap();
                 run_rustfmt(&build_rs_filename);
 
                 Ok(())