Re: [PATCH] PR58867 ASan and UBSan tests not run for installed testing.

[Maxim Ostapenko <m dot ostapenko at partner dot samsung dot com>]

Hm, as I see, others testsuites such as gfortran.exp, go.exp etc. do not 
call restore_ld_library_path at all. Perhaps we could simply follow this 
way?

Would failing tests still fail if remove restore_ld_library_path from 
{asan, tsan, ubsan}_finish?

On 10/08/2014 03:40 PM, Marcus Shawcroft wrote:
> On 8 October 2014 11:10, Maxim Ostapenko
> <m.ostapenko@partner.samsung.com> wrote:
> > Does it work without restore_ld_library_path in {asan, ubsan, tsan}_finish?
> >
> > I see two opportunities to fix the issue:
> >
> > 1) Implement a stack of saved contexts.
> >
> > 2) Implement new functions, say {asan, ubsan, tsan}_restore_ld_library_path,
> > to be able {asan, ubsan, tsan}_finish functions restore context correctly.
> >
> > What solution is preferable?
> Option 1 has the advantage that it places all of the context save and
> restore in one place rather than spreading it around the
> infrastructure.
>
> Please can we revert this patch while a correct implementation is being worked?
>
> Cheers
> /Marcus

diff --git a/gcc/testsuite/lib/asan-dg.exp b/gcc/testsuite/lib/asan-dg.exp
index 9769138..c98fd3c 100644
--- a/gcc/testsuite/lib/asan-dg.exp
+++ b/gcc/testsuite/lib/asan-dg.exp
@@ -132,7 +132,6 @@ proc asan_finish { args } {
 	    unset TEST_ALWAYS_FLAGS
 	}
     }
-    restore_ld_library_path_env_vars
 }
 
 # Symbolize lines like
diff --git a/gcc/testsuite/lib/tsan-dg.exp b/gcc/testsuite/lib/tsan-dg.exp
index 54ec404..6f7a4d9 100644
--- a/gcc/testsuite/lib/tsan-dg.exp
+++ b/gcc/testsuite/lib/tsan-dg.exp
@@ -143,5 +143,4 @@ proc tsan_finish { args } {
     } else {
 	unset dg-do-what-default
     }
-    restore_ld_library_path_env_vars
 }
diff --git a/gcc/testsuite/lib/ubsan-dg.exp b/gcc/testsuite/lib/ubsan-dg.exp
index 5a7a653..87c460f 100644
--- a/gcc/testsuite/lib/ubsan-dg.exp
+++ b/gcc/testsuite/lib/ubsan-dg.exp
@@ -114,5 +114,4 @@ proc ubsan_finish { args } {
 	    unset TEST_ALWAYS_FLAGS
 	}
     }
-    restore_ld_library_path_env_vars
 }