[cfe-commits] [patch] Add a runtime option to match --with-gcc-toolchain
Rafael Ávila de Espíndola
rafael.espindola at gmail.com
Sat Feb 18 17:44:31 PST 2012
On 18/02/12 04:44 PM, Chandler Carruth wrote:
> Sorry for delay.... =[
>
> Anyways, this looks really cool. A few super minor points. Only the
> first one really matters, check in whenever.
r150898.
> 1) Can we get a name without the 'ccc' in it? This is something that I
> think users might actually need when cross compiling, so much like
> '-target' I'd like it to be an exposed and supported aspect of the
> driver. --gcc-toolchain (or a fewer number of dashes as folks like)
> seems fine.
I went with -gcc-toolchain to match -target.
> 2) Is there no PathV2 routine that can remove extraneous '/'s from a
> path? Maybe we should add one if not...
I couldn't find anything that worked with StringRef.
> 3) Should we instead of adding a new test, just fold these cases into
> the existing header search and ld invocation tests? Meh, either way,
> it's not a big deal.
Cheers,
Rafael
More information about the cfe-commits
mailing list