Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for cells with non-local storage #730

Merged
merged 23 commits into from
Oct 7, 2024
Merged

Add support for cells with non-local storage #730

merged 23 commits into from
Oct 7, 2024

Conversation

Jezurko
Copy link
Collaborator

@Jezurko Jezurko commented Sep 23, 2024

  • Move storage attributes from hl to core
  • Add Interface for storage information on declarations
  • Modify cell declarations to keep storage information
  • Nuke GlobalRefs and replace them with DeclRefs
  • Lower non-local cells to llvm
  • Add tests

Copy link
Contributor

github-actions bot commented Oct 4, 2024

Cpp-Linter Report ⚠️

Some files did not pass the configured checks!

clang-format reports: 11 file(s) not formatted
  • include/vast/Dialect/Core/CoreAttributes.hpp
  • include/vast/Dialect/Core/Interfaces/DeclStorageInterface.hpp
  • lib/vast/CodeGen/DefaultDeclVisitor.cpp
  • lib/vast/CodeGen/DefaultStmtVisitor.cpp
  • lib/vast/Conversion/FromHL/Passes.cpp
  • lib/vast/Conversion/ToLLVM/IRsToLLVM.cpp
  • lib/vast/Conversion/ToMem/EvictStaticLocals.cpp
  • lib/vast/Conversion/ToMem/RefsToSSA.cpp
  • lib/vast/Conversion/ToMem/VarsToCells.cpp
  • lib/vast/Dialect/Core/CoreAttributes.cpp
  • lib/vast/Dialect/HighLevel/HighLevelOps.cpp

Have any feedback or feature suggestions? Share it here.

@Jezurko Jezurko marked this pull request as ready for review October 4, 2024 15:41
@Jezurko Jezurko requested a review from xlauko as a code owner October 4, 2024 15:41
@xlauko xlauko merged commit 843fbb6 into master Oct 7, 2024
12 checks passed
@xlauko xlauko deleted the cells-storage branch October 7, 2024 12:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants