intel/llvm
1
0
Fork 0
mirror of https://github.com/intel/llvm.git synced 2026-01-16 05:32:28 +08:00
Code Issues Packages Projects Releases Wiki Activity

[analyzer] SATestBuild.py: Move additional checkers logic so SATestAdd.py can use it as well.

Browse Source 
Move the logic looking for additional checkers in the SA_ADDITIONAL_CHECKERS
environmental variable from SATestBuild's main() to runScanBuild(). This allows
SATestAdd.py to use the variable as well. Without it, we won't include
additional checkers when building reference results for the build bot.

Differential Revision: http://reviews.llvm.org/D12891

llvm-svn: 247767
This commit is contained in:
Devin Coughlin
2015-09-16 01:52:32 +00:00
parent 3ca7a3eb41
commit f3695c87cd
1 changed files with 6 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
clang/utils/analyzer/SATestBuild.py
View File

@@ -210,9 +210,14 @@ def runScanBuild(Dir, SBOutputDir, PBuildLogFile):
if not os.path.exists(BuildScriptPath):
print "Error: build script is not defined: %s" % BuildScriptPath
sys.exit(-1)
AllCheckers = Checkers
if os.environ.has_key('SA_ADDITIONAL_CHECKERS'):
AllCheckers = AllCheckers + ',' + os.environ['SA_ADDITIONAL_CHECKERS']
SBOptions = "--use-analyzer " + Clang + " "
SBOptions += "-plist-html -o " + SBOutputDir + " "
SBOptions += "-enable-checker " + Checkers + " "
SBOptions += "-enable-checker " + AllCheckers + " "
SBOptions += "--keep-empty "
# Always use ccc-analyze to ensure that we can locate the failures
# directory.
@@ -614,7 +619,4 @@ if __name__ == '__main__':
IsReference = True
UpdateSVN = True
if os.environ.has_key('SA_ADDITIONAL_CHECKERS'):
Checkers = Checkers + ',' + os.environ['SA_ADDITIONAL_CHECKERS']
testAll(IsReference, UpdateSVN, Strictness)