[all-commits] [llvm/llvm-project] 438863: [OpenACC][Sema] Implement warning for 'cache' inva...

[Erich Keane via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 438863a09e2d907c36c4114228ef1eeee130e16b
      https://github.com/llvm/llvm-project/commit/438863a09e2d907c36c4114228ef1eeee130e16b
  Author: erichkeane <ekeane at nvidia.com>
  Date:   2025-07-03 (Thu, 03 Jul 2025)

  Changed paths:
    M clang/include/clang/Basic/DiagnosticSemaKinds.td
    M clang/include/clang/Sema/Scope.h
    M clang/include/clang/Sema/SemaOpenACC.h
    M clang/lib/Parse/ParseOpenACC.cpp
    M clang/lib/Sema/Scope.cpp
    M clang/lib/Sema/SemaExpr.cpp
    M clang/lib/Sema/SemaOpenACC.cpp
    M clang/test/AST/ast-print-openacc-cache-construct.cpp
    M clang/test/ParserOpenACC/parse-cache-construct.c
    M clang/test/ParserOpenACC/parse-cache-construct.cpp
    M clang/test/SemaOpenACC/cache-construct-ast.cpp
    M clang/test/SemaOpenACC/cache-construct.cpp
    A clang/test/SemaOpenACC/cache-warn-invalid-varloc.cpp

  Log Message:
  -----------
  [OpenACC][Sema] Implement warning for 'cache' invalid var ref

The 'cache' construct is lowered as marking the acc.loop in ACC MLIR.
This results in any variable references that are not inside of the
acc.loop being invalid.  This patch adds a warning to that effect, and
ensures that the variable references won't be added to the AST during
parsing so we don't try to lower them.

This results in loss of instantiation-diagnostics for these, however
that seems like an acceptable consequence to ignoring it.



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