diff options
| author | GeorgeHuyubo <113479859+GeorgeHuyubo@users.noreply.github.com> | 2025-11-10 15:21:06 -0800 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2025-11-10 15:21:06 -0800 |
| commit | 46dc4b841daad98c8358ac873827b6de7ce104c5 (patch) | |
| tree | adba61499a3a7b8b4cd2293c8435f7f7c50bc4e2 /llvm/test/tools/llvm-rc/inline-manifest.test | |
| parent | e5e74e987751c88eeaa453a9a2ef7840f9d44a62 (diff) | |
| download | llvm-main.zip llvm-main.tar.gz llvm-main.tar.bz2 | |
After commit fce58897ce82 enabled the locate_module callback for main
executables, the TestLocationsAfterRebuild.py test started failing on
remote platforms. The test was hardcoded to expect breakpoint location
"1.3" to exist after three rebuilds, but the new module loading behavior
changed how breakpoint locations are managed.
This patch fixes the test by:
- Removing the @skipIfRemote decorator to re-enable testing on remote
platforms
- Dynamically querying the actual number of breakpoint locations instead
of assuming a specific count
- Using loc.GetID() to get actual location IDs rather than assuming
sequential IDs (1, 2, 3)
- Iterating through all valid locations and verifying each can be
disabled/enabled
The fix maintains the original test intent (validating that breakpoint
location IDs remain valid after rebuilds) while adapting to the new
module loading behavior where the number and IDs of locations may vary
across platforms.
Co-authored-by: George Hu <georgehuyubo@gmail.com>
Diffstat (limited to 'llvm/test/tools/llvm-rc/inline-manifest.test')
0 files changed, 0 insertions, 0 deletions
