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

Michael Kruse via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 16 14:30:47 PDT 2021


Meinersbur added a comment.

OK, I found the header include is generated in CMakeLists.txt in include/llvm/Frontend. This patch removes the "-gen-directive-impl" target and integrates it into "-gen-directive-gen". IMHO "-gen-directive-impl" is the better name for for the implementation file, following the scheme set by "AttrImpl.inc" and "IntrinsicImpl.inc".

Sorry for me not noticing that there have been 3 tablegen targets per td.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D93560/new/

https://reviews.llvm.org/D93560



More information about the llvm-commits mailing list