From f770fa158ca12451d35c05f1f36c00dbfd1544a7 Mon Sep 17 00:00:00 2001
From: Axel Naumann <Axel.Naumann@cern.ch>
Date: Thu, 29 Oct 2020 17:40:10 +0100
Subject: [PATCH] [test] Remove ineffective re-load guard in stressInterpreter:

even when interpreter, the JIT will take the stressReflection symbol
from the binary, where `#if !defined(__CLING__)` does not protect from
re-inclusion (and thus unloading) of stressInterpreter.cxx in the
interpreter. Make that more obvious by removing the ineffective
do-not-reinclude guards.
---
 test/stressInterpreter.cxx | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/test/stressInterpreter.cxx b/test/stressInterpreter.cxx
index cbc7b825a2a..7323b26b3d0 100644
--- a/test/stressInterpreter.cxx
+++ b/test/stressInterpreter.cxx
@@ -272,11 +272,10 @@ bool InterpreterStress::stressReflection() {
    // This is fast
    int ntimes = fNtimes * 800;
 
-#if !defined(__CINT__) && !defined(__CLING__)
    TString macro(fBinary);
    macro += ".cxx";
    gInterpreter->LoadMacro(macro);
-#endif
+
    int numfuncs = Klass::last_klf - Klass::first_klf + 1;
    bool success = true;
    for (Int_t i = 0; success && i < ntimes; ++i) {
-- 
GitLab