[cfe-commits] r139677 - /cfe/trunk/lib/Driver/Driver.cpp

[Chad Rosier]

On Jun 12, 2013, at 7:23 AM, Rafael Espíndola <rafael.espindola at gmail.com> wrote:

>> [driver] Add support for the COMPILER_PATH environment variable, which adds the
>> specified path(s) to the list of prefix directories.
>> rdar://10097714
>> 
> 
> Sorry for the really late review, but why this environment variable
> instead of implementing the -B command line option?

When Apple made the transition from llvm-gcc to clang I added this to be backward compatible with
llvm-gcc and gcc.  Nick was using this feature to select the appropriate linker for testing.  The -B option
was added by Chandler around r99159 and then extended by Joerg in r125111, so I'm not sure why
we didn't just use it then..

Are you proposing we deprecate this feature?  I could be easily convinced either way, but would
prefer we leave it in place to maintain backward compatibility.  Also, there's really no maintenance
cost with leaving it in place.

  Chad

> Cheers,
> Rafael

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130614/128f17a3/attachment.html>