-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
compilers/clike: Speedup cross_compute_int #13926
base: master
Are you sure you want to change the base?
Conversation
357fc38
to
95390c0
Compare
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.
LGTM now, can we rebase and check CI?
Also could you please verify that this codepath is indeed tested on the CI in one of the cross-compile jobs? We also need to ensure that the old codepath is hit in one of the other jobs. |
Given the CI failures, this isn't ready to merge yet. Methinks I'll need to catch the exception thrown by get_define in cross_compute_int |
8c25d74
to
2e70987
Compare
Expand the expression passed into cross_compute_int using the preprocessor first and then try to evaluate the expanded expression using the host machine compiler and test if the result is valid. Co-authored-by: Charles Brunet <[email protected]>
3c4fdd8
to
e8c3fb1
Compare
that'd mean |
You want to add such a test to |
e8c3fb1
to
0834eb0
Compare
0834eb0
to
e8c3fb1
Compare
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.
LGTM, will merge on Friday if there are no objections.
Expand the expression passed into cross_compute_int using the preprocessor first and then try to evaluate the expanded expression using the host machine compiler and test if the result is valid.
See #13910 for previous discussion