-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Overlay: Add manual Java overlay annotations & discard predicates #19813
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
6f5bc41
to
8672313
Compare
8672313
to
db52a3c
Compare
db52a3c
to
26896ea
Compare
The |
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 happy with this if @aschackmull is happy
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 think we can simplify the boilerplate somewhat.
54589c8
to
bcadf31
Compare
bcadf31
to
a9f1540
Compare
a9f1540
to
64f27e2
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
The base branch was changed.
@ginsbach I've merged in main and added a missing QLDoc since @aschackmull's review. |
This PR builds on top of #19779, adding a few additional manual overlay annotations and defining entity discard predicates for Java.
The
overlay[local?]
annotations in the experimental queries are needed to ensure that virtual dispatch (which depends indirectly onActiveExperimentalModels
) becomes local. In addition to the manualoverlay[local?]
annotations, the PR adds anoverlay[global]
annotation inDataFlowImplCommon.qll
to ensure lambda flow is global.The entity discard predicates defined in this PR are not exhaustive and additional discarding may be needed. The PR adds entity discard predicates to discard the following: