-
Notifications
You must be signed in to change notification settings - Fork 58
fix(codegen): Convert pointer to actual type when passing by-value #1039
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
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #1039 +/- ##
==========================================
+ Coverage 95.71% 95.73% +0.02%
==========================================
Files 153 153
Lines 42715 42883 +168
==========================================
+ Hits 40883 41054 +171
+ Misses 1832 1829 -3 ☔ View full report in Codecov by Sentry. |
...usty__codegen__tests__debug_tests__expression_debugging__for_conditions_location_marked.snap
Outdated
Show resolved
Hide resolved
This reverts commit 9037523.
This reverts commit 7e2183b.
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.
Nice find!
Resolves #1037
We were missing an edge-case where an argument is passed into a function A by-ref (INOUT) which in turn is passed into a function B by-val (INPUT) which generated the following IR for a
DINT
arrayThis PR fixes it, in that the commented IR is generated - specifically before passing the argument into function B it is bitcast into its actual type.
(A minimal example to reproduce on master)