Hi! Is the attached patch for "libgomp: Allow overriding via 'GOMP_OFFLOAD_PLUGINS' the configured set of libgomp plugins" OK for master and release branches? If approving this patch, please respond with "Reviewed-by: NAME <EMAIL>" so that your effort will be recorded in the commit log, see <https://gcc.gnu.org/wiki/Reviewed-by>.
Grüße Thomas ----------------- Mentor Graphics (Deutschland) GmbH, Arnulfstraße 201, 80634 München / Germany Registergericht München HRB 106955, Geschäftsführer: Thomas Heurung, Alexander Walter
>From 13f04e776c803766140b0b657dad15029e2b65be Mon Sep 17 00:00:00 2001 From: Thomas Schwinge <tho...@codesourcery.com> Date: Mon, 27 Apr 2020 14:01:50 +0200 Subject: [PATCH] libgomp: Allow overriding via 'GOMP_OFFLOAD_PLUGINS' the configured set of libgomp plugins This can be useful for testing the same libgomp build on several machines, with different offload capabilities/configurations. This is subject to change, and thus intentionally not documented. libgomp/ * target.c [PLUGIN_SUPPORT] (gomp_target_init): Allow overriding via 'GOMP_OFFLOAD_PLUGINS' the configured set of libgomp plugins. --- libgomp/target.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/libgomp/target.c b/libgomp/target.c index 36425477dcb..d1de7fab87c 100644 --- a/libgomp/target.c +++ b/libgomp/target.c @@ -25,6 +25,7 @@ /* This file contains the support of offloading. */ +#define _GNU_SOURCE #include "libgomp.h" #include "oacc-plugin.h" #include "oacc-int.h" @@ -42,6 +43,7 @@ #ifdef PLUGIN_SUPPORT #include <dlfcn.h> #include "plugin-suffix.h" +#include "secure_getenv.h" #endif #define FIELD_TGT_EMPTY (~(size_t) 0) @@ -3163,7 +3165,10 @@ gomp_target_init (void) num_devices = 0; devices = NULL; - cur = OFFLOAD_PLUGINS; + /* This is subject to change, and thus intentionally not documented. */ + cur = secure_getenv ("GOMP_OFFLOAD_PLUGINS"); + if (!cur) + cur = OFFLOAD_PLUGINS; if (*cur) do { -- 2.17.1