LGTM

================
Comment at: test/libcxx/test/executor.py:128
@@ -122,1 +127,3 @@
+                             for f in file_deps]
+                self.copy_in(file_deps, dev_paths)
             return self._execute_command_remote(cmd, target_cwd, env)
----------------
sorry, what I meant was:

```
to_copy = ([exe_path], [target_exe])
if file_deps is not None:
    dev_paths = [os.path.join(target_cwd, os.path.basename(f))
                 for f in file_deps]
    to_copy[0] += file_deps
    to_copy[1] += dev_paths
self.copy_in(to_copy[0], to_copy[1])
```

http://reviews.llvm.org/D8118

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



_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to