On Jun 12, 2013, at 7:23 AM, Rafael Espíndola <[email protected]> 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
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
