-
Notifications
You must be signed in to change notification settings - Fork 51
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
Use NewPM wherever possible #482
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #482 +/- ##
==========================================
- Coverage 82.92% 75.03% -7.90%
==========================================
Files 23 23
Lines 3069 3256 +187
==========================================
- Hits 2545 2443 -102
- Misses 524 813 +289
☔ View full report in Codecov by Sentry. |
bfff46d
to
48e5e55
Compare
Thanks @pchintalapudi for the initial version and the LLVM.jl updates! |
Anywhere we fail to convert to NewPM/it's very tedious points to a hole in the NewPM API, so this serves to flush out remaining API issues.
Known issues: