[all-commits] [llvm/llvm-project] 4e6fa7: [lldb][test] Explicitly add target triple to no_un...

[Michael Buch via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 4e6fa78f4dc8ba01eb63ccaf010f985df6e4a361
      https://github.com/llvm/llvm-project/commit/4e6fa78f4dc8ba01eb63ccaf010f985df6e4a361
  Author: Michael Buch <michaelbuch12 at gmail.com>
  Date:   2024-10-16 (Wed, 16 Oct 2024)

  Changed paths:
    R lldb/test/Shell/SymbolFile/DWARF/no_unique_address-alignment.cpp
    R lldb/test/Shell/SymbolFile/DWARF/no_unique_address-base-alignment.cpp
    A lldb/test/Shell/SymbolFile/DWARF/x86/no_unique_address-alignment.cpp
    A lldb/test/Shell/SymbolFile/DWARF/x86/no_unique_address-base-alignment.cpp

  Log Message:
  -----------
  [lldb][test] Explicitly add target triple to no_unique_address Shell tests (#112523)

Follow up to https://github.com/llvm/llvm-project/pull/111902.

Makes sure all the `no_unique_address` tests are in the same place and
we don't rely on the host target triple (which means we don't need to
account for `[[msvc::no_unique_address]]` on Windows).

Now that we don't compile with the host compiler, this patch also adds
`-c` to the compilation command since we don't actually need the linked
binary in the test anyway (and on Darwin linking through Clang requires
the `xcrun` prefix to set up the SDK paths, etc.). We already do this in
`no_unique_address-with-bitfields.cpp` anyway.



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