Skip to content

Commit 4c7b5df

Browse files
code cleanup
1 parent 420f2ee commit 4c7b5df

File tree

2 files changed

+12
-22
lines changed

2 files changed

+12
-22
lines changed

crates/intrinsic-test/src/arm/functions.rs

Lines changed: 9 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@ use crate::common::intrinsic_types::IntrinsicTypeDefinition;
1111
use crate::common::write_file;
1212
use itertools::Itertools;
1313
use rayon::prelude::*;
14-
use std::collections::BTreeMap;
1514

1615
// The number of times each intrinsic will be called.
1716
const PASSES: u32 = 20;
@@ -162,7 +161,6 @@ fn generate_rust_program_arm(
162161

163162
fn compile_c_arm(
164163
intrinsics_name_list: &Vec<String>,
165-
_filename_mapping: BTreeMap<&String, String>,
166164
compiler: &str,
167165
target: &str,
168166
cxx_toolchain_dir: Option<&str>,
@@ -180,8 +178,8 @@ fn compile_c_arm(
180178
command = command.add_arch_flags(vec!["faminmax", "lut", "sha3"]);
181179
}
182180

183-
command = if target == "aarch64_be-unknown-linux-gnu" {
184-
command
181+
if target == "aarch64_be-unknown-linux-gnu" {
182+
command = command
185183
.set_linker(
186184
cxx_toolchain_dir.unwrap_or("").to_string() + "/bin/aarch64_be-none-linux-gnu-g++",
187185
)
@@ -192,14 +190,12 @@ fn compile_c_arm(
192190
"/aarch64_be-none-linux-gnu/include/c++/14.2.1/aarch64_be-none-linux-gnu",
193191
"/aarch64_be-none-linux-gnu/include/c++/14.2.1/backward",
194192
"/aarch64_be-none-linux-gnu/libc/usr/include",
195-
])
196-
} else {
197-
if compiler.contains("clang") {
198-
command.add_extra_flag(format!("-target {target}").as_str())
199-
} else {
200-
command.add_extra_flag("-flax-vector-conversions")
201-
}
202-
};
193+
]);
194+
}
195+
196+
if !compiler.contains("clang") {
197+
command = command.add_extra_flag("-flax-vector-conversions");
198+
}
203199

204200
let compiler_commands = intrinsics_name_list
205201
.iter()
@@ -237,13 +233,7 @@ pub fn build_c(
237233

238234
match compiler {
239235
None => true,
240-
Some(compiler) => compile_c_arm(
241-
&intrinsics_name_list,
242-
filename_mapping,
243-
compiler,
244-
target,
245-
cxx_toolchain_dir,
246-
),
236+
Some(compiler) => compile_c_arm(&intrinsics_name_list, compiler, target, cxx_toolchain_dir),
247237
}
248238
}
249239

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

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -116,10 +116,10 @@ impl CompilationCommandBuilder {
116116

117117
command = command + " " + self.extra_flags.join(" ").as_str();
118118

119+
if let Some(target) = &self.target {
120+
command = command + " --target=" + target;
121+
}
119122
if let (Some(linker), Some(cxx_toolchain_dir)) = (&self.linker, &self.cxx_toolchain_dir) {
120-
if let Some(target) = &self.target {
121-
command = command + " --target=" + target;
122-
}
123123

124124
let include_args = self
125125
.include_paths

0 commit comments

Comments
 (0)