Skip to content

Commit 0e37477

Browse files
fix: vec_len -> simd_len (an error was present due to setting vec_len instead of simd_len for AVX register types)
1 parent e903362 commit 0e37477

File tree

3 files changed

+8
-7
lines changed

3 files changed

+8
-7
lines changed

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

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@ typedef float float32_t;
1515
typedef double float64_t;
1616
1717
#define __int64 long long
18+
#define __int32 int
1819
1920
#define mm512_extract_intrinsic_test_epi8(m, lane) \
2021
_mm_extract_epi8(_mm512_extracti64x2_epi64((m), (lane) / 16), (lane) % 16)

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

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -209,9 +209,9 @@ impl IntrinsicTypeDefinition for X86IntrinsicType {
209209
/// Determines the get lane function for this type.
210210
fn get_lane_function(&self) -> String {
211211
let total_vector_bits: Option<u32> = self
212-
.vec_len
212+
.simd_len
213213
.zip(self.bit_len)
214-
.and_then(|(vec_len, bit_len)| Some(vec_len * bit_len));
214+
.and_then(|(simd_len, bit_len)| Some(simd_len * bit_len));
215215

216216
match (self.bit_len, total_vector_bits) {
217217
(Some(8), Some(128)) => String::from("_mm_extract_epi8"),
@@ -231,7 +231,7 @@ impl IntrinsicTypeDefinition for X86IntrinsicType {
231231
(Some(32), Some(64)) => String::from("mm64_extract_intrinsic_test_epi32"),
232232
_ => unreachable!(
233233
"invalid length for vector argument: {:?}, {:?}",
234-
self.bit_len, self.vec_len
234+
self.bit_len, self.simd_len
235235
),
236236
}
237237
}
@@ -345,9 +345,9 @@ impl X86IntrinsicType {
345345
if param.type_data.matches("__m").next().is_some()
346346
&& param.type_data.matches("__mmask").next().is_none()
347347
{
348-
data.vec_len = match str::parse::<u32>(type_processed.as_str()) {
349-
// If bit_len is None, vec_len will be None.
350-
// Else vec_len will be (num_bits / bit_len).
348+
data.simd_len = match str::parse::<u32>(type_processed.as_str()) {
349+
// If bit_len is None, simd_len will be None.
350+
// Else simd_len will be (num_bits / bit_len).
351351
Ok(num_bits) => data.bit_len.and_then(|bit_len| Some(num_bits / bit_len)),
352352
Err(_) => None,
353353
};

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -106,7 +106,7 @@ fn xml_to_intrinsic(
106106
let mut args_test = args.iter();
107107

108108
// if one of the args has etype="MASK" and type="__m<int>d",
109-
// then set the bit_len and vec_len accordingly
109+
// then set the bit_len and simd_len accordingly
110110
let re = Regex::new(r"__m\d+").unwrap();
111111
let is_mask = |arg: &Argument<X86IntrinsicType>| arg.ty.param.etype.as_str() == "MASK";
112112
let is_vector = |arg: &Argument<X86IntrinsicType>| re.is_match(arg.ty.param.type_data.as_str());

0 commit comments

Comments
 (0)