Skip to content

Implement CallOpInterface for LLZK symbol lookups and fix up analyses accordingly#307

Merged
iangneal merged 14 commits intomainfrom
iangneal/analysis-tests
Feb 17, 2026
Merged

Implement CallOpInterface for LLZK symbol lookups and fix up analyses accordingly#307
iangneal merged 14 commits intomainfrom
iangneal/analysis-tests

Conversation

@iangneal
Copy link
Contributor

  • Implement CallOpInterface methods for function.call to use proper LLZK lookups (a new MLIR 20 feature).
  • Remove LLZK port of DenseAnalysis, as the reason it was ported was because of MLIR 18's CallOpInterface issue.
  • Added a PredecessorAnalysis as a test and example of how to write a DenseAnalysis.

DenseAnalysiRemove port of DenseAnalysiRemove port of DenseAnalysiRemove
port of DenseAnalysiRemove port of DenseAnalysiRemove port of
DenseAnalysi
@iangneal iangneal requested review from a team and raghav198 February 10, 2026 19:28
Copy link
Contributor

@raghav198 raghav198 left a comment

Choose a reason for hiding this comment

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

Just a few questions on a first pass

@tim-hoffman tim-hoffman force-pushed the iangneal/analysis-tests branch from 1fc541d to edfda39 Compare February 16, 2026 21:03
@tim-hoffman tim-hoffman requested a review from a team February 16, 2026 21:20
- Remove default join for IntervalAnalysis which caused invalid
  intervals when proper region traversal occurs
- Resolves #316
Copy link
Member

@tim-hoffman tim-hoffman left a comment

Choose a reason for hiding this comment

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

LGTM

@iangneal iangneal merged commit d3e053e into main Feb 17, 2026
9 checks passed
@iangneal iangneal deleted the iangneal/analysis-tests branch February 17, 2026 20:54
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