[all-commits] [llvm/llvm-project] 76bfbb: [libomptarget][amdgpu] Call into deviceRTL instead...

Simon Pilgrim via All-commits all-commits at lists.llvm.org
Mon Jan 4 08:54:30 PST 2021


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 76bfbb74d38b611f150e8e1a4becc11be95703da
      https://github.com/llvm/llvm-project/commit/76bfbb74d38b611f150e8e1a4becc11be95703da
  Author: Jon Chesterfield <jonathanchesterfield at gmail.com>
  Date:   2021-01-04 (Mon, 04 Jan 2021)

  Changed paths:
    M clang/lib/CodeGen/CGOpenMPRuntimeAMDGCN.cpp
    M clang/test/OpenMP/amdgcn_target_codegen.cpp
    M openmp/libomptarget/deviceRTLs/amdgcn/src/amdgcn_interface.h
    M openmp/libomptarget/deviceRTLs/amdgcn/src/target_impl.hip

  Log Message:
  -----------
  [libomptarget][amdgpu] Call into deviceRTL instead of ockl

[libomptarget][amdgpu] Call into deviceRTL instead of ockl

Amdgpu codegen presently emits a call into ockl. The same functionality
is already present in the deviceRTL. Adds an amdgpu specific entry point
to avoid the dependency. This lets simple openmp code (specifically, that
which doesn't use libm) run without rocm device libraries installed.

Reviewed By: ronlieb

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


  Commit: f7463ca3cc5ba8455c4611c5afa79c48d8a79326
      https://github.com/llvm/llvm-project/commit/f7463ca3cc5ba8455c4611c5afa79c48d8a79326
  Author: Simon Pilgrim <llvm-dev at redking.me.uk>
  Date:   2021-01-04 (Mon, 04 Jan 2021)

  Changed paths:
    M llvm/lib/ProfileData/GCOV.cpp

  Log Message:
  -----------
  [ProfileData] GCOVFile::readGCNO - silence undefined pointer warning. NFCI.

Silence clang static analyzer warning that 'fn' could still be in an undefined state - this shouldn't happen depending on the likely tag order, but the analyzer can't know that.


Compare: https://github.com/llvm/llvm-project/compare/c55b609b777d...f7463ca3cc5b


More information about the All-commits mailing list