[test] Add tests for previously failing interactive sessions#22900
[test] Add tests for previously failing interactive sessions#22900dpiparo wants to merge 1 commit into
Conversation
ferdymercury
left a comment
There was a problem hiding this comment.
Thanks for the test!
There was another potential test case mentioned that could be added
size_t al(size_t p, size_t alignment) { return ((p + (alignment - 1)) & ~(alignment - 1)); }
void printextra(size_t use, size_t p, size_t alignment) { auto realstart = al(p+use, alignment); std::cout << "alloc: " << p << " data: " << realstart << " waste: " << realstart - p - use << '\n' };
void printextra(size_t use, size_t p, size_t alignment) { auto realstart = al(p+use, alignment); std::cout << "alloc: " << p << " data: " << realstart << " waste: " << realstart - p - use << '\n'; };
printextra(24, 56, 64)
that lead to cling JIT session error: Failed to materialize symbols: { (main, { __ZNSt3__1lsB7v160006INS_11char_traitsIcEEEERNS_13basic_ostreamIcT_EES6_c }) }
which is also now solved in master.
Test Results 23 files 23 suites 3d 12h 56m 10s ⏱️ Results for commit eb0f9b9. ♻️ This comment has been updated with latest results. |
|
Thanks a lot @ferdymercury . Just to be sure: what is the expected output of that snippet? |
It's coming from #14788 but I believe the expected output should be: instead of and or in macos |
|
Thanks. I wanted to make sure I got it right. This might potentially be a different problem. |
|
The last change is an attempt to prove the point made above. |
Moreover increase coverage on arm, now that exception trapping in interpreted code is supported. Fixes root-project#6439
fixes #6439