[PATCH] [libcxx] Cleanup the cleanup of temporary files.

Eric Fiselier eric at efcs.ca
Mon Dec 22 11:27:39 PST 2014


================
Comment at: test/lit.cfg:135
@@ +134,3 @@
+        res = self._evaluate_test(test, use_verify, lit_config)
+        self._clean()
+        return res
----------------
mclow.lists wrote:
> Can `self._evaluate_test` throw?
> 
> If so, you'll skip the call to `_clean`.
> On the other hand, `_clean` appears to be empty now.
I would like @danalbert to comment on this. I think he uses _clean() in the android configuration.

http://reviews.llvm.org/D6739

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the cfe-commits mailing list