-
Notifications
You must be signed in to change notification settings - Fork 36
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
[Frontend] Replace IR with bytecode #1311
base: main
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1311 +/- ##
=======================================
Coverage 81.43% 81.44%
=======================================
Files 74 74
Lines 8172 8174 +2
Branches 861 861
=======================================
+ Hits 6655 6657 +2
Misses 1460 1460
Partials 57 57 ☔ View full report in Codecov by Sentry. 🚨 Try these New Features:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks Erick! Would also to see whether this makes a difference in compile time.
Btw does this affect the 0_*
stage in intermediate files or not really?
No. |
Context: Writing uncompressed human readable IR to disk takes longer to parse and dump than using compressed MLIR bytecode. Benchmark results discussed offline.
Description of the Change: Use compressed MLIR bytecode when sending a module to the compiler.
Benefits: Less compilation time (depending on how big the IR produced is)
Possible Drawbacks:
Related GitHub Issues:
[sc-78851]