summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJordan Rose <jordan_rose@apple.com>2013-06-10 19:34:30 +0000
committerJordan Rose <jordan_rose@apple.com>2013-06-10 19:34:30 +0000
commitc7b992edcfb0ccfe331aa0543da1dd3c9817685c (patch)
tree6b2521c3c4ed43386c3a3fa5c33c734df0dd79f9
parentbb4b894e0b2d0c39d40a178872127a5cd6becd8e (diff)
downloadbcm5719-llvm-c7b992edcfb0ccfe331aa0543da1dd3c9817685c.tar.gz
bcm5719-llvm-c7b992edcfb0ccfe331aa0543da1dd3c9817685c.zip
[analyzer] SATestBuild: Don't require reference results to have logs.
The Logs directory isn't used for testing, so it's filtered out ahead of time. However, there's then no reason to include it in version control at all. Don't error if it's not present. llvm-svn: 183689
-rwxr-xr-xclang/utils/analyzer/SATestBuild.py6
1 files changed, 4 insertions, 2 deletions
diff --git a/clang/utils/analyzer/SATestBuild.py b/clang/utils/analyzer/SATestBuild.py
index 8e09a7adab4..eb7796431a1 100755
--- a/clang/utils/analyzer/SATestBuild.py
+++ b/clang/utils/analyzer/SATestBuild.py
@@ -413,8 +413,10 @@ def runCmpResults(Dir):
RefList = glob.glob(RefDir + "/*")
NewList = glob.glob(NewDir + "/*")
- # Log folders are also located in the results dir, so ignore them.
- RefList.remove(os.path.join(RefDir, LogFolderName))
+ # Log folders are also located in the results dir, so ignore them.
+ RefLogDir = os.path.join(RefDir, LogFolderName)
+ if RefLogDir in RefList:
+ RefList.remove(RefLogDir)
NewList.remove(os.path.join(NewDir, LogFolderName))
if len(RefList) == 0 or len(NewList) == 0:
OpenPOWER on IntegriCloud