llvm-project.git/lldb/test/API/functionalities/thread/step_out/TestThreadStepOut.py, branch main Unnamed repository; edit this file 'description' to name the repository. [NFC][Py Reformat] Reformat python files in lldb 2023-05-25T19:54:09+00:00 Jonas Devlieghere jonas@devlieghere.com 2023-05-25T15:48:57+00:00 2238dcc39358353cac21df75c3c3286ab20b8f53 This is an ongoing series of commits that are reformatting our Python code. Reformatting is done with `black` (23.1.0). If you end up having problems merging this commit because you have made changes to a python file, the best way to handle that is to run `git checkout --ours <yourfile>` and then reformat it with black. RFC: https://discourse.llvm.org/t/rfc-document-and-standardize-python-code-style Differential revision: https://reviews.llvm.org/D151460 
This is an ongoing series of commits that are reformatting our Python
code. Reformatting is done with `black` (23.1.0).

If you end up having problems merging this commit because you have made
changes to a python file, the best way to handle that is to run `git
checkout --ours <yourfile>` and then reformat it with black.

RFC: https://discourse.llvm.org/t/rfc-document-and-standardize-python-code-style

Differential revision: https://reviews.llvm.org/D151460
[lldb][tests] Automatically call compute_mydir (NFC) 2022-06-17T21:34:49+00:00 Dave Lee davelee.com@gmail.com 2022-06-17T00:38:47+00:00 4cc8f2a017c76af25234afc7c380550e9c93135c Eliminate boilerplate of having each test manually assign to `mydir` by calling `compute_mydir` in lldbtest.py. Differential Revision: https://reviews.llvm.org/D128077 
Eliminate boilerplate of having each test manually assign to `mydir` by calling
`compute_mydir` in lldbtest.py.

Differential Revision: https://reviews.llvm.org/D128077
[lldb] [test] Update baseline test status for FreeBSD 2022-06-17T12:35:18+00:00 Michał Górny mgorny@moritz.systems 2022-06-17T09:16:30+00:00 13dfe0f0fc267f3f747a5e0802c0a8b01240d0af Fixes #19721 Fixes #18440 Partially fixes bug #47660 Fixes #47761 Fixes #47763 Sponsored by: The FreeBSD Foundation 
Fixes #19721
Fixes #18440
Partially fixes bug #47660
Fixes #47761
Fixes #47763

Sponsored by: The FreeBSD Foundation
[lldb] Enable TestThreadStepOut on linux 2022-02-16T10:18:13+00:00 Pavel Labath pavel@labath.sk 2022-02-16T10:16:36+00:00 9bde88182317325bb95c13190a4cf303d3ce4b54 After applying the same for as in TestThreadBacktraceRepeat, the test appears to pass reliably. The skip decorator was added many years ago, so it's not clear whether this is what caused it to hang. 
After applying the same for as in TestThreadBacktraceRepeat, the test
appears to pass reliably. The skip decorator was added many years ago,
so it's not clear whether this is what caused it to hang.
[lldb] Remove lldbtest.getBuildFlags 2021-12-30T11:19:24+00:00 Pavel Labath pavel@labath.sk 2021-12-30T10:22:26+00:00 d7dbe2c4a00ba2abd998328ad6b8023637bc71d9 It was being used only in some very old tests (which pass even without it) and its implementation is highly questionable. These days we have different mechanisms for requesting a build with a particular kind of c++ library (USE_LIB(STD)CPP in the makefile). 
It was being used only in some very old tests (which pass even without
it) and its implementation is highly questionable.

These days we have different mechanisms for requesting a build with a
particular kind of c++ library (USE_LIB(STD)CPP in the makefile).
Don't assume that stepping out of a function will land on the next line. 2021-03-19T00:44:17+00:00 Jim Ingham jingham@apple.com 2021-03-19T00:44:17+00:00 71c4da83b67a485f0cfacbce8b46eaa497df900e For instance, some recent clang emits this code on x86_64: 0x100002b99 <+57>: callq 0x100002b40 ; step_out_of_here at main.cpp:11 -> 0x100002b9e <+62>: xorl %eax, %eax 0x100002ba0 <+64>: popq %rbp 0x100002ba1 <+65>: retq and the "xorl %eax, %eax" is attributed to the same line as the callq. Since step out is supposed to stop just on returning from the function, you can't guarantee it will end up on the next line. I changed the test to check that we were either on the call line or on the next line, since either would be right depending on the debug information. 
For instance, some recent clang emits this code on x86_64:

    0x100002b99 <+57>: callq  0x100002b40               ; step_out_of_here at main.cpp:11
->  0x100002b9e <+62>: xorl   %eax, %eax
    0x100002ba0 <+64>: popq   %rbp
    0x100002ba1 <+65>: retq

