diff options
| author | jimingham <jingham@apple.com> | 2024-10-30 09:25:47 -0700 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-10-30 09:25:47 -0700 |
| commit | a575e6e5ca1eb7b2ae4b906f9bf3be2ba20a80a0 (patch) | |
| tree | cfe11117de6e55fdd640dec8bac29cec0b990ed0 /lldb/packages/Python/lldbsuite/test/configuration.py | |
| parent | 6bf4476ffb6bab661d59dee361ab845b2f68d9b1 (diff) | |
| download | llvm-a575e6e5ca1eb7b2ae4b906f9bf3be2ba20a80a0.zip llvm-a575e6e5ca1eb7b2ae4b906f9bf3be2ba20a80a0.tar.gz llvm-a575e6e5ca1eb7b2ae4b906f9bf3be2ba20a80a0.tar.bz2 | |
Fix a couple of tests that were incorrectly using configuration.dwarf_version (#114161)
The tests were using the variable directly to get the dwarf version used
for the test. That's only the overridden value, and won't be set if
we're using the compiler default. I also put a comment by the variable
to make sure people don't make the same mistake in the future.
Diffstat (limited to 'lldb/packages/Python/lldbsuite/test/configuration.py')
| -rw-r--r-- | lldb/packages/Python/lldbsuite/test/configuration.py | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/lldb/packages/Python/lldbsuite/test/configuration.py b/lldb/packages/Python/lldbsuite/test/configuration.py index 1bacd74..bcc1793 100644 --- a/lldb/packages/Python/lldbsuite/test/configuration.py +++ b/lldb/packages/Python/lldbsuite/test/configuration.py @@ -46,6 +46,10 @@ sdkroot = None make_path = None # The overriden dwarf verison. +# Don't use this to test the current compiler's +# DWARF version, as this won't be set if the +# version isn't overridden. +# Use lldbplatformutils.getDwarfVersion() instead. dwarf_version = 0 # Any overridden settings. |