[all-commits] [llvm/llvm-project] 079884: [OPENMP]Fix PR49698: OpenMP declare mapper causes ...

cilkplus via All-commits all-commits at lists.llvm.org
Wed Apr 21 10:39:39 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 079884225a5571f4caf1a9c71b6748db8192e383
      https://github.com/llvm/llvm-project/commit/079884225a5571f4caf1a9c71b6748db8192e383
  Author: Alexey Bataev <a.bataev at outlook.com>
  Date:   2021-04-21 (Wed, 21 Apr 2021)

  Changed paths:
    M clang/lib/CodeGen/CGOpenMPRuntime.cpp
    M clang/test/OpenMP/declare_mapper_codegen.cpp
    M openmp/libomptarget/src/omptarget.cpp
    A openmp/libomptarget/test/mapping/declare_mapper_nested_default_mappers_array.cpp
    A openmp/libomptarget/test/mapping/declare_mapper_nested_default_mappers_array_subscript.cpp
    A openmp/libomptarget/test/mapping/declare_mapper_nested_default_mappers_complex_structure.cpp
    A openmp/libomptarget/test/mapping/declare_mapper_nested_default_mappers_ptr_subscript.cpp
    A openmp/libomptarget/test/mapping/declare_mapper_nested_default_mappers_var.cpp

  Log Message:
  -----------
  [OPENMP]Fix PR49698: OpenMP declare mapper causes segmentation fault.

The implicitly generated mappings for allocation/deallocation in mappers
runtime should be mapped as implicit, also no need to clear member_of
flag to avoid ref counter increment. Also, the ref counter should not be
incremented for the very first element that comes from the mapper
function.

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




More information about the All-commits mailing list