Skip to content

C++: Update expected test results after frontend update - #22760

Merged
jketema merged 2 commits into
github:mainfrom
jketema:jketema/frontend
Oct 13, 2026
Merged

jketema merged 2 commits into
github:mainfrom
jketema:jketema/frontend

Conversation

@jketema

@jketema jketema commented Oct 6, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions github-actions Bot added the C++ label Oct 6, 2026
@jketema jketema added the depends on internal PR This PR should only be merged in sync with an internal Semmle PR label Oct 9, 2026
@jketema
jketema marked this pull request as ready for review October 13, 2026 06:23
@jketema
jketema requested review from a team as code owners October 13, 2026 06:23
Copilot AI balanced review requested due to automatic review settings October 13, 2026 06:23

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

The documentation and generated expectations consistently reflect the frontend update.

0 open findings

What changed in this PR

Updates C/C++ frontend compatibility documentation and aligns generated IR/AST expectations with the frontend’s revised temporary handling.

Changes:

  • Documents Clang 23 and Visual Studio 2026 support.
  • Updates generated expectations for temporary objects and destructor placement.
File Description
docs/​codeql/​reusables/​supported-versions-compilers.rst Updates supported compiler versions.
cpp/​ql/​test/​library-tests/​ir/​ir/​raw_ir.expected Refreshes raw IR expectations.
cpp/​ql/​test/​library-tests/​ir/​ir/​PrintAST.expected Refreshes AST expectations.
cpp/​ql/​test/​library-tests/​ir/​ir/​aliased_ir.expected Refreshes aliased IR expectations.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@geoffw0 geoffw0 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks reasonable.

@jketema
jketema merged commit 4812cdd into github:main Oct 13, 2026
11 of 12 checks passed
@jketema
jketema deleted the jketema/frontend branch October 13, 2026 08:40
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