[Openmp-commits] [PATCH] D96428: [OpenMP][NFC] Clang format libomptarget code (src & include)

Jon Chesterfield via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Wed Feb 10 09:50:30 PST 2021


JonChesterfield accepted this revision.
JonChesterfield added inline comments.


================
Comment at: openmp/libomptarget/include/omptarget.h:27
+
+// Don't format out enums and structs.
+// clang-format off
----------------
Probably don't need the extra comment, clang-format off is descriptive.


================
Comment at: openmp/libomptarget/src/rtl.cpp:13
 
+#include "rtl.h"
 #include "device.h"
----------------
Hopefully header order is not significant. I think there was something where one header required a definition of a macro before it was included, but that may have been fixed in the changes to debug print.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96428



More information about the Openmp-commits mailing list