aboutsummaryrefslogtreecommitdiff
path: root/rtic-macros/src/syntax/check.rs
diff options
context:
space:
mode:
authorHenrik Tjäder <henrik@tjaders.com>2023-02-04 16:47:17 +0100
committerHenrik Tjäder <henrik@tjaders.com>2023-03-01 00:35:13 +0100
commit9e445b3583c15c7701f3167eaa8dfe4afd541691 (patch)
tree167565d51598f42c0454d60b34e1170589ae1056 /rtic-macros/src/syntax/check.rs
parent4124fbdd61ff823c6217a2a16ebb4d813146116c (diff)
Move rtic macros to repo root, tune xtask
Diffstat (limited to 'rtic-macros/src/syntax/check.rs')
-rw-r--r--rtic-macros/src/syntax/check.rs66
1 files changed, 66 insertions, 0 deletions
diff --git a/rtic-macros/src/syntax/check.rs b/rtic-macros/src/syntax/check.rs
new file mode 100644
index 0000000..989d418
--- /dev/null
+++ b/rtic-macros/src/syntax/check.rs
@@ -0,0 +1,66 @@
+use std::collections::HashSet;
+
+use syn::parse;
+
+use crate::syntax::ast::App;
+
+pub fn app(app: &App) -> parse::Result<()> {
+ // Check that all referenced resources have been declared
+ // Check that resources are NOT `Exclusive`-ly shared
+ let mut owners = HashSet::new();
+ for (_, name, access) in app.shared_resource_accesses() {
+ if app.shared_resources.get(name).is_none() {
+ return Err(parse::Error::new(
+ name.span(),
+ "this shared resource has NOT been declared",
+ ));
+ }
+
+ if access.is_exclusive() {
+ owners.insert(name);
+ }
+ }
+
+ for name in app.local_resource_accesses() {
+ if app.local_resources.get(name).is_none() {
+ return Err(parse::Error::new(
+ name.span(),
+ "this local resource has NOT been declared",
+ ));
+ }
+ }
+
+ // Check that no resource has both types of access (`Exclusive` & `Shared`)
+ let exclusive_accesses = app
+ .shared_resource_accesses()
+ .filter_map(|(priority, name, access)| {
+ if priority.is_some() && access.is_exclusive() {
+ Some(name)
+ } else {
+ None
+ }
+ })
+ .collect::<HashSet<_>>();
+ for (_, name, access) in app.shared_resource_accesses() {
+ if access.is_shared() && exclusive_accesses.contains(name) {
+ return Err(parse::Error::new(
+ name.span(),
+ "this implementation doesn't support shared (`&-`) - exclusive (`&mut-`) locks; use `x` instead of `&x`",
+ ));
+ }
+ }
+
+ // check that dispatchers are not used as hardware tasks
+ for task in app.hardware_tasks.values() {
+ let binds = &task.args.binds;
+
+ if app.args.dispatchers.contains_key(binds) {
+ return Err(parse::Error::new(
+ binds.span(),
+ "dispatcher interrupts can't be used as hardware tasks",
+ ));
+ }
+ }
+
+ Ok(())
+}