[PATCH] D93560: [openacc][openmp] Reduce number of generated file and prefer inclusion of .inc

Valentin Clement via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 18 12:27:41 PST 2020


clementval created this revision.
clementval added reviewers: jdoerfert, kiranchandramohan, thakis.
Herald added subscribers: guansong, hiraditya, yaxunl, mgorny.
Herald added a reviewer: sscalpone.
clementval requested review of this revision.
Herald added subscribers: llvm-commits, sstefan1.
Herald added a project: LLVM.

Follow up from D92955 <https://reviews.llvm.org/D92955> and D83636 <https://reviews.llvm.org/D83636>. This patch makes the base cpp files
OMP.cpp and ACC.cpp normal files and they now include the XXX.inc file 
generated by tablegen. This reduces the number of file generated by the 
DirectiveEmitter backend and makes it closer to the proposal in D83636 <https://reviews.llvm.org/D83636>.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D93560

Files:
  llvm/include/llvm/Frontend/OpenACC/ACC.td
  llvm/include/llvm/Frontend/OpenMP/OMP.td
  llvm/include/llvm/TableGen/DirectiveEmitter.h
  llvm/lib/Frontend/OpenACC/ACC.cpp
  llvm/lib/Frontend/OpenACC/CMakeLists.txt
  llvm/lib/Frontend/OpenMP/CMakeLists.txt
  llvm/lib/Frontend/OpenMP/OMP.cpp
  llvm/test/TableGen/directive1.td
  llvm/test/TableGen/directive2.td
  llvm/test/TableGen/directive3.td
  llvm/utils/TableGen/DirectiveEmitter.cpp
  llvm/utils/TableGen/TableGen.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D93560.312874.patch
Type: text/x-patch
Size: 20258 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20201218/997667d4/attachment.bin>


More information about the llvm-commits mailing list