Skip to content

[6.38] [cling][test] Fix failing cling tests in master#21273

Open
dpiparo wants to merge 4 commits intoroot-project:v6-38-00-patchesfrom
root-project-bot:BP_6.38_pull_21232
Open

[6.38] [cling][test] Fix failing cling tests in master#21273
dpiparo wants to merge 4 commits intoroot-project:v6-38-00-patchesfrom
root-project-bot:BP_6.38_pull_21232

Conversation

@dpiparo
Copy link
Member

@dpiparo dpiparo commented Feb 13, 2026

Backport of #21232, requested by @foo. For your information @devajithvs

When cling is built with C++17, the implementation in ValuePrinter.cpp
is not compiled. When cling is later used with -std=c++20, we don't have
the implementation to print `source_location`

(cherry picked from commit 2a20a68)
Fix the following assertion failure in cling:

```
[cling]$ #include <filesystem>
Transaction.cpp:180: void cling::Transaction::forceAppend(DelayCallInfo):
Assertion `oldDCI != DCI && "Duplicates?!"' failed.
```

(cherry picked from commit 43610ec)
LLVM relies on libedit for line editing. History and other input
functionalities might not work as expected without this.

(cherry picked from commit effafd1)
@github-actions
Copy link

Test Results

    19 files      19 suites   3d 9h 15m 31s ⏱️
 3 715 tests  3 715 ✅ 0 💤 0 ❌
68 853 runs  68 853 ✅ 0 💤 0 ❌

Results for commit ef1b246.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants