-
Notifications
You must be signed in to change notification settings - Fork 33
Remove psyir if branch #2869
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
base: master
Are you sure you want to change the base?
Remove psyir if branch #2869
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #2869 +/- ##
==========================================
- Coverage 99.88% 99.80% -0.09%
==========================================
Files 359 360 +1
Lines 50860 50917 +57
==========================================
+ Hits 50804 50818 +14
- Misses 56 99 +43 ☔ View full report in Codecov by Sentry. |
| super().__init__() | ||
| self._known_reference_bool: Dict[str, bool] = {} | ||
| self._known_reference_int: Dict[str, bool] = {} | ||
| if json_file_abspath is not None: |
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.
This part would need to either be removed entirely (and the transformation purely on PSyIR information for determining compile-time constants) or replaced with functionality that enables a list of names/symbols to be provided to the apply method (to support run-time constants).
|
Thanks for this Hugo. Obviously this will need full testing etc. Also, the reading of the json file needs to be replaced with one or more parameters to |
e75f500 to
6e55a14
Compare
This PR adds a new PsyIR transformation.
What this transformation should do exactly should be discussed between @schreiberx @arporter and other interested.
Currently there is no test, it is only reusing the code present in Martin's PR #2801.