[all-commits] [llvm/llvm-project] ac9ee6: [acc] Improve LegalizeDataValues pass to handle da...

[Razvan Lupusoru via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: ac9ee618572537bcd77c58899aaab1d41dbad206
      https://github.com/llvm/llvm-project/commit/ac9ee618572537bcd77c58899aaab1d41dbad206
  Author: Razvan Lupusoru <razvan.lupusoru at gmail.com>
  Date:   2024-10-21 (Mon, 21 Oct 2024)

  Changed paths:
    M flang/test/Fir/OpenACC/legalize-data.fir
    M mlir/include/mlir/Dialect/OpenACC/OpenACC.h
    M mlir/include/mlir/Dialect/OpenACC/Transforms/Passes.h
    M mlir/include/mlir/Dialect/OpenACC/Transforms/Passes.td
    M mlir/lib/Dialect/OpenACC/Transforms/CMakeLists.txt
    R mlir/lib/Dialect/OpenACC/Transforms/LegalizeData.cpp
    A mlir/lib/Dialect/OpenACC/Transforms/LegalizeDataValues.cpp
    M mlir/test/Dialect/OpenACC/legalize-data.mlir

  Log Message:
  -----------
  [acc] Improve LegalizeDataValues pass to handle data constructs (#112990)

Renames LegalizeData to LegalizeDataValues since this pass fixes up SSA
values. LegalizeData suggested that it fixed data mapping.

This change also adds support to fix up ssa values for data clause
operations. Effectively, compute regions within a data region use the
ssa values from data operations also. The ssa values within data regions
but not within compute regions are not updated.

This change is to support the requirement in the OpenACC spec which
notes that a visible data clause is not just one on the current compute
construct but on the lexically containing data construct or visible
declare directive.



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