[all-commits] [llvm/llvm-project] 50d2a1: [OpenMP] Only test kmp_atomic_float10_max_min.c on...

Shilei Tian via All-commits all-commits at lists.llvm.org
Fri Jan 20 07:53:07 PST 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 50d2a193a7399ce99e321b93d6ee85184d3be4de
      https://github.com/llvm/llvm-project/commit/50d2a193a7399ce99e321b93d6ee85184d3be4de
  Author: Shilei Tian <i at tianshilei.me>
  Date:   2023-01-20 (Fri, 20 Jan 2023)

  Changed paths:
    M openmp/runtime/test/atomic/kmp_atomic_float10_max_min.c
    M openmp/runtime/test/lit.cfg
    M openmp/runtime/test/lit.site.cfg.in

  Log Message:
  -----------
  [OpenMP] Only test kmp_atomic_float10_max_min.c on X86

The test `openmp/runtime/test/atomic/kmp_atomic_float10_max_min.c` uses a compiler
flag `-mlong-double-80` that might not be supported by all targets. Currently it
requires `x86-registered-target`, but that requirement can be true when LLVM
supports X86 while the actual `libomp` arch is not X86. For example, when LLVM
is built on AArch64 with all targets enabled, `x86-registered-target` can be met.
If `libomp` is built with native target, aka. AArch64, the test will still be enabled,
causing test failure.

This patch only enables the test if the actual target is X86. The actual target
is determined by `LIBOMP_ARCH`.

Fix #53696.

Reviewed By: jlpeyton

Differential Revision: https://reviews.llvm.org/D142172




More information about the All-commits mailing list