[all-commits] [llvm/llvm-project] faf070: [lldb][test] TestExprDefinitionInDylib.py adjust t...

[Michael Buch via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: faf070f062ac7f3861092ab110982a613d7dfe1b
      https://github.com/llvm/llvm-project/commit/faf070f062ac7f3861092ab110982a613d7dfe1b
  Author: Michael Buch <michaelbuch12 at gmail.com>
  Date:   2025-10-02 (Thu, 02 Oct 2025)

  Changed paths:
    M lldb/test/API/lang/cpp/expr-definition-in-dylib/TestExprDefinitionInDylib.py

  Log Message:
  -----------
  [lldb][test] TestExprDefinitionInDylib.py adjust test to account for older compiler versions

 Skip tests that require `-gstructor-decl-linkage-names` on Clang versions that don't support it.

 Don't pass `-gno-structor-decl-linkage-names` on Clang versions where it the flag didn't exist but it was the default behaviour of the compiler anyway.

Drive-by:
- We used to run `self.expect("Bar()")` which would always fail. So the `error=True` would be true even if we didn't pass the `-gno-structor-linkage-names`. So it wasn't testing the behaviour properly. This patch changes these to `self.expect("expr Bar()")`.



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