From 40498a44b064de7ac53fff1a4ccbab6d9962cb6b Mon Sep 17 00:00:00 2001
From: Aaron Councilman <aaronjc4@illinois.edu>
Date: Tue, 18 Feb 2025 09:34:18 -0600
Subject: [PATCH] Require block size be multiple of 8 for cfd
- Without this, we either have to rewrite cfd to not use structs of
arrays in Juno or deal with getting the alignment of those arrays
right in Rust, both are annoying
---
juno_samples/rodinia/cfd/src/main.rs | 2 ++
1 file changed, 2 insertions(+)
diff --git a/juno_samples/rodinia/cfd/src/main.rs b/juno_samples/rodinia/cfd/src/main.rs
index 83db67d8..9a51ddfa 100644
--- a/juno_samples/rodinia/cfd/src/main.rs
+++ b/juno_samples/rodinia/cfd/src/main.rs
@@ -146,6 +146,8 @@ fn cfd_harness(args: CFDInputs) {
pre_euler,
} = args;
+ assert!(block_size % 8 == 0, "Hercules expects all arrays to be 32-byte aligned, cfd uses structs of arrays that are annoying to deal with if the block_size is not a multiple of 8");
+
let FarFieldConditions {
ff_variable,
ff_fc_momentum_x,
--
GitLab