From 11e02d5c24f9badc0ff34afc9538774f67cc8c50 Mon Sep 17 00:00:00 2001 From: Julian Lettner Date: Fri, 1 May 2020 09:51:46 -0700 Subject: [PATCH] [lit] Only update specific fields from remote test object Don't update whole test object from the remote (pickled) finished test object. Doing so also changes the config and suite members, which we want to avoid. --- llvm/utils/lit/lit/run.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/llvm/utils/lit/lit/run.py b/llvm/utils/lit/lit/run.py index d0e7e65..5aef77e 100644 --- a/llvm/utils/lit/lit/run.py +++ b/llvm/utils/lit/lit/run.py @@ -99,9 +99,11 @@ class Run(object): # Update local test object "in place" from remote test object. This # ensures that the original test object which is used for printing test - # results reflect the changes. + # results reflects the changes. def _update_test(self, local_test, remote_test): - local_test.__dict__.update(remote_test.__dict__) + # Needed for getMissingRequiredFeatures() + local_test.requires = remote_test.requires + local_test.result = remote_test.result # TODO(yln): interferes with progress bar # Some tests use threads internally, and at least on Linux each of these -- 2.7.4