[PATCH] D120825: [mlir][OpenMP] Generating enums in accordance with the guidelines

Shraiysh via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 2 08:44:55 PST 2022


shraiysh created this revision.
Herald added subscribers: awarzynski, sdasgup3, wenzhicui, wrengr, Chia-hungDuan, dcaballe, cota, teijeong, rdzhabarov, tatianashp, mehdi_amini, msifontes, jurahul, Kayjukh, grosul1, Joonsoo, liufengdb, aartbik, mgester, arpith-jacob, antiagainst, shauheen, rriddle, guansong, yaxunl.
Herald added a reviewer: ftynse.
Herald added projects: Flang, All.
shraiysh requested review of this revision.
Herald added a reviewer: jdoerfert.
Herald added subscribers: llvm-commits, sstefan1, stephenneuendorffer, nicolasvasilache, jdoerfert.
Herald added projects: MLIR, LLVM.

This patch changes the enums generated from `OMP.td` for MLIR according
to the enum naming guidelines in LLVM Coding Standards.

This also helps the issues we had with `static` being a C++ keyword and
also a value for the schedule clause.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D120825

Files:
  flang/lib/Lower/OpenMP.cpp
  llvm/include/llvm/Frontend/OpenMP/OMP.td
  llvm/unittests/Frontend/OpenMPParsingTest.cpp
  mlir/lib/Dialect/OpenMP/IR/OpenMPDialect.cpp
  mlir/lib/Target/LLVMIR/Dialect/OpenMP/OpenMPToLLVMIRTranslation.cpp
  mlir/test/Dialect/OpenMP/ops.mlir
  mlir/test/mlir-tblgen/directive-common.td
  mlir/tools/mlir-tblgen/DirectiveCommonGen.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D120825.412443.patch
Type: text/x-patch
Size: 10735 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220302/8db9dc60/attachment.bin>


More information about the llvm-commits mailing list