-
Notifications
You must be signed in to change notification settings - Fork 66
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
tests: Add integration tests for mixed-mode unwinding #1943
Conversation
853c949
to
9a968b1
Compare
Mmm the JITed stacks are consistently failing in CI... |
I have added some comments. I haven't checked the failures, but one of those could be the reason for the constant failure. |
It's unrelated to the test commit I added to test some hypothesis. Before, it was failing with empty JIT stacks |
21e2558
to
435a1ef
Compare
aka JIT <-> DWARF unwinding. This also tests that the symbolization using perfdump works. Note that asserting on the JIT stacks is temporarily disabled in CI as it's consistently failing and we have other features to work on that are higher priority. Landing this now will at least ensure that this is tested locally. Will make sure that this CI exception is lifted once we have more time. Also changed the running time for the profiler during non-CI runs to reduce feedback cycles. Signed-off-by: Francisco Javier Honduvilla Coto <javierhonduco@gmail.com>
bd41776
to
ec2fe0d
Compare
In the spirit of pragmatism I have disabled the assertions of JIT'd stacks in CI. Will fix this later on. With these changes, we will test these code paths locally, and the iteration times will be reduced, too. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🥇
aka JIT <-> DWARF unwinding. This also tests that the symbolization using
perfdump works.
Also changed the running time for the profiler during non-CI runs to
reduce feedback cycles.