Skip to content

Raghav/inline structs refactor2#212

Merged
tim-hoffman merged 5 commits intomainfrom
raghav/inline-structs-refactor2
Nov 14, 2025
Merged

Raghav/inline structs refactor2#212
tim-hoffman merged 5 commits intomainfrom
raghav/inline-structs-refactor2

Conversation

@raghav198
Copy link
Contributor

No description provided.

@raghav198 raghav198 requested a review from a team November 12, 2025 20:29
@tim-hoffman tim-hoffman requested a review from a team November 14, 2025 20:32
Copy link
Contributor

@shankarapailoor shankarapailoor left a comment

Choose a reason for hiding this comment

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

Approving on behalf of @tim-hoffman

@tim-hoffman tim-hoffman merged commit ef9ae0f into main Nov 14, 2025
10 checks passed
raghav198 added a commit that referenced this pull request Dec 9, 2025
* Registered new transformation pass

* Concatenation pass works

* deleted comments

* Basic inlining complete

* Fixed issue calling @Product

* Refactoring, added root-struct option

* Aligning subfunctions

* Refactor to make struct inlining available on demand (#212)

* For now, fail when we can't align some calls

* [NFC] Refactor inline structs pass to prepare for handling more "remaining uses" (#218)

* add debug output, clarify docs, fix typos
* refactoring to prepare for handling more "remaining uses"
* update debug output with new function names

* Brought back LightweightSignalEquivalenceAnalysis

* Added testcase

* Shankara/pcl include fix (#221)

* used the wrong cmake flag when guarding PCL imports

* Added libAnalysis to CMake

* Removed libAnalysis from CMake

* Namespacing

* Namespacing

* Put analysis back

* Link `LLZKAnalysis` in `LLZKTransforms` (#225)

also explicitly link LLZKDialect in the dialects that use it to fix linker error that surfaced as a result

* remove redundant "add_dependencies" from cmake (#226)

* clang-format

* Changelog

* FORMAT

* Addressing comments

* update manual build instructions to build clang dependency

* Addressing comments

* Add missing headers to `llzk-tblgen` lib (#239)

* Refactor to add calleeContainsWitnessGen() and getSingleResultTypeOfWitnessGen() instead of overloading ...Compute()

* nit

* revert

---------

Co-authored-by: Raghav Malik <raghav@veridise.com>
Co-authored-by: Timothy Hoffman <4001421+tim-hoffman@users.noreply.github.com>
Co-authored-by: shankarapailoor <shankarapailoor@gmail.com>
Co-authored-by: Tim Hoffman <timothy.hoffman@veridise.com>
@tim-hoffman tim-hoffman deleted the raghav/inline-structs-refactor2 branch March 3, 2026 17:21
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