[cfe-dev] [patch] Add configure option for C include search path

[Chandler Carruth]

On Wed, Nov 11, 2009 at 11:10 AM, Daniel Dunbar <daniel at zuster.org> wrote:
> On Wed, Nov 11, 2009 at 10:55 AM, Rafael Espindola <espindola at google.com> wrote:
>>> I completely agree. All I meant was that we shouldn't try to solve the
>>> general problem in this fashion. This is certainly a useful option,
>>> and once the -target world comes about it will just construct a custom
>>> configuration behind the scenes.
>>
>> Sure. The only two issues were were discussing were the C and C++
>> configure option format and those should probably remain the same.
>>
>> For C++ I have to first implement the StringRef version of
>> SplitString, but is the C option OK? I promise to update its use of
>> StringRef :-)
>
> The C option looks fine to me. The C++ option I think is too
> complicated, its trying to solve a small subset of the multilib
> problem and I think thats a bad idea, I think it should work exactly
> like the C option. Anything beyond that is the realm of something more
> sophisticated, I think.

However, this is a subset of the multilib problem we need solved
today. =/ I think it might be worth going this far (if no farther)
even before the general solution arrives. If you have a better
structure or approach to solve at least this much, I'm all ears. I
certainly agree this is far from simple or ideal.
-Chandler

>
>  - Daniel
>
>>>  - Daniel
>>
>> Cheers,
>> --
>> Rafael Ávila de Espíndola
>>
>