[all-commits] [llvm/llvm-project] a575e6: Fix a couple of tests that were incorrectly using ...

jimingham via All-commits all-commits at lists.llvm.org
Wed Oct 30 09:26:09 PDT 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: a575e6e5ca1eb7b2ae4b906f9bf3be2ba20a80a0
      https://github.com/llvm/llvm-project/commit/a575e6e5ca1eb7b2ae4b906f9bf3be2ba20a80a0
  Author: jimingham <jingham at apple.com>
  Date:   2024-10-30 (Wed, 30 Oct 2024)

  Changed paths:
    M lldb/packages/Python/lldbsuite/test/configuration.py
    M lldb/test/API/lang/cpp/namespace/TestNamespaceLookup.py
    M lldb/test/API/python_api/type/TestTypeList.py

  Log Message:
  -----------
  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.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications


More information about the All-commits mailing list