Skip to content

Conversation

jketema
Copy link
Contributor

@jketema jketema commented Aug 30, 2024

No description provided.

While here, remove some dead code related to fixed points from the database
scheme.
@jketema jketema added the depends on internal PR This PR should only be merged in sync with an internal Semmle PR label Sep 3, 2024
@jketema jketema marked this pull request as ready for review September 3, 2024 16:17
@jketema jketema requested a review from a team as a code owner September 3, 2024 16:17
@jketema jketema added the no-change-note-required This PR does not need a change note label Sep 3, 2024
@jketema jketema requested a review from geoffw0 September 4, 2024 13:29
Copy link
Contributor

@geoffw0 geoffw0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One question...

| file://:0:0:0:0 | VariableAddress: (unnamed local variable) | Variable address instruction 'VariableAddress: (unnamed local variable)' has no associated variable, in function '$@'. | coroutines.cpp:95:20:95:38 | co_returnable_void co_yield_value_void(int) | co_returnable_void co_yield_value_void(int) |
| file://:0:0:0:0 | VariableAddress: (unnamed local variable) | Variable address instruction 'VariableAddress: (unnamed local variable)' has no associated variable, in function '$@'. | coroutines.cpp:99:21:99:40 | co_returnable_value co_yield_value_value(int) | co_returnable_value co_yield_value_value(int) |
| file://:0:0:0:0 | VariableAddress: (unnamed local variable) | Variable address instruction 'VariableAddress: (unnamed local variable)' has no associated variable, in function '$@'. | coroutines.cpp:103:20:103:43 | co_returnable_void co_yield_and_return_void(int) | co_returnable_void co_yield_and_return_void(int) |
| file://:0:0:0:0 | VariableAddress: (unnamed local variable) | Variable address instruction 'VariableAddress: (unnamed local variable)' has no associated variable, in function '$@'. | coroutines.cpp:108:21:108:45 | co_returnable_value co_yield_and_return_value(int) | co_returnable_value co_yield_and_return_value(int) |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎉

…ine.ql

Co-authored-by: Geoffrey White <40627776+geoffw0@users.noreply.github.com>
@jketema jketema requested a review from geoffw0 September 4, 2024 16:47
@jketema jketema merged commit b6e38ff into github:main Sep 5, 2024
12 of 15 checks passed
@jketema jketema deleted the coroutine branch September 5, 2024 08:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C++ depends on internal PR This PR should only be merged in sync with an internal Semmle PR no-change-note-required This PR does not need a change note
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants