-
Notifications
You must be signed in to change notification settings - Fork 404
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
Handle libraries with call protection #1000
Conversation
Since we discovered that this PR should handle all the cases, can we proceed? From my pov after writing a test we can merge, what do you think @kuzdogan? |
Does this also cover the
and merge 👍 |
I suppose the actual function |
* legacy compilation placeholder test case * via-IR test case
I added the tests |
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## staging #1000 +/- ##
===========================================
- Coverage 77.67% 73.86% -3.82%
===========================================
Files 29 38 +9
Lines 1375 2154 +779
Branches 254 408 +154
===========================================
+ Hits 1068 1591 +523
- Misses 178 347 +169
- Partials 129 216 +87
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
See #996
View in Huly HI-710