Skip to content

feat: Add !riscv_stack.ptr<type>#808

Merged
math-fehr merged 1 commit into
mainfrom
nchappe/riscv-stack-slot-type
Jun 9, 2026
Merged

feat: Add !riscv_stack.ptr<type>#808
math-fehr merged 1 commit into
mainfrom
nchappe/riscv-stack-slot-type

Conversation

@nchappe

@nchappe nchappe commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

This uses xdslproject/xdsl#6086 as a reference

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

VeIR Benchmarks

Details
Benchmark suite Current: 0215d96 Previous: 54ffa4d Ratio
add-fold-worklist/create 2201500 ns (± 106368) 2200000 ns (± 106814) 1.00
add-fold-worklist/rewrite 3737500 ns (± 63810) 3734000 ns (± 92827) 1.00
add-fold-worklist-local/create 2315500 ns (± 104681) 2198000 ns (± 81073) 1.05
add-fold-worklist-local/rewrite 3040000 ns (± 41544) 3038000 ns (± 49176) 1.00
add-zero-worklist/create 2195000 ns (± 57996) 2291000 ns (± 97819) 0.96
add-zero-worklist/rewrite 2380000 ns (± 25772) 2439000 ns (± 58917) 0.98
add-zero-reuse-worklist/create 1807000 ns (± 54431) 1875000 ns (± 67005) 0.96
add-zero-reuse-worklist/rewrite 1956000 ns (± 29467) 2015000 ns (± 80799) 0.97
mul-two-worklist/create 2185000 ns (± 58884) 2092000 ns (± 41217) 1.04
mul-two-worklist/rewrite 5148000 ns (± 57295) 5237000 ns (± 166004) 0.98
add-fold-forwards/create 2192000 ns (± 69078) 2197000 ns (± 94670) 1.00
add-fold-forwards/rewrite 2998000 ns (± 36520) 3038000 ns (± 71174) 0.99
add-zero-forwards/create 2208000 ns (± 49310) 2160000 ns (± 58338) 1.02
add-zero-forwards/rewrite 1920000 ns (± 16380) 1906000 ns (± 50754) 1.01
add-zero-reuse-forwards/create 1893000 ns (± 87771) 2515500 ns (± 246272) 0.75
add-zero-reuse-forwards/rewrite 1569000 ns (± 33912) 1703000 ns (± 127540) 0.92
mul-two-forwards/create 2189000 ns (± 106159) 2203500 ns (± 267324) 0.99
mul-two-forwards/rewrite 3604000 ns (± 96365) 3638000 ns (± 67541) 0.99
add-zero-reuse-first/create 1806000 ns (± 65236) 1816000 ns (± 107472) 0.99
add-zero-reuse-first/rewrite 8000 ns (± 1081) 8000 ns (± 626) 1
add-zero-lots-of-reuse-first/create 1806000 ns (± 81962) 1819000 ns (± 54984) 0.99
add-zero-lots-of-reuse-first/rewrite 754000 ns (± 18542) 783000 ns (± 26508) 0.96

This comment was automatically generated by workflow using github-action-benchmark.

@nchappe nchappe force-pushed the nchappe/riscv-stack-slot-type branch from 52cd600 to 0215d96 Compare June 9, 2026 17:49

@tobiasgrosser tobiasgrosser left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Nice. Thank you!

@math-fehr math-fehr added this pull request to the merge queue Jun 9, 2026
Merged via the queue into main with commit 12d1ef2 Jun 9, 2026
5 checks passed
@math-fehr math-fehr deleted the nchappe/riscv-stack-slot-type branch June 9, 2026 22:14
nchappe added a commit that referenced this pull request Jun 10, 2026
nchappe added a commit that referenced this pull request Jun 10, 2026
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