Skip to content

Conversation

jketema
Copy link
Contributor

@jketema jketema commented Jul 27, 2022

These builtins are supported by our compiler frontend, and occur in internal testing. Test failures are expected, because this depends on an internal PR.

@jketema jketema requested a review from a team as a code owner July 27, 2022 08:29
@jketema jketema added the depends on internal PR This PR should only be merged in sync with an internal Semmle PR label Jul 27, 2022
* It is assigned `ErroneousType` as its type.
* If the expression could not be resolved, it has type `ErroneousType`. In the
* case of a database upgrade or downgrade, the original type from before the
* upgrade or downgrade is kept if that type can be represented.
Copy link
Contributor

Choose a reason for hiding this comment

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

This is great detail, thanks for adding it.

@jketema jketema changed the title C++: Support additional compiler builtins C++: Support additional compiler builtin operations Jul 27, 2022
Copy link
Contributor

@redsun82 redsun82 left a comment

Choose a reason for hiding this comment

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

🚀

@jketema jketema merged commit b50f4bc into github:main Jul 29, 2022
@jketema jketema deleted the builtin branch July 29, 2022 15:20
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 documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants