On Thu, 12 Feb 2026, Christophe Lyon wrote:
> > @@ -134,6 +148,14 @@ proc tsan_finish { args } {
> > global tsan_saved_dg-do-what-default
> > global tsan_saved_library_path
> > global ld_library_path
> > + global orig_tsan_options_saved
> > + global orig_tsan_options
> > +
> > + if { $orig_tsan_options_saved } {
> > + setenv TSAN_OPTIONS "$orig_tsan_options"
> > + } elseif [info exists env(TSAN_OPTIONS)] {
> > + unsetenv TSAN_OPTIONS
> > + }
>
> Is it OK that we don't reset $orig_tsan_options_saved to 0?
> (I noticed that ubsan_finish does not do it either)
I'm just following what's done for UBSAN_OPTIONS.
Probably we (or DejaGnu) could do with some more generic support for tests
to save and restore environment variables rather than having a lot of
separate slightly different copied and pasted variants.
--
Joseph S. Myers
[email protected]
