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

[core] Refactor the lift array alloc pass into two proper phases. #2287

Merged
merged 2 commits into from
Oct 21, 2024

Conversation

schweitzpgi
Copy link
Collaborator

@schweitzpgi schweitzpgi commented Oct 16, 2024

Start changing core code over to use SmallVector (and be more compatible with MLIR) instead of std::vector. Refactoring lifting of array values. Implement globalize array values. Update new pass and pass names across all the config and yaml files.

Requires merge of #2277

Description

@schweitzpgi schweitzpgi changed the title Start changing core code over to use SmallVector (and be more [core] Refactor the lift array alloc pass into two proper phases. Oct 16, 2024
Copy link
Collaborator

@annagrin annagrin left a comment

Choose a reason for hiding this comment

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

LGTM!

lib/Optimizer/Transforms/ConstPropComplex.cpp Outdated Show resolved Hide resolved
@schweitzpgi schweitzpgi force-pushed the ch-levis.tabela.valoroj branch 3 times, most recently from 2b35a7c to f82c8c5 Compare October 18, 2024 18:30
compatible with MLIR) instead of std::vector.  Refactoring
lifting of array values.  Implement globalize array values.
Update new pass and pass names across all the config and
yaml files.

Signed-off-by: Eric Schweitz <[email protected]>
@schweitzpgi schweitzpgi enabled auto-merge (squash) October 21, 2024 15:38
@schweitzpgi schweitzpgi merged commit 36975fc into NVIDIA:main Oct 21, 2024
132 checks passed
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.

2 participants