This is an archive of the discontinued LLVM Phabricator instance.

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

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

Diff Detail

Event Timeline

dcoughlin accepted this revision.Sep 21 2017, 3:01 PM

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.