[PATCH] D83636: omp: Make OMP tablegen more like all other tablegens.

[Valentin Clement via Phabricator via llvm-commits]

clementval added a comment.

@thakis Sorry this has been lying here for a while. I see the point of having this mixed "mode" where more complicated constructs are code gen inside the DirectiveEmitter and there is a way to do the simplest ones using macros and includes. Unfortunately, I think this patch is now behind the current version and OpenACC joined the frontend directory recently as it is under construction in Flang. If you think this is still valuable, I can offer to work on a new patch that cover both OpenMP and OpenACC and follow what you were trying to do here.

@jdoerfert Any comments?


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

https://reviews.llvm.org/D83636