This is an archive of the discontinued LLVM Phabricator instance.

Paths

[Analyzer] Use CC environment variable to select analyzer path in SATestBuild
ClosedPublic

Authored by george.karpenkov on Sep 21 2017, 2:53 PM.

Reviewers

dcoughlin
zaks.anna

Commits

Seems reasonable. Can you include a comment in the commit message about why this is needed?

Also, why did you choose an environment variable and not a flag?

This revision is now accepted and ready to land.Sep 21 2017, 3:01 PM

@dcoughlin: yes. Either would work, I thought that CC=mycompiler ./blah was already quite idiomatic.

This revision was automatically updated to reflect the committed changes.

Path

Size

cfe/

trunk/

utils/

analyzer/

5 lines

Show First 20 Lines • Show All 136 Lines • ▼ Show 20 Lines	def getSBOutputDirName(IsReferenceBuild) :
else :		else :
return SBOutputDirName		return SBOutputDirName

#------------------------------------------------------------------------------		#------------------------------------------------------------------------------
# Configuration setup.		# Configuration setup.
#------------------------------------------------------------------------------		#------------------------------------------------------------------------------

# Find Clang for static analysis.		# Find Clang for static analysis.
		if 'CC' in os.environ:
		Clang = os.environ['CC']
		else:
Clang = which("clang", os.environ['PATH'])		Clang = which("clang", os.environ['PATH'])
if not Clang:		if not Clang:
print "Error: cannot find 'clang' in PATH"		print "Error: cannot find 'clang' in PATH"
sys.exit(-1)		sys.exit(-1)

# Number of jobs.		# Number of jobs.
Jobs = int(math.ceil(detectCPUs() * 0.75))		Jobs = int(math.ceil(detectCPUs() * 0.75))

# Project map stores info about all the "registered" projects.		# Project map stores info about all the "registered" projects.
▲ Show 20 Lines • Show All 574 Lines • Show Last 20 Lines