[all-commits] [llvm/llvm-project] 0a2b6f: [OpenACC] Fix 'copyout' allowed modifiers alwaysin...

[Erich Keane via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 0a2b6f6c1cc796bcac38d1c6b44d76de6049ea12
      https://github.com/llvm/llvm-project/commit/0a2b6f6c1cc796bcac38d1c6b44d76de6049ea12
  Author: erichkeane <ekeane at nvidia.com>
  Date:   2025-06-26 (Thu, 26 Jun 2025)

  Changed paths:
    M clang/lib/Sema/SemaOpenACCClause.cpp
    M clang/test/AST/ast-print-openacc-combined-construct.cpp
    M clang/test/SemaOpenACC/combined-construct-copyout-ast.cpp
    M clang/test/SemaOpenACC/combined-construct-copyout-clause.c
    M clang/test/SemaOpenACC/compute-construct-copyout-clause.c
    M clang/test/SemaOpenACC/compute-construct-varlist-ast.cpp
    M clang/test/SemaOpenACC/data-construct-copyout-ast.cpp
    M clang/test/SemaOpenACC/data-construct-copyout-clause.c
    M clang/test/SemaOpenACC/declare-construct.cpp

  Log Message:
  -----------
  [OpenACC] Fix 'copyout' allowed modifiers alwaysin vs alwaysout

While doing lowering, I discovered that the restriction onthe allowed
modifiers for 'copyout' didn't make sense! After discussion on the
OpenACC standards mailing list I discovered that this was a copy/paste
error during standardization that they intend to fix, and really meant
for copyout to allow alwaysout instead of alwaysin.

When implementing, I blindly followed the standard :)

This patch corrects the implementation to do what was meant.



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