[Openmp-commits] [PATCH] D76843: [Openmp] Libomptarget plugin for NEC SX-Aurora

[Simon Moll via Phabricator via Openmp-commits]

simoll added inline comments.


================
Comment at: openmp/libomptarget/src/rtl.cpp:25
 // List of all plugins that can support offloading.
 static const char *RTLNames[] = {
+    /* SX-Aurora VE target  */ "libomptarget.rtl.ve.so",
----------------
JonChesterfield wrote:
> These could be sorted lexicographically, for the sake of removing uncertainty about where in the list to insert a new entry.
I think sorting this list should happen in a separate patch after this one has been committed.


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

https://reviews.llvm.org/D76843