[all-commits] [llvm/llvm-project] 53d8c6: [flang][Semantics][OpenMP] set intrinsic attr for ...

[Tom Eccles via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 53d8c6b1b1f7cfce9bd42032e8cb6cc1ddcf6c78
      https://github.com/llvm/llvm-project/commit/53d8c6b1b1f7cfce9bd42032e8cb6cc1ddcf6c78
  Author: Tom Eccles <tom.eccles at arm.com>
  Date:   2024-03-15 (Fri, 15 Mar 2024)

  Changed paths:
    M flang/lib/Semantics/resolve-directives.cpp
    A flang/test/Lower/OpenMP/parallel-reduction-rename.f90
    M flang/test/Lower/OpenMP/wsloop-reduction-min2.f90
    A flang/test/Semantics/OpenMP/reduction11.f90

  Log Message:
  -----------
  [flang][Semantics][OpenMP] set intrinsic attr for reductions (#85114)

Reductions such as min are intrinsic procedures. This distinguishes them
from user defined reductions. Previously, the intrinsic attribute was
not set when visiting reduction clauses causing them to be missed.

wsloop-reduction-min.f90 (the other min reduction test) worked because
it contained "min" used as an intrinsic inside of the body of the
reduction. This allowed ResolveNamesVisitor::HandleProcedureName to set
the correct attribute on that Symbol.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications