Skip to content

[clang] Correctly RAII reset two "new" Sema fields - #23783

Open
hahnjo wants to merge 3 commits into
root-project:masterfrom
hahnjo:cling-raii
Open

hahnjo wants to merge 3 commits into
root-project:masterfrom
hahnjo:cling-raii

Conversation

@hahnjo

@hahnjo hahnjo commented Oct 21, 2026

Copy link
Copy Markdown
Member

Reapply #15004 / #19137 which was lost during the upgrade to LLVM 20.

FYI @ktf

FunctionScopesStart and InventedParameterInfosStart were introduced
by upstream commit
llvm/llvm-project@5c845c1
and are handled in ContextRAII, but not our ContextAndScopeRAII.

(cherry picked from commit b702391
and a226dd4, it was lost (again)
during the upgrade to LLVM 20)
(cherry picked from commit 07021ea
and 8e9e1ae, it was lost (again)
during the upgrade to LLVM 20)
@hahnjo hahnjo added this to the 6.42.00 milestone Oct 21, 2026
@hahnjo
hahnjo requested a review from dpiparo October 21, 2026 12:11
@hahnjo hahnjo self-assigned this Oct 21, 2026
@hahnjo
hahnjo requested a review from vgvassilev as a code owner October 21, 2026 12:11
@hahnjo hahnjo added in:Cling experiment Affects an experiment / reported by its software & computimng experts labels Oct 21, 2026
@github-actions

github-actions Bot commented Oct 21, 2026 •

Copy link
Copy Markdown

Test Results

    24 files      24 suites   3d 12h 9m 5s ⏱️
 3 871 tests  3 845 ✅ 0 💤  26 ❌
76 336 runs  75 880 ✅ 0 💤 456 ❌

For more details on these failures, see this check.

Results for commit c68eb06.

♻️ This comment has been updated with latest results.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

experiment Affects an experiment / reported by its software & computimng experts in:Cling

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant