[all-commits] [llvm/llvm-project] dd32bf: [Flang, MLIR, OpenMP] Fix a few tests that were not ...

kiranchandramohan via All-commits all-commits at lists.llvm.org
Tue Jun 7 02:56:30 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: dd32bf9a7792dd3fd1ce3abe112ef6360ac04270
      https://github.com/llvm/llvm-project/commit/dd32bf9a7792dd3fd1ce3abe112ef6360ac04270
  Author: Kiran Chandramohan <kiran.chandramohan at arm.com>
  Date:   2022-06-07 (Tue, 07 Jun 2022)

  Changed paths:
    M flang/test/Lower/OpenMP/flush.f90
    M flang/test/Lower/OpenMP/parallel-sections.f90
    M flang/test/Lower/OpenMP/parallel.f90
    M flang/test/Lower/OpenMP/single.f90
    M mlir/include/mlir/Dialect/OpenMP/OpenMPOps.td
    M mlir/lib/Conversion/OpenMPToLLVM/OpenMPToLLVM.cpp
    M mlir/test/Dialect/OpenMP/ops.mlir
    M mlir/test/Target/LLVMIR/openmp-llvm.mlir

  Log Message:
  -----------
  [Flang,MLIR,OpenMP] Fix a few tests that were not converting to LLVM

A few OpenMP tests were retaining the FIR operands even after running
the LLVM conversion pass. To fix these tests the legality checkes for
OpenMP conversion are made stricter to include operands and results.
The Flush, Single and Sections operations are added to conversions or
legality checks. The RegionLessOpConversion is appropriately renamed
to clarify that it works only for operations with Variable operands.
The operands of the flush operation are changed to match those of
Variable Operands.

Fix for an OpenMP issue mentioned in
https://github.com/llvm/llvm-project/issues/55210.

Reviewed By: shraiysh, peixin, awarzynski

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




More information about the All-commits mailing list