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

[Jon Chesterfield via Phabricator via Openmp-commits]

JonChesterfield 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",
----------------
simoll wrote:
> 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.
Oh, sure. Before or after this lands is of course fine.


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

https://reviews.llvm.org/D76843