-
-
Notifications
You must be signed in to change notification settings - Fork 819
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
optimizer fixes and guardrails #2914
optimizer fixes and guardrails #2914
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2914 +/- ##
==========================================
- Coverage 88.47% 88.05% -0.43%
==========================================
Files 97 97
Lines 10613 10899 +286
Branches 2527 2637 +110
==========================================
+ Hits 9390 9597 +207
- Misses 770 836 +66
- Partials 453 466 +13
Continue to review full report at Codecov.
|
0c30769
to
df2fda9
Compare
df2fda9
to
fdcb403
Compare
they will be checked later, during ir-to-assembly
This pull request introduces 1 alert when merging c148926 into 4e38ec4 - view on LGTM.com new alerts:
|
there might be true dead code, so only run the exactly-once symbol checker on binop optimizations.
This pull request introduces 1 alert when merging c057cb6 into 4e38ec4 - view on LGTM.com new alerts:
|
What I did
How I did it
How to verify it
Commit message
Description for the changelog
Cute Animal Picture