[Openmp-commits] [PATCH] D96379: [OpenMP] Unify omptarget API and usage wrt. `__tgt_async_info`

Johannes Doerfert via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Tue Feb 9 22:36:40 PST 2021


jdoerfert added a comment.

I'll split the patch tomorrow and do some NFC pre commits without review to unify some style choices.



================
Comment at: openmp/libomptarget/src/private.h:16
 
+#include "rtl.h"
 #include <Debug.h>
----------------
tianshilei1992 wrote:
> Why do we need this include?
clangd automatically added it. sometimes that is a problem when I used something from the header but later removed the use. I'll get rid of the include if I can.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96379



More information about the Openmp-commits mailing list