[all-commits] [llvm/llvm-project] 887f70: [NFC][bolt][test] Change '|&' to '2>&1 |' for lit ...

[Connie via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 887f7002b65f7376c7a5004535bd08c95bdaa8f8
      https://github.com/llvm/llvm-project/commit/887f7002b65f7376c7a5004535bd08c95bdaa8f8
  Author: Connie <60797237+connieyzhu at users.noreply.github.com>
  Date:   2024-08-12 (Mon, 12 Aug 2024)

  Changed paths:
    M bolt/test/X86/encoding-validation.s
    M bolt/test/X86/gotpcrelx.s
    M bolt/test/X86/icf-jump-tables.test
    M bolt/test/X86/indirect-goto-pie.test
    M bolt/test/X86/jump-table-func-entry.s
    M bolt/test/X86/keep-nops.s
    M bolt/test/X86/linux-bug-table.s
    M bolt/test/X86/linux-orc.s
    M bolt/test/X86/linux-pci-fixup.s
    M bolt/test/X86/linux-smp-locks.s
    M bolt/test/X86/linux-static-calls.s
    M bolt/test/X86/linux-static-keys.s
    M bolt/test/X86/pt_gnu_relro.s
    M bolt/test/X86/unclaimed-jt-entries.s
    M bolt/test/X86/vararg.test
    M bolt/test/runtime/X86/unclaimed-jt-entries.s

  Log Message:
  -----------
  [NFC][bolt][test] Change '|&' to '2>&1 |' for lit internal shell support (#102402)

This patches changes all references to '|&' in bolt tests to instead use
the '2>&1 |' syntax for better consistency across testing and so that
lit's internal shell can be used to run these tests. This addresses a
suggestion made in the comments of this RFC:
https://discourse.llvm.org/t/rfc-enabling-the-lit-internal-shell-by-default/80179.

Fixes https://github.com/llvm/llvm-project/issues/102388



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