[all-commits] [llvm/llvm-project] 73c36a: [OpenMP][FIX] Allow declare variant to work with r...

Johannes Doerfert via All-commits all-commits at lists.llvm.org
Wed Sep 1 17:39:36 PDT 2021


  Branch: refs/heads/release/13.x
  Home:   https://github.com/llvm/llvm-project
  Commit: 73c36a9be5584a9185c7ca922a50c58603e4a35e
      https://github.com/llvm/llvm-project/commit/73c36a9be5584a9185c7ca922a50c58603e4a35e
  Author: Johannes Doerfert <johannes at jdoerfert.de>
  Date:   2021-09-01 (Wed, 01 Sep 2021)

  Changed paths:
    M clang/lib/AST/ASTContext.cpp
    A clang/test/AST/ast-dump-openmp-begin-declare-variant_reference.cpp

  Log Message:
  -----------
  [OpenMP][FIX] Allow declare variant to work with reference types

Reference types in the return or parameter position did cause the OpenMP
declare variant overload reasoning to give up. We should allow them as
we allow any other type.

This should fix the bug reported on the mailing list:
https://lists.llvm.org/pipermail/openmp-dev/2021-August/004094.html

Reviewed By: ABataev, pdhaliwal

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

(cherry picked from commit 2930c839a5877356db6966409f4f9f2cdbcf3a07)




More information about the All-commits mailing list