-
Notifications
You must be signed in to change notification settings - Fork 9
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
Allow for selection of diversly contextualized main functions in Bir #184
Merged
Merged
Changes from 13 commits
Commits
Show all changes
19 commits
Select commit
Hold shift + click to select a range
41235c5
Expand Bir.program and Bir interface to expose both bare main_functio…
mdurero bf82d9d
Add context_with_reset_function in Bir.program similar to context_fun…
mdurero 94d7448
Make contex_function_with_reset and context_function diverge
mdurero d967caf
Swap behaviour of Bir.main_statements and its first derivative with/n…
mdurero 095ce46
Remove the Bir-built context functions from the program function map …
mdurero 340ce1a
Replace context functions by an optional context record of 3 statemen…
mdurero 7eabd5b
Remove Test_interpreter.add_test_conds_to_combined_program, the condi…
mdurero 5eaac4f
Remove Bir_interface.context_agnostic_mpp_functions as program mpp_fu…
mdurero c79a4ce
Add context statements in Oir.program to pass it from the initial Bir…
mdurero f596448
Revert "Add context statements in Oir.program to pass it from the ini…
mdurero c0204da
Merge branch 'master' into main-function-selection-in-bir
mdurero f52cc2f
Transmit context statements transparently through Oir.program
mdurero adbdfa5
Merge remote-tracking branch 'origin/master' at release 98 into main-…
mdurero 5b66e73
Delete outdated comment
mdurero 5bcebcc
Merge branch 'master' at rev100 into main-function-selection-in-bir
mdurero d9bf2c9
Merge branch 'master' at release 102 into main-function-selection-in-bir
mdurero 29db93e
Merge remote-tracking branch 'origin/master' at release 103 into main…
mdurero 5168b57
Rename main_statements_with_reset to main_statements_with_context_and…
mdurero b8e0f56
Documentation comment on program context
mdurero File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -752,7 +752,9 @@ module Make (N : Bir_number.NumberInterface) = struct | |
(code_loc_start_value : int) : ctx = | ||
try | ||
let ctx = | ||
evaluate_stmts p ctx (Bir.main_statements p) [] code_loc_start_value | ||
evaluate_stmts p ctx | ||
(Bir.main_statements_with_reset p) | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Here a little comment about why we reset would be good |
||
[] code_loc_start_value | ||
in | ||
ctx | ||
with RuntimeError (e, ctx) -> | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
I'm ok with the change but you should put the rationale about why we need this context as a documentation comment here (you can start from your PR description which is quite informative).
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.
Documentation comment proposed in commit b8e0f56.