[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