Skip to content

feature: remove annotation requirement for registered lambda launch - #555

Merged
johnbowen42 merged 2 commits into
mainfrom
feature/bowen/remove-lambda-annotation-dependency
Oct 30, 2026
Merged

johnbowen42 merged 2 commits into
mainfrom
feature/bowen/remove-lambda-annotation-dependency

Conversation

@johnbowen42

Copy link
Copy Markdown
Collaborator

No description provided.

@johnbowen42
johnbowen42 force-pushed the feature/bowen/remove-lambda-annotation-dependency branch from bceda6a to b14d959 Compare October 23, 2026 18:41
@johnbowen42
johnbowen42 requested a review from ZwFink October 23, 2026 19:01
@johnbowen42 johnbowen42 self-assigned this Oct 23, 2026

@davidbeckingsale davidbeckingsale left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does the new analysis add significant compilation overhead?

@johnbowen42

johnbowen42 commented Oct 28, 2026 •

Copy link
Copy Markdown
Collaborator Author

Does the new analysis add significant compilation overhead?

In this case the only new analysis introduced is a walk from lambda callbase back to a GPU kernel. This doesn't add significant overhead. In general it will O(call stack size of lambda), which for something like RAJA is relatively short

@johnbowen42

Copy link
Copy Markdown
Collaborator Author

Does the new analysis add significant compilation overhead?

It's also worth mentioning that this PR essentially makes the register_lambda/jit_variable combination a fully-independent frontend for Proteus, removing the requirement of using annotations completely

@johnbowen42
johnbowen42 merged commit aa01b91 into main Oct 30, 2026
64 checks passed
@johnbowen42
johnbowen42 deleted the feature/bowen/remove-lambda-annotation-dependency branch October 30, 2026 19:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants