Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[lit-labs/compiler] initial README.md #4099

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.

Sign up for GitHub

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

Jump to bottom
Merged
AndrewJakubowicz merged 7 commits into from
Aug 15, 2023
Merged

[lit-labs/compiler] initial README.md #4099

AndrewJakubowicz merged 7 commits into from
Aug 15, 2023

Conversation

@AndrewJakubowicz
Copy link
Contributor

@AndrewJakubowicz AndrewJakubowicz commented Aug 15, 2023

Issue: #189
RFC: lit/rfcs#21
Full prototype PR: #3984

This is a documentation only change, to provide initial documentation for the landed compiler changes.

Thank you :D

@changeset-bot
Copy link

changeset-bot bot commented Aug 15, 2023 •
edited
Loading

🦋 Changeset detected

Latest commit: f964469

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 0 packages

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions
Copy link
Contributor

github-actions bot commented Aug 15, 2023 •
edited
Loading

📊 Tachometer Benchmark Results

Summary

nop-update

  • this-change, tip-of-tree, previous-release: unsure 🔍 -1% - +12% (-0.22ms - +2.25ms)
    this-change vs tip-of-tree

render

  • this-change: 70.40ms - 73.25ms
  • this-change, tip-of-tree, previous-release: unsure 🔍 -5% - +4% (-1.33ms - +1.20ms)
    this-change vs tip-of-tree
  • this-change, tip-of-tree, previous-release: unsure 🔍 -1% - +3% (-0.33ms - +1.31ms)
    this-change vs tip-of-tree
  • this-change, tip-of-tree, previous-release: unsure 🔍 -2% - +2% (-0.78ms - +0.89ms)
    this-change vs tip-of-tree

update

  • this-change: 782.94ms - 793.14ms
  • this-change, tip-of-tree, previous-release: unsure 🔍 -3% - +7% (-1.92ms - +5.29ms)
    this-change vs tip-of-tree
  • this-change, tip-of-tree, previous-release: unsure 🔍 -2% - +2% (-2.22ms - +1.92ms)
    this-change vs tip-of-tree
  • this-change, tip-of-tree, previous-release: unsure 🔍 -2% - +1% (-15.06ms - +4.51ms)
    this-change vs tip-of-tree

update-reflect

  • this-change: 757.62ms - 766.13ms
  • this-change, tip-of-tree, previous-release: unsure 🔍 -1% - +1% (-5.59ms - +11.67ms)
    this-change vs tip-of-tree

Results

this-change

render

VersionAvg timevs
70.40ms - 73.25ms-

update

VersionAvg timevs
782.94ms - 793.14ms-

update-reflect

VersionAvg timevs
757.62ms - 766.13ms-
this-change, tip-of-tree, previous-release

render

VersionAvg timevs this-change
vs tip-of-tree
tip-of-tree		vs previous-release
previous-release
this-change
27.45ms - 29.32ms-unsure 🔍
-5% - +4%
-1.33ms - +1.20ms		unsure 🔍
-4% - +5%
-1.12ms - +1.28ms
tip-of-tree
tip-of-tree		27.61ms - 29.30msunsure 🔍
-4% - +5%
-1.20ms - +1.33ms		-unsure 🔍
-4% - +5%
-0.99ms - +1.28ms
previous-release
previous-release		27.55ms - 29.06msunsure 🔍
-5% - +4%
-1.28ms - +1.12ms		unsure 🔍
-4% - +3%
-1.28ms - +0.99ms		-

update

VersionAvg timevs this-change
vs tip-of-tree
tip-of-tree		vs previous-release
previous-release
this-change
73.89ms - 78.63ms-unsure 🔍
-3% - +7%
-1.92ms - +5.29ms		unsure 🔍
-5% - +3%
-4.08ms - +2.58ms
tip-of-tree
tip-of-tree		71.86ms - 77.29msunsure 🔍
-7% - +2%
-5.29ms - +1.92ms		-unsure 🔍
-8% - +1%
-6.02ms - +1.14ms
previous-release
previous-release		74.68ms - 79.34msunsure 🔍
-3% - +5%
-2.58ms - +4.08ms		unsure 🔍
-2% - +8%
-1.14ms - +6.02ms		-

nop-update

VersionAvg timevs this-change
vs tip-of-tree
tip-of-tree		vs previous-release
previous-release
this-change
19.73ms - 21.64ms-unsure 🔍
-1% - +12%
-0.22ms - +2.25ms		unsure 🔍
-11% - +2%
-2.35ms - +0.49ms
tip-of-tree
tip-of-tree		18.89ms - 20.45msunsure 🔍
-11% - +1%
-2.25ms - +0.22ms		-faster ✔
3% - 15%
0.64ms - 3.26ms
previous-release
previous-release		20.56ms - 22.66msunsure 🔍
-3% - +12%
-0.49ms - +2.35ms		slower ❌
3% - 17%
0.64ms - 3.26ms		-
this-change, tip-of-tree, previous-release

render

VersionAvg timevs this-change
vs tip-of-tree
tip-of-tree		vs previous-release
previous-release
this-change
50.48ms - 51.70ms-unsure 🔍
-1% - +3%
-0.33ms - +1.31ms		unsure 🔍
-2% - +1%
-1.06ms - +0.68ms
tip-of-tree
tip-of-tree		50.05ms - 51.16msunsure 🔍
-3% - +1%
-1.31ms - +0.33ms		-unsure 🔍
-3% - +0%
-1.51ms - +0.15ms
previous-release
previous-release		50.67ms - 51.90msunsure 🔍
-1% - +2%
-0.68ms - +1.06ms		unsure 🔍
-0% - +3%
-0.15ms - +1.51ms		-

update

VersionAvg timevs this-change
vs tip-of-tree
tip-of-tree		vs previous-release
previous-release
this-change
109.22ms - 112.01ms-unsure 🔍
-2% - +2%
-2.22ms - +1.92ms		unsure 🔍
-1% - +2%
-1.57ms - +1.97ms
tip-of-tree
tip-of-tree		109.24ms - 112.29msunsure 🔍
-2% - +2%
-1.92ms - +2.22ms		-unsure 🔍
-1% - +2%
-1.52ms - +2.22ms
previous-release
previous-release		109.33ms - 111.51msunsure 🔍
-2% - +1%
-1.97ms - +1.57ms		unsure 🔍
-2% - +1%
-2.22ms - +1.52ms		-
this-change, tip-of-tree, previous-release

render

VersionAvg timevs this-change
vs tip-of-tree
tip-of-tree		vs previous-release
previous-release
this-change
44.77ms - 46.00ms-unsure 🔍
-2% - +2%
-0.78ms - +0.89ms		unsure 🔍
-1% - +2%
-0.58ms - +1.03ms
tip-of-tree
tip-of-tree		44.77ms - 45.90msunsure 🔍
-2% - +2%
-0.89ms - +0.78ms		-unsure 🔍
-1% - +2%
-0.60ms - +0.94ms
previous-release
previous-release		44.64ms - 45.69msunsure 🔍
-2% - +1%
-1.03ms - +0.58ms		unsure 🔍
-2% - +1%
-0.94ms - +0.60ms		-

update

VersionAvg timevs this-change
vs tip-of-tree
tip-of-tree		vs previous-release
previous-release
this-change
788.31ms - 802.40ms-unsure 🔍
-2% - +1%
-15.06ms - +4.51ms		unsure 🔍
-2% - +0%
-16.05ms - +3.51ms
tip-of-tree
tip-of-tree		793.84ms - 807.41msunsure 🔍
-1% - +2%
-4.51ms - +15.06ms		-unsure 🔍
-1% - +1%
-10.59ms - +8.59ms
previous-release
previous-release		794.85ms - 808.40msunsure 🔍
-0% - +2%
-3.51ms - +16.05ms		unsure 🔍
-1% - +1%
-8.59ms - +10.59ms		-

update-reflect

VersionAvg timevs this-change
vs tip-of-tree
tip-of-tree		vs previous-release
previous-release
this-change
792.31ms - 804.36ms-unsure 🔍
-1% - +1%
-5.59ms - +11.67ms		unsure 🔍
-0% - +2%
-2.36ms - +13.60ms
tip-of-tree
tip-of-tree		789.12ms - 801.47msunsure 🔍
-1% - +1%
-11.67ms - +5.59ms		-unsure 🔍
-1% - +1%
-5.51ms - +10.67ms
previous-release
previous-release		787.49ms - 797.95msunsure 🔍
-2% - +0%
-13.60ms - +2.36ms		unsure 🔍
-1% - +1%
-10.67ms - +5.51ms		-

tachometer-reporter-action v2 for Benchmarks

augustjk
augustjk approved these changes Aug 15, 2023
View reviewed changes
Copy link
Member

@augustjk augustjk left a comment

Choose a reason for hiding this comment

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

Excellent! Just a couple minor suggestions.

justinfagnani
justinfagnani approved these changes Aug 15, 2023
View reviewed changes
@AndrewJakubowicz AndrewJakubowicz merged commit 02a1308 into main Aug 15, 2023
@AndrewJakubowicz AndrewJakubowicz deleted the compiler-readme branch August 15, 2023 17:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@justinfagnani justinfagnani justinfagnani approved these changes

@augustjk augustjk augustjk approved these changes

@kevinpschaaf kevinpschaaf Awaiting requested review from kevinpschaaf kevinpschaaf is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@AndrewJakubowicz @justinfagnani @augustjk