and the "xorl %eax, %eax" is attributed to the same line as the callq.  Since
step out is supposed to stop just on returning from the function, you can't guarantee
it will end up on the next line.  I changed the test to check that we were either
on the call line or on the next line, since either would be right depending on the
debug information.
Reland [lldb] Fix TestThreadStepOut.py after "Flush local value map on every instruction" 2021-01-21T12:35:13+00:00 Raphael Isemann teemperor@gmail.com 2020-11-26T08:40:02+00:00 ed2853d2c82d7286ba510c8f65049d6f649017f0 The original patch got reverted as a dependency of cf1c774d6ace59c5adc9ab71b31e . That patch got relanded so it's also necessary to reland this patch. Original summary: After cf1c774d6ace59c5adc9ab71b31e762c1be695b1, Clang seems to generate code that is more similar to icc/Clang, so we can use the same line numbers for all compilers in this test. 
The original patch got reverted as a dependency of cf1c774d6ace59c5adc9ab71b31e .
That patch got relanded so it's also necessary to reland this patch.

Original summary:

After cf1c774d6ace59c5adc9ab71b31e762c1be695b1, Clang seems to generate code
that is more similar to icc/Clang, so we can use the same line numbers for
all compilers in this test.
[lldb] [test] Fix continue_to_breakpoint() args in TestThreadStepOut 2020-12-07T08:56:51+00:00 Michał Górny mgorny@moritz.systems 2020-12-07T08:31:35+00:00 77f0ea4b5b978c7caa0788ae8b1a2429089021bd The test is skipped/xfailing on all platforms, so it seems that the API got out of sync. Fix that so it returns to a 'proper' failure on FreeBSD. Differential Revision: https://reviews.llvm.org/D92746 
The test is skipped/xfailing on all platforms, so it seems that the API
got out of sync.  Fix that so it returns to a 'proper' failure
on FreeBSD.

Differential Revision: https://reviews.llvm.org/D92746
Revert "[FastISel] Flush local value map on ever instruction" and dependent patches 2020-12-01T22:26:23+00:00 David Blaikie dblaikie@gmail.com 2020-12-01T21:23:30+00:00 615f63e149f31d6a97b5233b4fe634db92e19aa9 This reverts commit cf1c774d6ace59c5adc9ab71b31e762c1be695b1. This change caused several regressions in the gdb test suite - at least a sample of which was due to line zero instructions making breakpoints un-lined. I think they're worth investigating/understanding more (& possibly addressing) before moving forward with this change. Revert "[FastISel] NFC: Clean up unnecessary bookkeeping" This reverts commit 3fd39d3694d32efa44242c099e923a7f4d982095. Revert "[FastISel] NFC: Remove obsolete -fast-isel-sink-local-values option" This reverts commit a474657e30edccd9e175d92bddeefcfa544751b2. Revert "Remove static function unused after cf1c774." This reverts commit dc35368ccf17a7dca0874ace7490cc3836fb063f. Revert "[lldb] Fix TestThreadStepOut.py after "Flush local value map on every instruction"" This reverts commit 53a14a47ee89dadb8798ca8ed19848f33f4551d5. 
This reverts commit cf1c774d6ace59c5adc9ab71b31e762c1be695b1.

This change caused several regressions in the gdb test suite - at least
a sample of which was due to line zero instructions making breakpoints
un-lined. I think they're worth investigating/understanding more (&
possibly addressing) before moving forward with this change.

Revert "[FastISel] NFC: Clean up unnecessary bookkeeping"
This reverts commit 3fd39d3694d32efa44242c099e923a7f4d982095.

Revert "[FastISel] NFC: Remove obsolete -fast-isel-sink-local-values option"
This reverts commit a474657e30edccd9e175d92bddeefcfa544751b2.

Revert "Remove static function unused after cf1c774."
This reverts commit dc35368ccf17a7dca0874ace7490cc3836fb063f.

Revert "[lldb] Fix TestThreadStepOut.py after "Flush local value map on every instruction""
This reverts commit 53a14a47ee89dadb8798ca8ed19848f33f4551d5.
[lldb] Fix TestThreadStepOut.py after "Flush local value map on every instruction" 2020-11-26T08:43:47+00:00 Raphael Isemann teemperor@gmail.com 2020-11-26T08:40:02+00:00 53a14a47ee89dadb8798ca8ed19848f33f4551d5 After cf1c774d6ace59c5adc9ab71b31e762c1be695b1, Clang seems to generate code that is more similar to icc/Clang, so we can use the same line numbers for all compilers in this test. 
After cf1c774d6ace59c5adc9ab71b31e762c1be695b1, Clang seems to generate code
that is more similar to icc/Clang, so we can use the same line numbers for
all compilers in this test.