[all-commits] [llvm/llvm-project] 482e2d: [OpenMP] Fix warnings about unused expressions whe...

[Jonathan Peyton via All-commits]

  Branch: refs/heads/release/15.x
  Home:   https://github.com/llvm/llvm-project
  Commit: 482e2dc26e6cc1f0ebc81966948bd6ce2aa05b1d
      https://github.com/llvm/llvm-project/commit/482e2dc26e6cc1f0ebc81966948bd6ce2aa05b1d
  Author: Martin Storsjö <martin at martin.st>
  Date:   2022-08-09 (Tue, 09 Aug 2022)

  Changed paths:
    M openmp/runtime/src/kmp_dispatch.cpp

  Log Message:
  -----------
  [OpenMP] Fix warnings about unused expressions when OMPT_LOOP_DISPATCH is a no-op. NFC.

This fixes warnings like these:

../runtime/src/kmp_dispatch.cpp:2159:24: warning: left operand of comma operator has no effect [-Wunused-value]
    OMPT_LOOP_DISPATCH(*p_lb, *p_ub, pr->u.p.st, status);
                       ^~~~~
../runtime/src/kmp_dispatch.cpp:2159:31: warning: left operand of comma operator has no effect [-Wunused-value]
    OMPT_LOOP_DISPATCH(*p_lb, *p_ub, pr->u.p.st, status);
                              ^~~~~
../runtime/src/kmp_dispatch.cpp:2159:46: warning: left operand of comma operator has no effect [-Wunused-value]
    OMPT_LOOP_DISPATCH(*p_lb, *p_ub, pr->u.p.st, status);
                                     ~~~~~~~ ^~
../runtime/src/kmp_dispatch.cpp:2159:50: warning: expression result unused [-Wunused-value]
    OMPT_LOOP_DISPATCH(*p_lb, *p_ub, pr->u.p.st, status);
                                                 ^~~~~~

(cherry picked from commit 3f25ad335b4ef882cd4e336f9d9ac1606220f572)


  Commit: 53cfe2c77e584a6272a65c945e576d891a99bba8
      https://github.com/llvm/llvm-project/commit/53cfe2c77e584a6272a65c945e576d891a99bba8
  Author: David Sherwood <david.sherwood at arm.com>
  Date:   2022-08-09 (Tue, 09 Aug 2022)

  Changed paths:
    A llvm/test/CodeGen/AArch64/vector-fcvt.ll

  Log Message:
  -----------
  [NFC][AArch64] Precommit vector-fcvt tests

Add tests which show code quality of uitofp and sitofp.

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

(cherry picked from commit 6953e754c7a887b22c56ce7b0526506e4aa86e7f)


  Commit: 9f4d9a0b7b75210b284816e88362f48e7f64aff8
      https://github.com/llvm/llvm-project/commit/9f4d9a0b7b75210b284816e88362f48e7f64aff8
  Author: David Sherwood <david.sherwood at arm.com>
  Date:   2022-08-09 (Tue, 09 Aug 2022)

  Changed paths:
    M llvm/lib/Target/AArch64/AArch64ISelLowering.cpp
    M llvm/test/CodeGen/AArch64/bitcast-promote-widen.ll
    M llvm/test/CodeGen/AArch64/sve-fixed-length-extract-subvector.ll
    M llvm/test/CodeGen/AArch64/sve-fixed-length-masked-gather.ll
    M llvm/test/CodeGen/AArch64/sve-fixed-length-masked-loads.ll
    M llvm/test/CodeGen/AArch64/sve-fixed-length-masked-scatter.ll
    M llvm/test/CodeGen/AArch64/sve-fixed-length-masked-stores.ll
    M llvm/test/CodeGen/AArch64/vector-fcvt.ll

  Log Message:
  -----------
  [AArch64][DAGCombine] Add performBuildVectorCombine 'extract_elt ~> anyext'

A build vector of two extracted elements is equivalent to an extract
subvector where the inner vector is any-extended to the
extract_vector_elt VT, because extract_vector_elt has the effect of an
any-extend.

  (build_vector (extract_elt_i16_to_i32 vec Idx+0) (extract_elt_i16_to_i32 vec Idx+1))
  => (extract_subvector (anyext_i16_to_i32 vec) Idx)

Depends on D130697

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

(cherry picked from commit 487fa6f8c3af87232f7ff9484568be7782f7f8b2)


  Commit: 65d2e9b4e123d9ea472f0f37a5ca64d9ca01bde1
      https://github.com/llvm/llvm-project/commit/65d2e9b4e123d9ea472f0f37a5ca64d9ca01bde1
  Author: David Sherwood <david.sherwood at arm.com>
  Date:   2022-08-09 (Tue, 09 Aug 2022)

  Changed paths:
    M llvm/include/llvm/CodeGen/ISDOpcodes.h
    M llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
    A llvm/test/CodeGen/AArch64/extract-subvec-combine.ll
    M llvm/test/CodeGen/AArch64/vector-fcvt.ll

  Log Message:
  -----------
  [DAGCombiner] Extend visitAND to include EXTRACT_SUBVECTOR

Eliminate an AND by redefining an anyext|sext|zext.

     (and (extract_subvector (anyext|sext|zext v) _) iN_mask)
  => (extract_subvector (zeroext_iN v))

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

(cherry picked from commit 41119a0f520d07a438f7434f17ce63cc3a942d1b)


  Commit: 63be0bcb6d302910e11df2c305ed707ca6ee7421
      https://github.com/llvm/llvm-project/commit/63be0bcb6d302910e11df2c305ed707ca6ee7421
  Author: Jonathan Peyton <jonathan.l.peyton at intel.com>
  Date:   2022-08-09 (Tue, 09 Aug 2022)

  Changed paths:
    M openmp/cmake/DetectTestCompiler/CMakeLists.txt
    M openmp/cmake/OpenMPTesting.cmake
    M openmp/runtime/test/CMakeLists.txt
    M openmp/runtime/test/lit.cfg
    M openmp/runtime/test/lit.site.cfg.in

  Log Message:
  -----------
  [OpenMP][libomp] Detect if test compiler has omp.h

omp50_taskdep_depobj.c relies on the test compiler's omp.h file.
If the test compiler does not have an omp.h file, then use the one
within the build tree.

Fixes: https://github.com/llvm/llvm-project/issues/56820
Differential Revision: https://reviews.llvm.org/D131000

(cherry picked from commit 9cf6511bff97007401238f6cff6bf80cb9af04a5)


Compare: https://github.com/llvm/llvm-project/compare/0fd385c1b09c...63be0bcb6d30